New patch set uploaded.  Current Lily default behaviour is kept as
default now.


http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc
File lily/stem-tremolo.cc (right):

http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc#newcode42
lily/stem-tremolo.cc:42: style = ly_symbol2scm ("constant");
On 2011/07/21 08:39:53, joeneeman wrote:
You can remove these two lines and use
style == ly_symbol2scm ("varying")
below

Done.

http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc#newcode93
lily/stem-tremolo.cc:93: style = ly_symbol2scm ("constant");
On 2011/07/21 08:39:53, joeneeman wrote:
These two lines are redundant.

Isn't their purpose to guard against undefined (empty) style property?

http://codereview.appspot.com/4636081/

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

Reply via email to