Re: Incompatible stem warnings may display nonsensic durations.

[David Kastrup]

Ralph Palmer <address@hidden> writes:

> On Sun, Apr 10, 2011 at 12:54 PM, David Kastrup <address@hidden> wrote:
>
>
>     Patch at
>     <URL:http://codereview.appspot.com/4390042>
>     
>     Check the warnings output from the regression test file in there:
>     
>     
>     This results in warnings like
>     
>     incompatible-stem-warning.ly:12:25: warning: adding note head to
>     incompatible stem (type = -2147483648)
>                 << c'\breve
>                             e'8 >>
>     
>     
>
> Hmm - Just  checked the url at the top, and it seems this bug report
> is moot, since David appears to have pushed the patch. The actual
> problem was not clear to me : I don't understand the warning in the
> first place. 

The type given in the warning is nonsensical for negative duration logs.
The type of the stem is determined from the first of the parallel notes
for this voice, and if that does not match the type of the other
parallel notes, no consistent result can be output (you only get one
stem per voice).

> Anyone else see a reason to turn this into an official issue?

I don't think there is a point to turning a fixed problem into an issue
unless there is a problem with the fix.

-- 
David Kastrup