Fixed the maximal value of the "SYSEX Hauptwerk 32 Byte LCD" midi sen…

…d events GrandOrgue#1686 (GrandOrgue#1708)
oleg68 · Nov 5, 2023 · 6e7b9fd · 6e7b9fd
1 parent 2e2747d
commit 6e7b9fd
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,4 @@
+- Fixed the maximal value of the "SYSEX Hauptwerk 32 Byte LCD" midi send events https://github.com/GrandOrgue/grandorgue/issues/1686
 - Fixed discard in the Organ Settings dialog when multiple objects are selected https://github.com/GrandOrgue/grandorgue/issues/1674
 - Moved all Organ Settings dialog buttons to the single button row https://github.com/GrandOrgue/grandorgue/issues/1674
 - Decreased number of decimal digits in the Organ Settings dialog https://github.com/GrandOrgue/grandorgue/issues/1627

diff --git a/src/grandorgue/midi/GOMidiSender.cpp b/src/grandorgue/midi/GOMidiSender.cpp
@@ -124,15 +124,18 @@ void GOMidiSender::Load(GOConfigReader &cfg, wxString group, GOMidiMap &map) {
         0x1f,
         false,
         0);
-    if (HasLength(eventType))
+    if (HasLength(eventType)) {
+      unsigned maxLength = LengthLimit(eventType);
+
       m_events[i].length = cfg.ReadInteger(
         CMBSetting,
         group,
         wxString::Format(wxT("MIDISendLength%03d"), i + 1),
         0,
-        0x1f,
+        maxLength,
         false,
-        LengthLimit(eventType));
+        maxLength);
+    }
   }
 }