Re: [frogs] Patch for Issue 897 - Key signature should be inherited by all Voices (no matter their accidental style) |
[ Thread Index |
Date Index
| More lilynet.net/frogs Archives
]
- To: Owen Tuz <owentuz@xxxxxxxxx>
- Subject: Re: [frogs] Patch for Issue 897 - Key signature should be inherited by all Voices (no matter their accidental style)
- From: Neil Puttock <n.puttock@xxxxxxxxx>
- Date: Wed, 22 Dec 2010 23:18:16 +0000
- Cc: frogs@xxxxxxxxxxx
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:cc:content-type; bh=sPiLodwjnOQqGAHdgJRkIMQhikq7Yyqps9NeuQFpiZg=; b=W47wbiBcg5MBhveYqn2RWnU+5wkBpLu7euRUp4suaer7rIS9pgdX9JUQyVyQjz5iqE JfFGVHxKP+D9UrWrG+3/u/zauaTVTRqs9h0vi3NUCwJJTOdzUMvykDbjl2h0NLpACoNL eNJPZTi3qCswQ3tMFjTgO3zF91pWiwC1RTF6Y=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=hQo/Yk3o45oe9S/d1si+2cPjCqeeC8QsFsxhOLF0X7vrvVpgmuTOKrnA8nBSiNiXuL opAjrDA1zmFP/v3XAdMC8T6Ghp9crBUjJAFpxWJgyzAN5WS6ks2jgmMOS9QevpPEjXkt 7fscL1cyHIWUolM0T+wOtu8n2bYgJjKfABrPc=
On 22 December 2010 15:16, Owen Tuz <owentuz@xxxxxxxxx> wrote:
> If someone could verify that this does indeed fix the issue and not create
> anything new and horrible, that'd be great. There's ly code to reproduce the
> issue at the link above.
Have you run a regresson test check? That's the quickest way of
checking it doesn't break anything.
I'm not sure whether moving the engraver will affect any accidental
rules adversely, but it *will* have a detrimental effect on accidental
placement: the Accidental_engraver creates one AccidentalPlacement
grob per timestep in a stave, so if you move it to the Voice context,
each voice's accidentals will be oblivious to any other accidentals,
causing collisions.
Here's a simple example showing the problem:
\version "2.13.44"
\relative c' {
<< { cis1 } \\ { ais } >>
}
\layout {
\context {
\Staff
\remove "Accidental_engraver"
}
\context {
\Voice
\consists "Accidental_engraver"
}
}
Cheers,
Neil
---
----
Join the Frogs!