- **Patch**: new --> review
- **Comment**:

Passes make, make check and a full make doc.



---

** [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:** Mon Jun 18, 2018 12:06 PM 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](http://lists.gnu.org/archive/html/lilypond-user/2018-06/msg00222.html) 
with the following minimal example:

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

~~~~
:::TeX
\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 testlilyissues-a...@lists.sourceforge.net 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
testlilyissues-a...@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto
  • [Lilypond-... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto
    • [Lily... Auto mailings of changes to Lily Issues via Testlilyissues-auto

Reply via email to