[frogs] Re: Tracker 836: Add facility to change output file-name for a \book block

[ Thread Index | Date Index | More lilynet.net/frogs Archives ]



http://codereview.appspot.com/143055/diff/19/1020
File scm/lily-library.scm (right):

http://codereview.appspot.com/143055/diff/19/1020#newcode99
Line 99: (for-each (lambda (symbol)
I would recommend indenting like so:

  (for-each
    (lambda (symbol)
      (let ((..

http://codereview.appspot.com/143055/diff/19/1020#newcode142
Line 142: (if (not book-filename)
The if block should be indented two more spaces here.

http://codereview.appspot.com/143055/diff/19/1020#newcode143
Line 143: (ly:parser-output-name parser)
These two lines should begin in the same column as (not ...)

http://codereview.appspot.com/143055/diff/19/1020#newcode149
Line 149: (let ((book-output-suffix (ly:parser-lookup parser
'book-output-suffix)))
indent the if block

http://codereview.appspot.com/143055/diff/19/1020#newcode151
Line 151: (ly:parser-lookup parser 'output-suffix)
align these with (not ...)

http://codereview.appspot.com/143055

---

----
Join the Frogs!


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/