LGTM. I haven't run the regtests, though, to make sure there are no differences.
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) As we are calling cdr on the handle, it would probably be better to check for a list using ly_is_list(handle) http://codereview.appspot.com/4875054/diff/1/lily/bar-check-iterator.cc File lily/bar-check-iterator.cc (right): http://codereview.appspot.com/4875054/diff/1/lily/bar-check-iterator.cc#newcode52 lily/bar-check-iterator.cc:52: if (to_boolean (tr->get_property ("ignoreBarChecks"))) scm_is_true http://codereview.appspot.com/4875054/ _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel