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

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


A few whitespace errors (tab following spaces) and one indenting
mistake.  Then I think it's good to go.




http://codereview.appspot.com/150044/diff/11/1015
File ly/music-functions-init.ly (right):

http://codereview.appspot.com/150044/diff/11/1015#newcode611
ly/music-functions-init.ly:611: (filter
Tab following space -- we never want to have that.  Start the line with
as many tabs as desired, followed by spaces if necessary to create the
proper indentation.  Or use all spaces -- that's fine too.  But not
space tab space.

http://codereview.appspot.com/150044/diff/11/1015#newcode612
ly/music-functions-init.ly:612: (lambda (m) (eq? 'NoteEvent
(ly:music-property m 'name)))
Tab following space

http://codereview.appspot.com/150044/diff/11/1015#newcode613
ly/music-functions-init.ly:613: (ly:music-property ev-chord
'elements))))
Tab following space

http://codereview.appspot.com/150044/diff/11/1015#newcode624
ly/music-functions-init.ly:624: (for-each (lambda (m)
(for-each should align with (ly:pitch?.

(if (test-expression)
    (true-expression)
    (false-expression))

http://codereview.appspot.com/150044/diff/11/1015#newcode626
ly/music-functions-init.ly:626: (begin
After changing for-each, you'll need to change (begin to match.

http://codereview.appspot.com/150044/diff/11/1015#newcode633
ly/music-functions-init.ly:633: trill-events)))))
Tab following spaces

http://codereview.appspot.com/150044

---
----
Join the Frogs!


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