Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(917)

Issue 9964043: Use @dots{} instead of ... in notation manual where appropriate (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 11 months ago by dak
Modified:
10 years, 9 months ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Use @dots{} instead of ... in notation manual where appropriate This uses a typographically preferable variant, and also may render better in Info format in some circumstances.

Patch Set 1 #

Total comments: 17

Patch Set 2 : Also treat instances of .. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+56 lines, -56 lines) Patch
M Documentation/notation/ancient.itely View 3 chunks +3 lines, -3 lines 0 comments Download
M Documentation/notation/changing-defaults.itely View 1 5 chunks +8 lines, -8 lines 0 comments Download
M Documentation/notation/cheatsheet.itely View 1 chunk +1 line, -1 line 0 comments Download
M Documentation/notation/chords.itely View 2 chunks +4 lines, -4 lines 0 comments Download
M Documentation/notation/input.itely View 1 24 chunks +32 lines, -32 lines 0 comments Download
M Documentation/notation/repeats.itely View 1 chunk +1 line, -1 line 0 comments Download
M Documentation/notation/rhythms.itely View 1 1 chunk +1 line, -1 line 0 comments Download
M Documentation/notation/simultaneous.itely View 4 chunks +6 lines, -6 lines 0 comments Download

Messages

Total messages: 7
Jean-Charles
There is also notation/rhythms.itely:2320:If a manual beam is specified with @code{[..]} set the beam For ...
10 years, 11 months ago (2013-06-04 18:17:21 UTC) #1
dak
Also treat instances of ..
10 years, 11 months ago (2013-06-04 18:30:59 UTC) #2
dak
https://codereview.appspot.com/9964043/diff/1/Documentation/notation/simultaneous.itely File Documentation/notation/simultaneous.itely (right): https://codereview.appspot.com/9964043/diff/1/Documentation/notation/simultaneous.itely#newcode49 Documentation/notation/simultaneous.itely:49: @funindex <...> On 2013/06/04 18:17:21, Jean-Charles wrote: > @funindex ...
10 years, 11 months ago (2013-06-04 18:33:41 UTC) #3
Jean-Charles
On 2013/06/04 18:33:41, dak wrote: > https://codereview.appspot.com/9964043/diff/1/Documentation/notation/simultaneous.itely#newcode49 > Documentation/notation/simultaneous.itely:49: @funindex <...> > On 2013/06/04 18:17:21, ...
10 years, 11 months ago (2013-06-04 18:36:06 UTC) #4
dak
lilyfan@orange.fr writes: > There is also > > notation/rhythms.itely:2320:If a manual beam is specified with ...
10 years, 11 months ago (2013-06-05 10:45:26 UTC) #5
Graham Percival
I find ... easier to read in texinfo source rather than @dots{}, but oh well. ...
10 years, 11 months ago (2013-06-06 21:31:40 UTC) #6
dak
10 years, 11 months ago (2013-06-06 23:40:55 UTC) #7
On 2013/06/06 21:31:40, Graham Percival wrote:
> I find ... easier to read in texinfo source rather than @dots{}, but oh well. 
I
> won't object to the patch.

Well, you'll see that I did not change any ... inside of a comment or @ignore
section.  Which is where ... is only in the Texinfo source.  So where _only_
source readability is concerned, I'm in concord with your desire.

But Texinfo is not a markdown language.  It is not intended as the normal form
of reading a document.  The readability of the target format takes priority.  I
have to admit that the actual motivation was Info getting confused about ... in
connection with a cross reference.  But the change nevertheless makes sense as
it results in visual consistency.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b