http://codereview.appspot.com/4490045/diff/12001/lily/completion-note-heads-engraver.cc File lily/completion-note-heads-engraver.cc (right):
http://codereview.appspot.com/4490045/diff/12001/lily/completion-note-heads-engraver.cc#newcode207 lily/completion-note-heads-engraver.cc:207: event->set_property("autosplit-end", ly_bool2scm (false)); The style guide in Lilypond Contributor's Guide (http://lilypond.org/doc/v2.13/Documentation/contributor/code-style) does not cover the question. However, some people consider inline conditions not to be very readable (for a (Java) example see AvoidInlineConditionals at http://checkstyle.sourceforge.net/config_coding.html). Although this is not an extreme case, I'd rather vote for readability and leave the code as it is. http://codereview.appspot.com/4490045/ _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
