[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: morgenlied stafflet snafu
From: |
David Kastrup |
Subject: |
Re: morgenlied stafflet snafu |
Date: |
Sat, 14 Mar 2015 23:21:43 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
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