Re: morgenlied stafflet snafu

[David Kastrup]

David Kastrup <address@hidden> writes:

> Han-Wen Nienhuys <address@hidden> writes:
>
>> I'm at commit 2728f57934651f8de7ae99064305b72d27990932
>>
>> When I run input/regression/morgenlied through lilypond, I get a small
>> bit of staff that shouldnt be there,
>
> Here is a much smaller example:
>
> {
>   \time 6/8
>   \new Staff { c2. }
> }
>
> This would likely be
> Issue 4138: Patch: Make \time work with \tweak and \footnote
>
> and <URL:https://code.google.com/p/lilypond/issues/detail?id=4138>.
>
> I am surprised this did not turn up in the regtests.  I have an idea how
> to fix this.

Actually, this looks rather bad to fix.  With

{
  \tweak color #red \time 6/8 \new Staff { c2. }
}

you have the option to create a Staff context when encountering the
tweaked time in which case the tweak will make it through to the
engraver but \new Staff will create a new Staff.

Or you don't create a Staff for \time in which case the \tweak cannot
really arrive in not-yet-existent staves.

At the current point of time, there is a TimeSignatureEvent generated,
and as an event it does not just create a Staff but a whole Bottom
context (which will usually be a Voice).  Which may be sort of overkill.

I'm not particularly happy with the options here.

-- 
David Kastrup