http://codereview.appspot.com/2313044/diff/15001/scm/ly-syntax-constructors.scm
File scm/ly-syntax-constructors.scm (right):

http://codereview.appspot.com/2313044/diff/15001/scm/ly-syntax-constructors.scm#newcode20
scm/ly-syntax-constructors.scm:20: (defmacro define-ly-syntax (args .
body)
On 2010/11/04 21:21:09, Valentin Villenave wrote:
Is there a way to document defmacros with docstrings instead of
comments?

I don't know.  The code as is fixes the crash when compiling this stuff
using Guile V1.9 and that's the aim of the patch.
I suspect the whole file could be re-written using (define-syntax) and
(syntax-rules), but I don't have enough scheme-fu and thinking about it
made my head hurt.
Ian

http://codereview.appspot.com/2313044/diff/15001/scm/ly-syntax-constructors.scm#newcode39
scm/ly-syntax-constructors.scm:39: 'location
On 2010/11/04 21:21:09, Valentin Villenave wrote:
Are you using Emacs-like indentation? "tabs for indentation, spaces
for
alignment"; it seems that the rest of the file is indented that way.
By the way
isn't there a space char missing here?

Done.

http://codereview.appspot.com/2313044/

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

Reply via email to