Comment 4 for bug 198400

Revision history for this message
ironstorm (ironstorm-gmail) wrote :

> Okay, then please don't use my first bugfix so that buttons don't get handled twice.
> Use the attached one :)

I'm actually going to write it to use a QSet or QList or something with all of the non-button controls so it will be something like this psuedo code (note: since some of these vars aren't defined yet):

QSet<int> non_buttons;
non_button_controls << XFADER << MAIN_VOL << MONITOR_SPLIT << MONITOR_MIX;
non_button_controls << MONITOR_A << GAIN_A << VOLUME_A << LEFT_JOG << LEFT_BASS << LEFT_MID << LEFT_HIGH << LEFT_PITCH;
// same thing for Deck B / RIGHT side

if ( second == 0 && !non_button_controls.contains(first)) {
...