lilypond-devel
[Top][All Lists]
Advanced

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

Re: Add new merge-function.ly file for rests (issue4005046)


From: percival . music . ca
Subject: Re: Add new merge-function.ly file for rests (issue4005046)
Date: Sun, 23 Jan 2011 00:07:32 +0000

A few nitpicks.  BTW, expect to produce 3-6 drafts.


http://codereview.appspot.com/4005046/diff/1/ly/merge-function.ly
File ly/merge-function.ly (right):

http://codereview.appspot.com/4005046/diff/1/ly/merge-function.ly#newcode3
ly/merge-function.ly:3:
Having an example is a nice idea, but don't do it here.  Make a separate
file Documentation/snippets/new/merging-rests.ly or something like that.

http://codereview.appspot.com/4005046/diff/1/ly/merge-function.ly#newcode8
ly/merge-function.ly:8: \include "merge-rests"
The whole point of this work is so that you _won't_ need to \include
merge-rests, so I'd omit this part.  :)

http://codereview.appspot.com/4005046/



reply via email to

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