On Apr 8, 2011, at 5:27 PM, n.putt...@gmail.com wrote:

> LGTM, just needs a regtest.
> 
> 
> http://codereview.appspot.com/4384046/diff/5001/lily/page-spacing.cc
> File lily/page-spacing.cc (right):
> 
> http://codereview.appspot.com/4384046/diff/5001/lily/page-spacing.cc#newcode88
> lily/page-spacing.cc:88: return footnote_height
> (footnote_height
> 
> to preserve indent
> 
> http://codereview.appspot.com/4384046/diff/5001/ly/paper-defaults-init.ly
> File ly/paper-defaults-init.ly (right):
> 
> http://codereview.appspot.com/4384046/diff/5001/ly/paper-defaults-init.ly#newcode111
> ly/paper-defaults-init.ly:111: footnote-footer-padding = 0.5\mm
> add to `set-paper-dimension-variables' (same for footnote-padding)
> otherwise it's 0.5 staff-space

Thanks!
Pushed as c676b77a6e685d6efbbeb358f8488725fbab34d2

Cheers,
MS
_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to