Antonio Lotti

I think this inversion was done to make it easier to append shorter durations to the DurationType enum (so that the previous values in the list are not touched), while at the same time have a comparison between TDurations (i.e note or rest time durations) looking more "natural" (i.e. a shorter TDuration being smaller than a longer TDuration in comparison conditions). Indeed, three or four years ago, the DurationType enum was enlarged to include also 512th and 1024th, which were not initially present....

Hello. I think I found a false positive while using CppCheck on MuseScore source code. It is triggered by the use of an overloaded comparison in a condition involving an implicit conversion. Attached you can find a minimal example. CppCheck v2.2 prompts two warnings: - noExplicitConstructor - knownConditionTrueFalse <error id="knownConditionTrueFalse" severity="style" msg="Condition &amp;#039;d==TDuration::DurationType::V_16TH&amp;#039; is always false" verbose="Condition &amp;#039;d==TDuration::DurationType::V_16TH&amp;#039;...