[Top][All Lists]
[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/