RepeatList should use measureIndex instead of measure->no()
Reported version
3.0
Priority
P3 - Low
Type
Functional
Frequency
Few
Severity
S4 - Minor
Reproducibility
Always
Status
closed
Regression
No
Workaround
Yes
Project
Having a jump marker (such as Fine) in a measure where the consecutive measure has measure number -1 offset will result in the second measure being included into the jump when it shouldn't.
Report, start of analysis and example score in https://musescore.org/en/node/285671
This score is affected by using no()
where measureIndex()
should be used. See
https://github.com/musescore/MuseScore/blob/master/libmscore/repeatlist…
Using index()
should work as well and be a little more performant.
Likely this change should happen for the whole of the RepeatList class internal logic…
Fix version
3.5.1
Comments
Fixed by https://github.com/musescore/MuseScore/pull/6248
Automatically closed -- issue fixed for 2 weeks with no activity.