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
