Crash by undoing the removal failure of the repeats palette symbols separating multimeasure rests

• Jun 17, 2017 - 11:25
Reported version
2.1
Priority
P1 - High
Type
Graphical (UI)
Severity
S1 - Blocker
Status
closed
Regression
No
Workaround
No
Project

Sequence symbols cannot be deleted from a mscz file.

Selecting the symbols by mouse turns them blue. Pressing 'del' or 'ctrl-x' deselects the symbols, but leaves them in the score.

Attachment Size
Swingin'_The_Blues.mscz 5.97 KB

Comments

Title segno, coda, 'D.S. al Coda' cannot be deleted Crash by undoing the removal failure of the repeats palette symbols separating multimeasure rests

2.1 version / Windows10

1) Create five measures for any instrument
2) Select measure 3 and double-click on any symbol of the repeats & jumps palette:
eg Segno, Coda, Fine, D.S, etc. - except "measure repeat"/first symbol and barlines/last symbols.
3) Press "M"

- The test file at this step: test cut repeat.mscz

4) Select the symbol -> Cut
First result: Failure, the symbol remains

5) Undo

Result: crash

A bit surprised to find that this issue apparently simple / basic (minimal steps) is already active in former versions, 2.0.3, 2.0.2 and 2.0.1 and possibly more.

So, a bit of history ... :)

Until the beginning of September 2014, the segnos, codas etc did not automatically break the mmrests: #31436: Segno et al do not display at start of mmrest
This was fixed with this nightly / commit: https://github.com/musescore/MuseScore/pull/1269/files

The deletion of these symbols - when the mmrests were displayed - was already not possible. Was it expected or not? I do not know (probably not?)

But the removal (or its attempt) of the symbols did not cause yet a crash. Another undo, and we go back to the step without the mmrests display.

- The crash after first undo occurs several months later, on February 25, 2015. (a month before the 2.0 release)

This nightly works (in the sense, no crash after undo): 781c5b7
Not the next one: 94ad084

And so, this is an unexpected side effect of this last mentioned nightly/commit.
It was to fix: #48561: Crash when toggling visibility of a line break over a multimeasure rest