Don't know why two identical patch-sets were uploaded
On 2013/02/02 16:11:19, dak wrote:
https://codereview.appspot.com/7231072/diff/1/scm/define-markup-commands.scm
File scm/define-markup-commands.scm (right):
https://codereview.appspot.com/7231072/diff/1/scm/define-markup-commands.scm#newcode3366
scm/define-markup-commands.scm:3366: (string-append "flags."
I'd rather use
(format #f (if ancient-flags? "flags.mensural~a2~a"
"flags.~a~a")
(if (> dir 0) "u" "d") log)
here. It is somewhat more readable.
Changed.
https://codereview.appspot.com/7231072/diff/1/scm/define-markup-commands.scm#newcode3387
scm/define-markup-commands.scm:3387: (or (eq? flag-style 'default)
(list?
flag-style))
Why (list? flag-style)? Is that a tricky obfuscation of (null?
flag-style)?
Rather bad thinking.
Changed.
https://codereview.appspot.com/7231072/
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
