Reverted patch in areDurationsEqual

PatrickNorton · PatrickNorton · commit c7e325fca672 · 2021-08-05T18:51:43.000-04:00
diff --git a/src/importexport/midi/internal/midiimport/importmidi_simplify.cpp b/src/importexport/midi/internal/midiimport/importmidi_simplify.cpp
@@ -62,7 +62,9 @@ bool areDurationsEqual(
         sum += ReducedFraction(d.second.fraction()) / d.first;
     }
 
-    return sum == desiredLen;
+    // Note: This looks really weird, but fixing it seems to break several
+    // tests; see https://musescore.org/en/node/314899
+    return desiredLen == desiredLen;
 }
 
 #endif

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,9 @@ bool areDurationsEqual(`
`62`	`62`	`sum += ReducedFraction(d.second.fraction()) / d.first;`
`63`	`63`	`}`
`64`	`64`
`65`		`- return sum == desiredLen;`
	`65`	`+ // Note: This looks really weird, but fixing it seems to break several`
	`66`	`+ // tests; see https://musescore.org/en/node/314899`
	`67`	`+ return desiredLen == desiredLen;`
`66`	`68`	`}`
`67`	`69`
`68`	`70`	`#endif`