On 27 févr. 2013, at 07:06, k-ohara5...@oco.net wrote: > Also needs some decruftification. > > > https://codereview.appspot.com/7304068/diff/22001/lily/align-interface.cc > File lily/align-interface.cc (right): > > https://codereview.appspot.com/7304068/diff/22001/lily/align-interface.cc#newcode299 > lily/align-interface.cc:299: vector<Grob *> non_empty_elems; > The disassembly and reassembly of the array is ridiculous, but with the > comment at least people will not be confused, merely laugh. > > https://codereview.appspot.com/7304068/diff/22001/scm/define-grobs.scm > File scm/define-grobs.scm (right): > > https://codereview.appspot.com/7304068/diff/22001/scm/define-grobs.scm#newcode2031 > scm/define-grobs.scm:2031: (horizon-padding . 100) > You don't need this anymore, and it will terribly confuse some poor > songwriter somewhere, if left in.
Removed. Thanks for the review! Cheers, MS _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel
