OS: Windows 10 64bit
Mixxx: 2.0 32 and 64bits
Controller: Hercules DJ Console 4-Mx (contributed mapping as relesed in 2.0).
New in 1.12/2.0, the "SYNC" button enables master sync for the deck if it is pressed and NOT released immediately. (Visually, it remains "on" after releasing).
This works from the UI, and also if mapped to a controller with XML, but not from when doing so from javascript (either with setValue or with setParameter).
I've been able to pinpoint the problem to an incorrect behaviour of the QTimer in this file:
https://github.com/mixxxdj/mixxx/blob/1.12/src/control/controlbehavior.cpp
Method void ControlPushButtonBehavior::setValueFromMidiParameter(
MidiOpCode o, double dParam, ControlDoublePrivate* pControl)
When i press it from my controller, it enters the "pressed" block (LONGPRESSLATCHING), the action is executed, and the timer is started (it returns an id for the timer, so i assume that means that it started).
When i release the button, after several seconds, the "else" block is executed and enters inside the if that contains the "timer.isActive()". Said timer is expected to return false instead, because the time has elapsed, so i don't know what happens.
I've read in QT's documentation that: " In multithreaded applications, you can use QTimer in any thread that has an event loop. To start an event loop from a non-GUI thread, use QThread::exec(). "
Of course, we are in the midi thread, and is a non-GUI thread.
Ok, I just realized that the setValue/
So, the question is: should we modify setValue/