lilypond-devel
[Top][All Lists]
Advanced

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

Re: Fix #1205. (issue2145047)


From: n . puttock
Subject: Re: Fix #1205. (issue2145047)
Date: Mon, 07 Feb 2011 21:27:34 +0000

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/



reply via email to

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