On Mon, Jan 24, 2011 at 2:46 AM, Han-Wen Nienhuys <hanw...@gmail.com> wrote:
>> I hope that my clearer code will assuage your fears that this affects
>> anything other than certain limited scenarios - in the regtests, it
>
> that is not so much my fear. I am concerned with putting in code that
> we know is flawed in advance, when we have a much better mechanism to
> deal with this (see below).
Actually, there are some serious show-stoppers in this approach. See
the attached file. It appears your code kicks into action even if
there are no collisions. We should probably double check if the
regression test detects beam changes accurately. If not, we have a
serious hole in our coverage.
(I was going to look for more subtle failure cases, but this error
prevents me from demonstrating them.)
\new Staff \relative c' {
<<
\new Voice
{ \stemUp c8[ c c c] c8[ c c c]
c8[ c c c]
}
\new Voice {
\voiceThree s2
\stemUp e'8[ e e e]
\set fontSize = #-4 c8[ c c c]
}
>>
}
--
Han-Wen Nienhuys - han...@xs4all.nl - http://www.xs4all.nl/~hanwen
<<attachment: g2.preview.png>>
_______________________________________________ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel
