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

Issue 173260043: Issue 4196: Add \compound-meter markup command (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 5 months ago by Dan Eble
Modified:
9 years, 4 months ago
Reviewers:
dak
CC:
lilypond-devel_gnu.org
Visibility:
Public.

Description

Add \compound-meter markup command This command can produce compound, simple, and single-digit time signatures using minor extensions of the list format that \compoundMeter uses. In this patch, compound time signature formatting relies on this command. In the future, all numeric time signatures may rely on it.

Patch Set 1 #

Patch Set 2 : Rename to \compound-meter #

Unified diffs Side-by-side diffs Delta from patch set Stats (+75 lines, -14 lines) Patch
A input/regression/markup-compound-meter.ly View 1 1 chunk +28 lines, -0 lines 0 comments Download
M ly/music-functions-init.ly View 1 1 chunk +1 line, -1 line 0 comments Download
M scm/time-signature-settings.scm View 1 2 chunks +46 lines, -13 lines 0 comments Download

Messages

Total messages: 7
Dan Eble
New patch for review. The notation reference has a black rectangle at the ends of ...
9 years, 5 months ago (2014-11-15 17:34:41 UTC) #1
dak
On 2014/11/15 17:34:41, Dan Eble wrote: > New patch for review. > > The notation ...
9 years, 5 months ago (2014-11-15 17:41:54 UTC) #2
dak
On 2014/11/15 17:41:54, dak wrote: > On 2014/11/15 17:34:41, Dan Eble wrote: > > New ...
9 years, 5 months ago (2014-11-15 17:43:48 UTC) #3
Dan Eble
On 2014/11/15 17:43:48, dak wrote: > Oops. It's a markup command. Have to think about ...
9 years, 5 months ago (2014-11-15 19:49:53 UTC) #4
Dan Eble
The whole c++-to-scheme work is in https://codereview.appspot.com/177060043/ . There is some cruft in it that ...
9 years, 5 months ago (2014-11-19 13:05:44 UTC) #5
dak
On 2014/11/19 13:05:44, Dan Eble wrote: > The whole c++-to-scheme work is in https://codereview.appspot.com/177060043/ . ...
9 years, 5 months ago (2014-11-19 14:14:20 UTC) #6
Dan Eble
9 years, 5 months ago (2014-11-19 23:05:39 UTC) #7
Rename to \compound-meter
Sign in to reply to this message.

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