From 4e9fba3706384e605f578529856b9613a26a9a1d Mon Sep 17 00:00:00 2001 From: Casper Jeukendrup <48658420+cbjeukendrup@users.noreply.github.com> Date: Sun, 23 Feb 2025 19:39:55 +0100 Subject: [PATCH] Fix Clang warning and potential bug "Operator '?:' has lower precedence than '+'; '+' will be evaluated first" Introduced by 6205752864d772a03f591c525c9d749534746624 --- src/notation/internal/notationnoteinput.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/notation/internal/notationnoteinput.cpp b/src/notation/internal/notationnoteinput.cpp index 2d403f40f7cfe..4605047ba0c99 100644 --- a/src/notation/internal/notationnoteinput.cpp +++ b/src/notation/internal/notationnoteinput.cpp @@ -591,7 +591,7 @@ void NotationNoteInput::moveInputNotes(bool up, PitchMode mode) switch (mode) { case PitchMode::CHROMATIC: - newVal.pitch = val.pitch + up ? 1 : -1; + newVal.pitch = val.pitch + (up ? 1 : -1); break; case PitchMode::DIATONIC: { const int oldLine = mu::engraving::noteValToLine(val, is.staff(), is.tick());