Increase number of MIDI messages to buffer, fix bug #1527410

Chatty MIDI controllers like the Stanton SCS.1d (moving platter) can queue up a large number of messages if the system is for whatever reason slow to respond to them. If more than this number queue up, Mixxx stops responding to that controller. Experimental stress-testing showed 500 messages at once in the queue (during stupid things like application window switching, but still.) This may not be the best fix, but it's quick and breaks nothing for the 2.0 release.
mixxxdj · Dec 18, 2015 · 469a37f · 469a37f
1 parent 5b9ea4f
commit 469a37f
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/controllers/midi/portmidicontroller.h b/src/controllers/midi/portmidicontroller.h
@@ -20,7 +20,9 @@
 #include <portmidi.h>
 #include "controllers/midi/midicontroller.h"
 
-#define MIXXX_PORTMIDI_BUFFER_LEN 64 /**Number of MIDI messages to buffer*/
+// Mixxx completely stops responding to the controller if more than this number of messages queue up.
+//  Don't lower this (much.) The SCS.1d accumulated 500 messages in a single poll during stress-testing.
+#define MIXXX_PORTMIDI_BUFFER_LEN 1024 /**Number of MIDI messages to buffer*/
 #define MIXXX_PORTMIDI_NO_DEVICE_STRING "None" /**String to display for no MIDI devices present */
 
 /** A PortMidi-based implementation of MidiController */