On 2011/02/06 21:13:34, nicolas.sceaux wrote:

http://codereview.appspot.com/2145047/diff/6001/scm/define-music-display-methods.scm
File scm/define-music-display-methods.scm (right):


http://codereview.appspot.com/2145047/diff/6001/scm/define-music-display-methods.scm#newcode927
scm/define-music-display-methods.scm:927: (format #f "\\tempo
~{~a~a~}~a = ~a~a"
The indentation is wrong.
The usual indentation with this kind of macro is:
(with-xxx something
   body) ; two spaces
or:
(with-xxx
     something ; four spaces
   body) ; two spaces
so that the special 'something' argument is not confused with the
regular body.

For the same reason, the new indentation of "\\melismaEnd" on line 915
is
incorrect.

Thanks, fixed.

I relied on the default indentation provided by emacs, but I've added
the following to my .emacs file to make sure:

(put 'with-music-match 'scheme-indent-function 1)

Cheers,
Neil

http://codereview.appspot.com/2145047/

_______________________________________________
lilypond-devel mailing list
lilypond-devel@gnu.org
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to