One minor comment, otherwise lgtm
http://codereview.appspot.com/32148/diff/1/7 File lily/spanner.cc (right): http://codereview.appspot.com/32148/diff/1/7#newcode426 Line 426: Spanner::after_line_breaking (SCM grob) Can you think of a name that describes what the function actually does? Like Spanner::suicide_if_spanned_time_is_empty, but maybe less verbose... http://codereview.appspot.com/32148 _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel