LGTM. Don't forget to fix your copyright on beam-collision-engraver.
One set of braces to be removed. Thanks, Carl http://codereview.appspot.com/3928041/diff/17001/lily/beam-collision-engraver.cc File lily/beam-collision-engraver.cc (right): http://codereview.appspot.com/3928041/diff/17001/lily/beam-collision-engraver.cc#newcode4 lily/beam-collision-engraver.cc:4: Copyright (C) 1997--2010 Han-Wen Nienhuys <[email protected]> Copyright 2011 Mike Solomon http://codereview.appspot.com/3928041/diff/17001/lily/beam.cc File lily/beam.cc (right): http://codereview.appspot.com/3928041/diff/17001/lily/beam.cc#newcode969 lily/beam.cc:969: { No brackets here http://codereview.appspot.com/3928041/ _______________________________________________ lilypond-devel mailing list [email protected] http://lists.gnu.org/mailman/listinfo/lilypond-devel
