Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Nested beam crash on sequence with Stem Direction change #86

Closed
arshiacont opened this issue Aug 16, 2019 · 3 comments
Closed

Nested beam crash on sequence with Stem Direction change #86

arshiacont opened this issue Aug 16, 2019 · 3 comments

Comments

@arshiacont
Copy link

The following crashes:

{[ \staff<1> 
   \clef<"f4"> \meter<"2/4"> \stemsUp \beamBegin:1 \beamBegin:2 d-1/16
 \stemsDown d1/16 \stemsDown g/16 \stemsDown d/16 \beamEnd:2 \beamEnd:1 \stemsDown \beamBegin:1 \beamBegin:2 b&0/16
 \stemsDown b&/16 \stemsDown d1/16 \stemsDown b&0/16 \beamEnd:2 \beamEnd:1 ]
  }

while this one doesn't:

{[ \staff<1> 
   \clef<"f4"> \meter<"2/4"> \stemsUp \beamBegin:1  d-1/16
 \stemsDown d1/16 \stemsDown g/16 \stemsDown d/16 \beamEnd:1 \stemsDown \beamBegin:1 \beamBegin:2 b&0/16
 \stemsDown b&/16 \stemsDown d1/16 \stemsDown b&0/16 \beamEnd:2 \beamEnd:1 ]
  }

Note that we restored the use of nested beams as of issue #58

In the debugger, I can see a call to SimpleBeamList::GetHeadPosition() on GRBeam::checkEndStemsReverse where the beam is Null.

@dfober
Copy link
Member

dfober commented Aug 19, 2019

fixed

@dfober dfober closed this as completed Aug 19, 2019
@arshiacont
Copy link
Author

Thanks @dfober . You might have forgotten to Push ? :)

@dfober
Copy link
Member

dfober commented Aug 19, 2019 via email

arshiacont added a commit to Antescofo/guidolib that referenced this issue Feb 22, 2022
….4.5

* commit '35078ff097ca8236f4d3736f65a9e8ab909ea59f':
  fix crash bug with nested beams (issue grame-cncm#86)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants