2011/5/8 Trevor Daniels <t.dani...@treda.co.uk> > > lemniskata.bernoull...@gmail.com wrote Sunday, May 08, 2011 7:17 AM > >> Style nitpick. > > >> http://codereview.appspot.com/4489042/diff/1/lily/side-position-interface.cc#newcode65 >> lily/side-position-interface.cc:65: bool include_staff = >> I see quite a lot of whitespace diffs here and there. IIRC we care about >> them, so please fix :) >> > > If you look more carefully you'll see these changes > are actually correcting whitespace errors in the > original code.
Ooops! Sorry :) I remember that someone once told me that such canges should be done separately from the actual coding stuff - that's why i thought your changes were accidental (i didn't examine them closely indeed). Honestly i'm not sure if this policy of separating style fixes and coding is good. Surely it's better to have these separated, but this requires additional time and work. I found this a little discouraging - style fixes like that are best done 'by the way'. > But thanks for taking the trouble > to review my fix :) I thought it's worth a try since we have too few reviewers. Too bad that i didn't say anything useful :) cheers, Janek
_______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
