Double coda roadmap is wrong

• Feb 12, 2018 - 13:43
Reported version
2.2
Type
Functional
Severity
S4 - Minor
Status
closed
Project
Tags

Test case from Miré on the french forum: https://musescore.org/fr/node/269228
The file plays correctly in 2.1 but not in 2.2-dev

Attachment Size
Test 2-1vs 2-2.mscz 7.87 KB

Comments

The culprit is the new logic used to rewind the playbackCount upon jumping at https://github.com/musescore/MuseScore/blob/master/libmscore/repeatlist…

In this example it leads to measure 6 being rewound when taking the first jump; but because it isn't (re)played during that jump and holds the startRepeat for all following measures the playthrough after the 2nd jump isn't considered the 'final' playthrough of that repeating section. Therefor it ignores jump instructions.

I think the solution here is to only rewind measures from jumpTo up to playUntil and then from continueAt to currentMeasure. I still have to see that it doesn't break any of the other current tests and add the posted score as a new testscenario.

Status (old) patch (code needs review) fixed
Status fixed

issue status not updated correctly, probably because Jojo and Git posted simultaneously