Re: Adds padding between Hairpins and SpanBars. (issue 5438060)

[address@hidden]

On Jan 27, 2012, at 8:10 PM, Keith OHara wrote:

> On 2011/11/28 09:11:47, mike_apollinemike.com wrote:
>> If the hairpins stop before span bars but
>> extend all the way when span bar's don't exist
>> (including when they are not
>> present because of the RemoveEmptyStaffContext),
>> then I'd much rather go with
>> your patch, as it is much less invasive than mine.
>> 
> 
> Mike,
> Did you need for your own reasons that broken Hairpins stop early if and only 
> if any concurrent Hairpin hits a span bar, or did you think somebody else 
> wanted that?
> 

It was a pure aesthetic judgement on my part - I should have polled people 
before but forgot.

> It would be much simpler to simply end all Hairpins the same way, so they 
> don't need to check the concurrent Hairpins.  I think this is what everyone 
> is asking for, but you added a regression test 'hairpin-span-bar.ly' that 
> explicitly requires Hairpin lengths to change based on context.

This is probably a one-two liner on/off flag w/ an extra property - I'm 
musicking now, so if you have some time & can add it, I'd be much obliged.  
Otherwise, I'll be un-musicking in a weekish.

Cheers,
MS