http://codereview.appspot.com/144049/diff/1/2#newcode15 Line 15: binding-offset = 5 \mm I think this deserves a separate test, otherwise it just looks the same as an ordinary page with slightly bigger but equal margins.
Ok...
http://codereview.appspot.com/144049/diff/1/3 File lily/output-def.cc (right): http://codereview.appspot.com/144049/diff/1/3#newcode173 Line 173: - left_margin_default You'll have to add parentheses if you want to keep this indentation.
I don't know why I changed the old indentation... But why doesn't this work? It at least compiles fine... ;)
http://codereview.appspot.com/144049/diff/1/4 File ly/paper-defaults-init.ly (right): http://codereview.appspot.com/144049/diff/1/4#newcode90 Line 90: twoside = ##f I'm not too fond of this name, but can't think of anything better. :)
LaTeX exactly uses this term, so I didn't much think about it. Anyway, I followed your suggestions.
http://codereview.appspot.com/144049/diff/1/5 File scm/page.scm (right): http://codereview.appspot.com/144049/diff/1/5#newcode320 Line 320: (ly:stencil-translate page-stencil (cons (prop 'right-margin) 0)))) You could simplify this a bit: (if (ly:output-def-lookup layout 'twoside #f) (set! page-stencil (ly:stencil-translate page-stencil (cons (prop (if (even? number) 'left-margin 'right-margin)) 0)))
Done.
http://codereview.appspot.com/144049/diff/1/6 File scm/paper.scm (right): http://codereview.appspot.com/144049/diff/1/6#newcode242 Line 242: (("foot-separation" . ,h) - '()) oops ;)
Argh... fixed. Regards, Michael _______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel
