Bad tie lengths on mirrored end notes
Windows 10, GIT commit: fecf0a7
Discovered while working on #187151: Hairpins shifting out of position on systems containing ties from stem.
1) Load attached score
2) examine first tie
Result: tie is *way* too short (barely even visible at all):
This case has never worked correctly that I can tell, going back to when I first added the minimum tie length enforcement well before 2.0 beta (and I doubt it was any better before that. It is a bit contrived, in that normally if the end note is mirrored to the left, so will the start note, and the tie length will seem OK, as in the second example.
Still, it shows a flaw in the logic here. Simply removing the qAbs() here fixes this case:
But I would want to understand better why I thought that was a good idea originally before comitting to this - I suspect some other case gets broken. On the other hand, I know I also modified the handling of seconds shortly after this.
BTW, logically, we should also factor in the 0.12 whether or not there is a non-zero X offset. But doing so seems to create unnecessarily long ties in cases, like if I added one to the the "D".