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

Issue 8538050: Doc: expand explanation of negative measurePosition (3080) (Closed)

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

Description

Doc: expand explanation of negative measurePosition (3080) based on text provided by Federico Bruni

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+8 lines, -7 lines) Patch
M Documentation/notation/rhythms.itely View 2 chunks +8 lines, -7 lines 3 comments Download

Messages

Total messages: 6
Graham Percival
LGTM
11 years ago (2013-04-21 08:36:45 UTC) #1
Jean-Charles
https://codereview.appspot.com/8538050/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): https://codereview.appspot.com/8538050/diff/1/Documentation/notation/rhythms.itely#newcode1422 Documentation/notation/rhythms.itely:1422: is usually positive and indicates how much of the ...
11 years ago (2013-04-21 09:25:48 UTC) #2
Trevor Daniels
Thanks for the review! https://codereview.appspot.com/8538050/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): https://codereview.appspot.com/8538050/diff/1/Documentation/notation/rhythms.itely#newcode1422 Documentation/notation/rhythms.itely:1422: is usually positive and indicates ...
11 years ago (2013-04-21 12:15:23 UTC) #3
Jean-Charles
https://codereview.appspot.com/8538050/diff/1/Documentation/notation/rhythms.itely File Documentation/notation/rhythms.itely (right): https://codereview.appspot.com/8538050/diff/1/Documentation/notation/rhythms.itely#newcode1422 Documentation/notation/rhythms.itely:1422: is usually positive and indicates how much of the ...
11 years ago (2013-04-21 12:28:43 UTC) #4
janek
LGTM
11 years ago (2013-04-23 13:20:48 UTC) #5
Trevor Daniels
11 years ago (2013-04-24 07:45:47 UTC) #6
Pushed to staging as
092b8418e76ef17d6ca2b8a0521b98b1c1afc3af
Closing ...
Sign in to reply to this message.

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