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

Issue 7798045: Add warning about using \relative with tagged music (3253) (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 1 month ago by Trevor Daniels
Modified:
11 years ago
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Add warning about using \relative with tagged music (3253) - add warning as a known issue - remove earlier ignored known issue

Patch Set 1 #

Total comments: 3

Patch Set 2 : Respond to comments from David and Keith #

Total comments: 1

Patch Set 3 : Repond to Janek's comment #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -9 lines) Patch
M Documentation/notation/input.itely View 1 2 1 chunk +7 lines, -9 lines 0 comments Download

Messages

Total messages: 7
dak
https://codereview.appspot.com/7798045/diff/1/Documentation/notation/input.itely File Documentation/notation/input.itely (right): https://codereview.appspot.com/7798045/diff/1/Documentation/notation/input.itely#newcode2247 Documentation/notation/input.itely:2247: @end example That's \relative without reference pitch... This particular ...
11 years, 1 month ago (2013-03-18 18:37:37 UTC) #1
Keith
I'm confused by the new text. I also fail to see the motivation for explaining ...
11 years, 1 month ago (2013-03-19 08:53:07 UTC) #2
Trevor Daniels
Respond to comments from David and Keith
11 years ago (2013-04-07 21:20:37 UTC) #3
Graham Percival
LGTM
11 years ago (2013-04-09 07:23:43 UTC) #4
janek
generally LGTM https://codereview.appspot.com/7798045/diff/5001/Documentation/notation/input.itely File Documentation/notation/input.itely (right): https://codereview.appspot.com/7798045/diff/5001/Documentation/notation/input.itely#newcode2246 Documentation/notation/input.itely:2246: Calling @code{\relative} on a sequence obtained by ...
11 years ago (2013-04-13 14:14:57 UTC) #5
Trevor Daniels
Repond to Janek's comment
11 years ago (2013-04-13 17:38:50 UTC) #6
Trevor Daniels
11 years ago (2013-04-14 21:41:23 UTC) #7
Pushed to staging as
ee0ef98a35634718033d505deb8a355e05513d6a
Closing ...
Sign in to reply to this message.

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