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

[Keith OHara]

 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 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.
--
Keith