LGTM, although I have no knowledge of the spacing code.
http://codereview.appspot.com/2316042/diff/1/Documentation/notation/spacing.itely File Documentation/notation/spacing.itely (right): http://codereview.appspot.com/2316042/diff/1/Documentation/notation/spacing.itely#newcode198 Documentation/notation/spacing.itely:198: @stro...@i{fixed vertical dimensions}} Please use @subsubheading for this kind of stuff. In the long term, we might even make "Vertical dimensions" into a @subsection, then make things like "Fixed vertical dimensions", "Flexible vertical dimensions", etc., into @unnumberedsubsubsec. But for this first patch, I'd rather keep on focusing on documenting the behaviour rather than playing games with texinfo sectioning. :) http://codereview.appspot.com/2316042/ _______________________________________________ lilypond-devel mailing list [email protected] http://lists.gnu.org/mailman/listinfo/lilypond-devel
