Re: Partial ties not performed

[Michael Welsh Duggan]

Han-Wen Nienhuys <address@hidden> writes:

> Michael Welsh Duggan wrote:
>> Partial ties are not picked up by the tie engraver.  The included
>> patch fixes the problem, but I have no idea whether it fits into the
>> spirit of the code.
>
> looks good.
>
> a few nits:
>
>  - Changelog entry missing
>> +      if (an->tie_event_) {
>> +        now_tied_heads_.push_back (inf);
>> +      } else {
>> +        now_heads_.push_back (inf);
>> +      }
>
> don't use braces for statements.
>
>> -  Audio_note (Pitch p, Moment m, int transposing_i = 0);
>> +  Audio_note (Pitch p, Moment m, bool tie_event, int transposing_i = 0);
>>  
>
> While you're at it, can you see whether you could remove the
> transposing_i argument?  We've stopped using default args a long time
> ago.
>
>
> Can you fix and resend ?

Fixed and resent.

txtE7vIcPRHl1.txt
Description: Text Data

-- 
Michael Welsh Duggan
(address@hidden)