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

Issue 8187044: Add Changes entries for \temporary, \omit, \hide, \single, multiple tags (Closed)

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

Description

Add Changes entries for \temporary, \omit, \hide, \single, multiple tags

Patch Set 1 #

Total comments: 4

Patch Set 2 : Address Thomas' comments #

Unified diffs Side-by-side diffs Delta from patch set Stats (+51 lines, -0 lines) Patch
M Documentation/changes.tely View 1 1 chunk +51 lines, -0 lines 0 comments Download

Messages

Total messages: 6
janek
LGTM
11 years, 1 month ago (2013-03-30 18:17:31 UTC) #1
thomasmorley65
Some nitpicks. Otherwise LGTM https://codereview.appspot.com/8187044/diff/1/Documentation/changes.tely File Documentation/changes.tely (right): https://codereview.appspot.com/8187044/diff/1/Documentation/changes.tely#newcode73 Documentation/changes.tely:73: Two ways of letting graphical ...
11 years, 1 month ago (2013-03-30 19:25:04 UTC) #2
dak
I'll address the other points presently, or on Tuesday, depending on my connectivity. https://codereview.appspot.com/8187044/diff/1/Documentation/changes.tely File ...
11 years, 1 month ago (2013-03-30 19:36:03 UTC) #3
dak
Address Thomas' comments
11 years, 1 month ago (2013-03-30 19:38:09 UTC) #4
thomasmorley65
On 2013/03/30 19:38:09, dak wrote: > Address Thomas' comments The description doesn't explain why \relative ...
11 years, 1 month ago (2013-03-30 20:11:06 UTC) #5
Trevor Daniels
11 years, 1 month ago (2013-04-01 21:53:30 UTC) #6
LGTM

Trevor
Sign in to reply to this message.

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