[Top][All Lists]
[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/
- Add new merge-function.ly file for rests (issue4005046), Carl . D . Sorensen, 2011/01/22
- Re: Add new merge-function.ly file for rests (issue4005046),
percival . music . ca <=
- Re: Add new merge-function.ly file for rests (issue4005046), percival . music . ca, 2011/01/22
- Re: Add new merge-function.ly file for rests (issue4005046), pkx166h, 2011/01/23
- Re: Add new merge-function.ly file for rests (issue4005046), n . puttock, 2011/01/23
- Re: Add new merge-function.ly file for rests (issue4005046), pkx166h, 2011/01/24
- Re: Add new merge-function.ly file for rests (issue4005046), pkx166h, 2011/01/24