Done - attached is a fresh patch set. The first three are what's already on Rietveld, and the 4th is Carl's formatting & copyright changes.
Cheers, MS
0004-Changes-suggested-by-Carl.patch
Description: Binary data
0003-Intermediary-37-patch.patch
Description: Binary data
0002-Fixes-formatting-problems.patch
Description: Binary data
0001-Fixing-issue-37-with-extra-position-callback.patch
Description: Binary data
On Jan 8, 2011, at 9:40 PM, [email protected] wrote: > 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
