http://codereview.appspot.com/4875054/diff/1/lily/ambitus-engraver.cc File lily/ambitus-engraver.cc (right):
http://codereview.appspot.com/4875054/diff/1/lily/ambitus-engraver.cc#newcode177 lily/ambitus-engraver.cc:177: Rational sig_alter = !scm_is_false (handle) On 2011/08/18 14:03:06, Cécile Hauchemaille wrote:
So, might scm_is_pair also work ?
Yes, of course, that's the correct check... http://codereview.appspot.com/4875054/diff/1/lily/auto-beam-engraver.cc File lily/auto-beam-engraver.cc (right): http://codereview.appspot.com/4875054/diff/1/lily/auto-beam-engraver.cc#newcode172 lily/auto-beam-engraver.cc:172: return !scm_is_false (scm_call_4 (get_property ("autoBeamCheck"), On 2011/08/18 13:31:26, Bertrand Bordage wrote:
Indent like this:
return !scm_is_false (scm_call_4 ( get_property ("autoBeamCheck"), context ()->self_scm (), [...]
Or more correct: align the context ()->self_scm () with the get_property: return !scm_is_false (scm_call_4 (get_property ("autoBeamCheck"), context ()->self_scm (), ... http://codereview.appspot.com/4875054/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel