lilypond-devel
[Top][All Lists]
Advanced

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

Re: Adds support for cross-staff-stems (issue 6344092)


From: dak
Subject: Re: Adds support for cross-staff-stems (issue 6344092)
Date: Thu, 30 Aug 2012 23:12:46 +0000


http://codereview.appspot.com/6344092/diff/12002/scm/music-functions.scm
File scm/music-functions.scm (right):

http://codereview.appspot.com/6344092/diff/12002/scm/music-functions.scm#newcode1784
scm/music-functions.scm:1784: (if (<= 2 (length stems))
Why not use "and" here and leave out the #f below?  Then one does not
hunt for #f to figure out what happens when the condition is false.

http://codereview.appspot.com/6344092/diff/12002/scm/music-functions.scm#newcode1801
scm/music-functions.scm:1801: (let ((span (ly:engraver-make-grob trans
'Stem '())))
Why not have "root" as the cause, the last element of
ly:engraver-make-grob?  That would make the stem span tweakable.

http://codereview.appspot.com/6344092/



reply via email to

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