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

Issue 177060043: Convert ly::time-signature::print from C++ to Scheme. (Closed)

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

Description

Convert ly::time-signature::print from C++ to Scheme. NOTE: This is not in good condition for review. I'm only posting it to provide some context for the patch in issue 4196. The user may add a new time signature style by calling add-simple-time-signature-style and providing a procedure returning markup. The markup command \numbered-time-signature formats a variety of numeric time signatures.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+229 lines, -237 lines) Patch
A input/regression/markup-time-signature.ly View 1 chunk +29 lines, -0 lines 0 comments Download
M input/regression/time-signature-mensural.ly View 1 chunk +4 lines, -4 lines 0 comments Download
A input/regression/time-signature-user-defined.ly View 1 chunk +22 lines, -0 lines 0 comments Download
D lily/include/time-signature.hh View 1 chunk +0 lines, -41 lines 0 comments Download
M lily/slur-scoring.cc View 2 chunks +1 line, -2 lines 0 comments Download
D lily/time-signature.cc View 1 chunk +0 lines, -164 lines 0 comments Download
M lily/time-signature-engraver.cc View 1 chunk +0 lines, -1 line 0 comments Download
M ly/music-functions-init.ly View 1 chunk +1 line, -1 line 0 comments Download
M scm/define-grob-interfaces.scm View 1 chunk +18 lines, -0 lines 0 comments Download
M scm/define-markup-commands.scm View 2 chunks +19 lines, -0 lines 0 comments Download
M scm/lily.scm View 1 chunk +1 line, -0 lines 0 comments Download
A scm/time-signature.scm View 1 chunk +40 lines, -0 lines 0 comments Download
M scm/time-signature-settings.scm View 2 chunks +94 lines, -24 lines 0 comments Download

Messages

Total messages: 1
Dan Eble
9 years, 4 months ago (2014-11-19 12:47:07 UTC) #1

          
Sign in to reply to this message.

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