Are you still waiting for someone to review this? If so, here are a
couple minor things:


http://codereview.appspot.com/1689041/diff/2001/3001
File scm/define-markup-commands.scm (right):

http://codereview.appspot.com/1689041/diff/2001/3001#newcode848
scm/define-markup-commands.scm:848: X RIGHT fill-space line-stencils)))
These two set!s could be written more concisely as
(set! line-stencils
  (stack-stencils-padding-list
   X RIGHT fill-space empty-stencil)))

http://codereview.appspot.com/1689041/diff/2001/3001#newcode850
scm/define-markup-commands.scm:850: (if (> word-count 1)
I know there aren't many comments in the code, but that doesn't mean you
can't add one...

http://codereview.appspot.com/1689041/show

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

Reply via email to