lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Add scripts/auxiliar/guileindent.scm (issue 4896043)


From: tdanielsmusic
Subject: Re: Add scripts/auxiliar/guileindent.scm (issue 4896043)
Date: Sun, 14 Aug 2011 07:48:48 +0000

L pretty GTM.
It would look even better with a little manual adjustment first.


http://codereview.appspot.com/4896043/diff/1/scm/music-functions.scm
File scm/music-functions.scm (right):

http://codereview.appspot.com/4896043/diff/1/scm/music-functions.scm#newcode673
scm/music-functions.scm:673: (make-music 'RestEvent 'duration
(ly:music-property mus 'duration))
remove the double space, I think

http://codereview.appspot.com/4896043/diff/1/scm/music-functions.scm#newcode973
scm/music-functions.scm:973: )))
Is it out of the question to move orphaned brackets to the previous
line?

http://codereview.appspot.com/4896043/diff/1/scm/music-functions.scm#newcode1470
scm/music-functions.scm:1470: (lambda (m) (eq? 'NoteEvent
(ly:music-property m 'name)))
Same question for bare (if and (list
I'd be happy with 1 space in this case and 2 spaces when the function is
not bare [as indent seems to do for all bare (function calls].  Don't
know what emacs does though.

http://codereview.appspot.com/4896043/



reply via email to

[Prev in Thread] Current Thread [Next in Thread]