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


_______________________________________________
lilypond-devel mailing list
[email protected]
https://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to