Allow 128th as the shortest actual measure duration

and report an error rather than crash if the score has something shorter See musescore#8890 and musescore#8893, backport of musescore#8894, part 1
Jojo-Schmitz · Sep 26, 2021 · 92f310b · 92f310b
1 parent 89ce2cd
commit 92f310b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/libmscore/measure.cpp b/libmscore/measure.cpp
@@ -2022,7 +2022,7 @@ void Measure::read(XmlReader& e, int staffIdx)
             else
                   qDebug("illegal measure size <%s>", qPrintable(e.attribute("len")));
             irregular = true;
-            if (_len.numerator() <= 0 || _len.denominator() <= 0) {
+            if (_len.numerator() <= 0 || _len.denominator() <= 0 || _len.denominator() > 128) {
                   e.raiseError(QObject::tr("MSCX error at line %1: invalid measure length: %2").arg(e.lineNumber()).arg(_len.toString()));
                   return;
                   }

diff --git a/mscore/measureproperties.ui b/mscore/measureproperties.ui
@@ -214,6 +214,11 @@
               <string>64</string>
              </property>
             </item>
+            <item>
+             <property name="text">
+              <string>128</string>
+             </property>
+            </item>
            </widget>
           </item>
           <item row="0" column="4" colspan="2">