"Jonas Hahnfeld" <[email protected]> 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 <[email protected]>
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](https://sourceforge.net/p/testlilyissues/issues/5217/attachment/out-test-baseline-merge-rests-engraver.png)
 (23.8 kB; image/png)
- 
[out-test-merge-rests-engraver.png](https://sourceforge.net/p/testlilyissues/issues/5217/attachment/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 [email protected] 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
[email protected]
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
  • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development
    • ... Auto mailings of changes to Lily Issues via Testlilyissues-auto via Automated messages for lilypond development

Reply via email to