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.

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)?

https://codereview.appspot.com/7231072/

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

Reply via email to