Re: [PATCH] Re: Clef support for cue notes (issue2726043)

2010-12-29 Thread Trevor Daniels
Reinhold Kainhofer wrote Tuesday, December 28, 2010 9:10 PM I have now uploaded a final patch for cue notes with custom clef, which I think is in a state so that it can be included in lilypond master: http://codereview.appspot.com/2726043/ Any further objectsions/suggestions? Or can I push

Re: [PATCH] Re: Clef support for cue notes (issue2726043)

2010-12-29 Thread Reinhold Kainhofer
Am Mittwoch, 29. Dezember 2010, um 12:13:08 schrieb Trevor Daniels: I see this has been pushed now, but I wondered why ambitus was removed from the space-alist of Clef? AFAIU, the space-alist controls spacing of the current grob to a following grob of the listed break-align-symbol. Now, in

Re: [PATCH] Re: Clef support for cue notes (issue2726043)

2010-12-29 Thread Trevor Daniels
Reinhold, you wrote Wednesday, December 29, 2010 12:13 PM Am Mittwoch, 29. Dezember 2010, um 12:13:08 schrieb Trevor Daniels: I see this has been pushed now, but I wondered why ambitus was removed from the space-alist of Clef? AFAIU, the space-alist controls spacing of the current grob to

[PATCH] Re: Clef support for cue notes (issue2726043)

2010-12-28 Thread Reinhold Kainhofer
I have now uploaded a final patch for cue notes with custom clef, which I think is in a state so that it can be included in lilypond master: http://codereview.appspot.com/2726043/ Any further objectsions/suggestions? Or can I push to master? I have included almost all comments so far, except