Menu

#5562 crash: Assertion `!is_empty ()' failed.

Fixed
Crash
2019-10-07
2019-09-28
No

[cf7be2a0d5093fb890083d09c1e320d8e06d3991]

This small input file

{
  \omit Flag
  \omit NoteHead

  b( b)
}

makes lilypond crash with

../flower/include/interval.hh:227:
  T Interval_t<T>::center() const [with T = double]:
  Assertion `!is_empty ()' failed.

https://codereview.appspot.com/546970043

Related

Issues: #5561

Discussion

  • Malte Meyn

    Malte Meyn - 2019-09-29

    You can even omit the \omit Flag to reproduce that bug (if it is one, see my comment at issue 5561)

     
  • Malte Meyn

    Malte Meyn - 2019-09-30
    • status: New --> Started
    • assigned_to: Malte Meyn
     
  • Malte Meyn

    Malte Meyn - 2019-09-30
    • Description has changed:

    Diff:

    --- old
    +++ new
    @@ -15,3 +15,4 @@
       T Interval_t&lt;T&gt;::center() const [with T = double]:
       Assertion `!is_empty ()&#39; failed.
     ~~~~
    +https://codereview.appspot.com/546970043
    
    • Needs: -->
    • Type: -->
     
  • Anonymous

    Anonymous - 2019-10-01
    • Needs: -->
    • Patch: new --> review
    • Type: --> Enhancement
     
  • Anonymous

    Anonymous - 2019-10-01

    Passes make. make check and a full make doc

     
  • Anonymous

    Anonymous - 2019-10-03
    • Patch: review --> countdown
     
  • Anonymous

    Anonymous - 2019-10-03

    Patch on countdown for Oct 5th

     
  • Anonymous

    Anonymous - 2019-10-06
    • Patch: countdown --> push
     
  • Anonymous

    Anonymous - 2019-10-06

    Patch counted down - please push

     
  • Malte Meyn

    Malte Meyn - 2019-10-07
    • labels: --> Fixed_2_21_0
    • status: Started --> Fixed
    • Patch: push -->
     
  • Malte Meyn

    Malte Meyn - 2019-10-07

    commit 7e6a95662522ef586614bf547539b2b05e4afb23
    Author: Malte Meyn lilypond@maltemeyn.de
    Date: Mon Sep 30 16:58:59 2019 +0200

    Issue 5561/5562: slurs work without NoteHead stencil
    
    While searching for slur configurations LilyPond crashed on NoteHeads
    without stencil because of empty extents. Also, when there cannot be
    found a viable slur config, this is reported as a warning rather than a
    failed assertion.
    

    This fixes also [#5561].

     

    Related

    Issues: #5561

  • Malte Meyn

    Malte Meyn - 2019-10-07
    • Type: Enhancement --> Crash