Re: Lily 2.7.7 and MIDI

[Michael Welsh Duggan]

Han-Wen Nienhuys <address@hidden> writes:

> Michael Welsh Duggan wrote:
>
>> I've been looking into this.  The problem is twofold: 1) Time
>> offsets devolve into "1"s (instead of "384"s).  This is due to
>>    the addition of Rational::operator bool().  A rational offset is
>
>> 2) Audio_tempo objects are not being added to the Audio_staffs.  This
>>    results in the tempo not being passed to the midi file.  In the
>>    past, it was added by
>> Staff_performer::play_element(Audio_element).  
>
> I didn't apply your patch, but many, MANY thanks for looking into this 
> and tracking the cause. The MIDI backend is my least favorite part of 
> the code, and I always try to postpone fixing things in it. This again 
> proved that it's actually a good strategy  :-)
>
> can you check if current CVS works for you?

Almost.  Just sent a secondary patch to lily-devel -- another instance
of bad rational->int.

-- 
Michael Welsh Duggan
(address@hidden)