https://codereview.appspot.com/97110045/diff/80001/ly/music-functions-init.ly
File ly/music-functions-init.ly (right):

https://codereview.appspot.com/97110045/diff/80001/ly/music-functions-init.ly#newcode144
ly/music-functions-init.ly:144: #{ \once\override
Staff.TimeSignature.stencil =
On 2014/05/10 20:49:09, janek wrote:
Shouldn't this override be at Score level?

Any other ideas?
I have the impression that Staff. works just as well?

https://codereview.appspot.com/97110045/diff/80001/ly/music-functions-init.ly#newcode149
ly/music-functions-init.ly:149: #(map (lambda (x) #{ \markup
\center-column #(map number->string x) #})
On 2014/05/10 20:49:08, janek wrote:
This assumes that the user will input lists of 2 elements.  What if
the user
enters just one number, or more than two, for example
\alternatingTimeSignatures
#'((6 1 8)(5)) ?  The function should either handle such input
gracefully or
emit a warning/error.

Entering more than two numbers seems gracefully enough:
A column of the given numbers will be created and the first two are used
for the effective time. The result is self-explaining IMO.

But entering only one number will actually fail with an error that
doesn't point the user to the right place or cause. So I agree that this
should be improved.

https://codereview.appspot.com/97110045/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to