|
From: | joeneeman |
Subject: | Re: modifying default behaviour of tremolo slashes (issue4636081) |
Date: | Sun, 24 Jul 2011 07:11:41 +0000 |
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#newcode93 lily/stem-tremolo.cc:93: style = ly_symbol2scm ("constant"); On 2011/07/23 19:58:19, Janek Warchol wrote:
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?
If style is empty (or not a symbol) then (style == ly_symbol2scm ("varying")) will be false and you'll just return 'default. http://codereview.appspot.com/4636081/
[Prev in Thread] | Current Thread | [Next in Thread] |