[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] Re: #5217 random results for the merge-rests-engraver regression test

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

"Jonas Hahnfeld" address@hidden writes:

So the reason is that staff_priority_less compares pointers in the
default case. I'm not entirely sure why those change even without
ASLR, but maybe Guile's garbage collection and / or timing of
allocation play a role?

This is pretty appalling. I see that

• /* if neither grob has an outside-staff priority, the ordering will have no
• effect -- we just need to choose a consistent ordering. We do this to
• avoid the side-effect of calculating extents. */
• if (isinf (priority_1))
• return g1 < g2;

has been added in
commit 8eafccec8c45891d8e00366ca2f67f67a9c93cf9
Author: Joe Neeman address@hidden
Date: Sun Jan 21 14:04:13 2007 +0200

Enable one-pass stretching for PianoStaff

and seems of little relation to the commit message. It would likely be
saner to just return false instead of introducing some random order that
serves no point. Of course this will still warrant using a stable sort.

--
David Kastrup

---

[issues:#5217] random results for the merge-rests-engraver regression test

Status: Started
Created: Fri Oct 20, 2017 07:28 AM UTC by Knut Petersen
Last Updated: Tue Nov 05, 2019 06:20 PM UTC
Owner: Knut Petersen
Attachments:

• out-test-baseline-merge-rests-engraver.png (23.8 kB; image/png)
• out-test-merge-rests-engraver.png (23.7 kB; image/png)

There is some randomness in the placement of dots. The merge-rests-engraver.ly regtest (git-version557dc7) exposes the problem as the output might be one of the two attached pngs. The probability for both results is identical on my system.

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto