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 _______________________________________________ lilypond-devel mailing list [email protected] https://lists.gnu.org/mailman/listinfo/lilypond-devel
