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

[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 to master?

I see this has been pushed now, but I wondered why
ambitus was removed from the space-alist of Clef?

(Clef
     . (
 (avoid-slur . inside)
 (break-align-anchor . 
,ly:break-aligned-interface::calc-extent-aligned-anchor)
 (break-align-symbol . clef)
 (break-visibility . ,begin-of-line-visible)
 (glyph-name . ,ly:clef::calc-glyph-name)
 (non-musical . #t)
- (space-alist . ((ambitus . (extra-space . 2.0))
+ (space-alist . ((cue-clef . (extra-space . 2.0))
   (staff-bar . (extra-space . 0.7))
   (key-cancellation . (minimum-space . 3.5))
   (key-signature . (minimum-space . 3.5))
   (time-signature . (minimum-space . 4.2))
   (first-note . (minimum-fixed-space . 5.0))
   (next-note . (extra-space . 0.5))
   (right-edge . (extra-space . 0.5))))

Trevor