[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5345 Crash with specific combination of pitch and omitted note head

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

• Patch: countdown --> push
• Comment:

Patch counted down, please push.

---

[issues:#5345] Crash with specific combination of pitch and omitted note head

Status: Started
Created: Fri Jun 15, 2018 12:11 AM UTC by Simon Albrecht
Last Updated: Tue Jun 19, 2018 06:52 AM UTC
Owner: David Kastrup

Issue 5345: Stop crash while merging ledger lines

The old code (part of issue 4979) used quite broken code for merging
intervals that could crash eating all memory under certain conditions
and failed to account for merging certain interval combinations.

The incremental broken merge code has been thrown out and replaced by
using the non-incremental Interval_set::union which has been designed
for that purpose.

http://codereview.appspot.com/343270043

---

Reported by Lukas-Fabian Moser in this thread with the following minimal example:

WARNING: Only test with appropriate configuration of ulimit, see the e-mail exchange.

\version "2.19.80"

\new Staff <<
  {
    \clef bass
    e,4
  }
  {
    \omit NoteHead
    e,4
  }
>>

Lukas says:

Everything here (except the \version) seems to be necessary. I can reproduce the crash with some other configurations as well (another pitch in tenor clef, for instance), but removing any single command seems to resolve the problem. Also, putting each "voice" into a regular \new Voice (as in Harm's proposal) makes the problem disappear. Just as taking f, instead of e, for the pitch does (which could point to ledger lines).

David K. says the problem appears to be in the page builder.

---

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.

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot

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