[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Fix 1198: Fix displayLilyMusic for \time and #(set-time-signature) (

From: n . puttock
Subject: Re: Fix 1198: Fix displayLilyMusic for \time and #(set-time-signature) (issue1986042)
Date: Fri, 13 Aug 2010 21:25:12 +0000

Hi Carl,


File scm/define-music-display-methods.scm (right):
scm/define-music-display-methods.scm:887: (let* ((arguments
(ly:music-property expr 'time-signature-arguments))
I'd prefer separate properties since it would make the output of
\displayMusic less cryptic.

There's already 'numerator and 'denominator (which are currently used
for \times, even though their docstrings mention \time).
scm/define-music-display-methods.scm:897: "#(set-time-signature ~a ~a
"#(set-time-signature ~a ~a '~a)~a"
File scm/define-music-types.scm (right):
scm/define-music-types.scm:53: (define (make-time-signature-set music)
We should probably think about moving these to a separate file now
there's more than one callback.

(naturally this can wait for another patch)
File scm/music-functions.scm (left):
scm/music-functions.scm:507: (define-public
(make-beam-rule-time-signature-set num den rest)
Do you want to keep this?

reply via email to

[Prev in Thread] Current Thread [Next in Thread]