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