Hey Sean,

I'm going to give you a hard time because you're a developer. :)

Two questions:
1) How is this useful in its current incarnation if you can't have
multiple <options> blocks?

2) Why does this have to go in 1.9? None of the mappings are using it. ??

3) What are the SCS.3d changes you made? Should we cherry-pick those
changes and put them in trunk before everything else?

Thanks,
Albert


On Tue, Nov 16, 2010 at 4:58 PM, Sean M. Pappalardo
<email address hidden> wrote:
> Sean M. Pappalardo has proposed merging lp:~mixxxdevelopers/mixxx/features_softtakeover into lp:mixxx.
>
> Requested reviews:
>  Mixxx Development Team (mixxxdevelopers)
> Related bugs:
>  #555547 Add soft-takeover for MIDI controllers (XML and script)
>  https://bugs.launchpad.net/bugs/555547
>
>
> This branch adds soft-takeover capabilities for MIDI controllers, both for simple XML mappings and MIDI scripting.
>
> I'm requesting that this be included in time for Mixxx v1.9.0.
>
> Note that the main goal was to add this for MIDI scripting. I did it for the XML mappings as well for completeness, but that part isn't totally useful until Mixxx can accept multiple <option>s for simple XML mappings. Adding that capability is beyond the scope of the feature itself since everything is hard-coded for a single <option>, and it would require modification of GUI classes as well, something I don't feel qualified to do, especially not with the hope of getting this merged in time for 1.9.0.
> --
> https://code.launchpad.net/~mixxxdevelopers/mixxx/features_softtakeover/+merge/41027
> Your team Mixxx Development Team is requested to review the proposed merge of lp:~mixxxdevelopers/mixxx/features_softtakeover into lp:mixxx.
>
> === modified file 'mixxx/res/midi/Stanton-SCS3d-scripts.js'
> --- mixxx/res/midi/Stanton-SCS3d-scripts.js     2010-06-29 11:58:44 +0000
> +++ mixxx/res/midi/Stanton-SCS3d-scripts.js     2010-11-17 00:57:57 +0000
> @@ -64,7 +64,7 @@
>  StantonSCS3d.triggerS4 = 0xFF;
>
>  // Signals to (dis)connect by mode: Group, Key, Function name
> -StantonSCS3d.modeSignals = {"fx":[       ["[Flanger]", "lfoDepth", "StantonSCS3d.FXDepthLEDs"],
> +StantonSCS3d.modeSignals = {"fx":[    ["[Flanger]", "lfoDepth", "StantonSCS3d.FXDepthLEDs"],
>                                       ["[Flanger]", "lfoDelay", "StantonSCS3d.FXDelayLEDs"],
>                                       ["[Flanger]", "lfoPeriod", "StantonSCS3d.FXPeriodLEDs"],
>                                       ["CurrentChannel", "reverse", "StantonSCS3d.B11LED"],
> @@ -80,77 +80,77 @@
>                             "loop2":[ ["CurrentChannel", "pfl", "StantonSCS3d.B11LED"] ],
>                             "loop3":[ ["CurrentChannel", "pfl", "StantonSCS3d.B11LED"] ],
>                             "trig":[  ["CurrentChannel", "pfl", "StantonSCS3d.B11LED"],
> -                                                                         ["CurrentChannel", "hotcue_1_enabled", "StantonSCS3d.BsALED"],
> -                                                                         ["CurrentChannel", "hotcue_2_enabled", "StantonSCS3d.BsBLED"],
> -                                                                         ["CurrentChannel", "hotcue_3_enabled", "StantonSCS3d.BsCLED"],
> -                                                                         ["CurrentChannel", "hotcue_4_enabled", "StantonSCS3d.BsDLED"],
> -                                                                         ["CurrentChannel", "hotcue_5_enabled", "StantonSCS3d.BsELED"],
> -                                                                         ["CurrentChannel", "hotcue_6_enabled", "StantonSCS3d.BsFLED"],
> -                                                                         ["CurrentChannel", "hotcue_7_enabled", "StantonSCS3d.BsGLED"],
> -                                                                         ["CurrentChannel", "hotcue_8_enabled", "StantonSCS3d.BsHLED"],
> -                                                                         ["CurrentChannel", "hotcue_9_enabled", "StantonSCS3d.BsILED"],
> -                                                                         ["CurrentChannel", "hotcue_10_enabled", "StantonSCS3d.BsJLED"],
> -                                                                         ["CurrentChannel", "hotcue_11_enabled", "StantonSCS3d.BsKLED"],
> -                                                                         ["CurrentChannel", "hotcue_12_enabled", "StantonSCS3d.BsLLED"],
> -                                                                         ["CurrentChannel", "hotcue_1_activate", "StantonSCS3d.BsAaLED"],
> -                                                                         ["CurrentChannel", "hotcue_2_activate", "StantonSCS3d.BsBaLED"],
> -                                                                         ["CurrentChannel", "hotcue_3_activate", "StantonSCS3d.BsCaLED"],
> -                                                                         ["CurrentChannel", "hotcue_4_activate", "StantonSCS3d.BsDaLED"],
> -                                                                         ["CurrentChannel", "hotcue_5_activate", "StantonSCS3d.BsEaLED"],
> -                                                                         ["CurrentChannel", "hotcue_6_activate", "StantonSCS3d.BsFaLED"],
> -                                                                         ["CurrentChannel", "hotcue_7_activate", "StantonSCS3d.BsGaLED"],
> -                                                                         ["CurrentChannel", "hotcue_8_activate", "StantonSCS3d.BsHaLED"],
> -                                                                         ["CurrentChannel", "hotcue_9_activate", "StantonSCS3d.BsIaLED"],
> -                                                                         ["CurrentChannel", "hotcue_10_activate", "StantonSCS3d.BsJaLED"],
> -                                                                         ["CurrentChannel", "hotcue_11_activate", "StantonSCS3d.BsKaLED"],
> -                                                                         ["CurrentChannel", "hotcue_12_activate", "StantonSCS3d.BsLaLED"] ],
> +                                      ["CurrentChannel", "hotcue_1_enabled", "StantonSCS3d.BsALED"],
> +                                      ["CurrentChannel", "hotcue_2_enabled", "StantonSCS3d.BsBLED"],
> +                                      ["CurrentChannel", "hotcue_3_enabled", "StantonSCS3d.BsCLED"],
> +                                      ["CurrentChannel", "hotcue_4_enabled", "StantonSCS3d.BsDLED"],
> +                                      ["CurrentChannel", "hotcue_5_enabled", "StantonSCS3d.BsELED"],
> +                                      ["CurrentChannel", "hotcue_6_enabled", "StantonSCS3d.BsFLED"],
> +                                      ["CurrentChannel", "hotcue_7_enabled", "StantonSCS3d.BsGLED"],
> +                                      ["CurrentChannel", "hotcue_8_enabled", "StantonSCS3d.BsHLED"],
> +                                      ["CurrentChannel", "hotcue_9_enabled", "StantonSCS3d.BsILED"],
> +                                      ["CurrentChannel", "hotcue_10_enabled", "StantonSCS3d.BsJLED"],
> +                                      ["CurrentChannel", "hotcue_11_enabled", "StantonSCS3d.BsKLED"],
> +                                      ["CurrentChannel", "hotcue_12_enabled", "StantonSCS3d.BsLLED"],
> +                                      ["CurrentChannel", "hotcue_1_activate", "StantonSCS3d.BsAaLED"],
> +                                      ["CurrentChannel", "hotcue_2_activate", "StantonSCS3d.BsBaLED"],
> +                                      ["CurrentChannel", "hotcue_3_activate", "StantonSCS3d.BsCaLED"],
> +                                      ["CurrentChannel", "hotcue_4_activate", "StantonSCS3d.BsDaLED"],
> +                                      ["CurrentChannel", "hotcue_5_activate", "StantonSCS3d.BsEaLED"],
> +                                      ["CurrentChannel", "hotcue_6_activate", "StantonSCS3d.BsFaLED"],
> +                                      ["CurrentChannel", "hotcue_7_activate", "StantonSCS3d.BsGaLED"],
> +                                      ["CurrentChannel", "hotcue_8_activate", "StantonSCS3d.BsHaLED"],
> +                                      ["CurrentChannel", "hotcue_9_activate", "StantonSCS3d.BsIaLED"],
> +                                      ["CurrentChannel", "hotcue_10_activate", "StantonSCS3d.BsJaLED"],
> +                                      ["CurrentChannel", "hotcue_11_activate", "StantonSCS3d.BsKaLED"],
> +                                      ["CurrentChannel", "hotcue_12_activate", "StantonSCS3d.BsLaLED"] ],
>                             "trig2":[ ["CurrentChannel", "pfl", "StantonSCS3d.B11LED"],
> -                                                                         ["CurrentChannel", "hotcue_13_enabled", "StantonSCS3d.BsALED"],
> -                                                                         ["CurrentChannel", "hotcue_14_enabled", "StantonSCS3d.BsBLED"],
> -                                                                         ["CurrentChannel", "hotcue_15_enabled", "StantonSCS3d.BsCLED"],
> -                                                                         ["CurrentChannel", "hotcue_16_enabled", "StantonSCS3d.BsDLED"],
> -                                                                         ["CurrentChannel", "hotcue_17_enabled", "StantonSCS3d.BsELED"],
> -                                                                         ["CurrentChannel", "hotcue_18_enabled", "StantonSCS3d.BsFLED"],
> -                                                                         ["CurrentChannel", "hotcue_19_enabled", "StantonSCS3d.BsGLED"],
> -                                                                         ["CurrentChannel", "hotcue_20_enabled", "StantonSCS3d.BsHLED"],
> -                                                                         ["CurrentChannel", "hotcue_21_enabled", "StantonSCS3d.BsILED"],
> -                                                                         ["CurrentChannel", "hotcue_22_enabled", "StantonSCS3d.BsJLED"],
> -                                                                         ["CurrentChannel", "hotcue_23_enabled", "StantonSCS3d.BsKLED"],
> -                                                                         ["CurrentChannel", "hotcue_24_enabled", "StantonSCS3d.BsLLED"],
> -                                                                         ["CurrentChannel", "hotcue_13_activate", "StantonSCS3d.BsAaLED"],
> -                                                                         ["CurrentChannel", "hotcue_14_activate", "StantonSCS3d.BsBaLED"],
> -                                                                         ["CurrentChannel", "hotcue_15_activate", "StantonSCS3d.BsCaLED"],
> -                                                                         ["CurrentChannel", "hotcue_16_activate", "StantonSCS3d.BsDaLED"],
> -                                                                         ["CurrentChannel", "hotcue_17_activate", "StantonSCS3d.BsEaLED"],
> -                                                                         ["CurrentChannel", "hotcue_18_activate", "StantonSCS3d.BsFaLED"],
> -                                                                         ["CurrentChannel", "hotcue_19_activate", "StantonSCS3d.BsGaLED"],
> -                                                                         ["CurrentChannel", "hotcue_20_activate", "StantonSCS3d.BsHaLED"],
> -                                                                         ["CurrentChannel", "hotcue_21_activate", "StantonSCS3d.BsIaLED"],
> -                                                                         ["CurrentChannel", "hotcue_22_activate", "StantonSCS3d.BsJaLED"],
> -                                                                         ["CurrentChannel", "hotcue_23_activate", "StantonSCS3d.BsKaLED"],
> -                                                                         ["CurrentChannel", "hotcue_24_activate", "StantonSCS3d.BsLaLED"] ],
> +                                      ["CurrentChannel", "hotcue_13_enabled", "StantonSCS3d.BsALED"],
> +                                      ["CurrentChannel", "hotcue_14_enabled", "StantonSCS3d.BsBLED"],
> +                                      ["CurrentChannel", "hotcue_15_enabled", "StantonSCS3d.BsCLED"],
> +                                      ["CurrentChannel", "hotcue_16_enabled", "StantonSCS3d.BsDLED"],
> +                                      ["CurrentChannel", "hotcue_17_enabled", "StantonSCS3d.BsELED"],
> +                                      ["CurrentChannel", "hotcue_18_enabled", "StantonSCS3d.BsFLED"],
> +                                      ["CurrentChannel", "hotcue_19_enabled", "StantonSCS3d.BsGLED"],
> +                                      ["CurrentChannel", "hotcue_20_enabled", "StantonSCS3d.BsHLED"],
> +                                      ["CurrentChannel", "hotcue_21_enabled", "StantonSCS3d.BsILED"],
> +                                      ["CurrentChannel", "hotcue_22_enabled", "StantonSCS3d.BsJLED"],
> +                                      ["CurrentChannel", "hotcue_23_enabled", "StantonSCS3d.BsKLED"],
> +                                      ["CurrentChannel", "hotcue_24_enabled", "StantonSCS3d.BsLLED"],
> +                                      ["CurrentChannel", "hotcue_13_activate", "StantonSCS3d.BsAaLED"],
> +                                      ["CurrentChannel", "hotcue_14_activate", "StantonSCS3d.BsBaLED"],
> +                                      ["CurrentChannel", "hotcue_15_activate", "StantonSCS3d.BsCaLED"],
> +                                      ["CurrentChannel", "hotcue_16_activate", "StantonSCS3d.BsDaLED"],
> +                                      ["CurrentChannel", "hotcue_17_activate", "StantonSCS3d.BsEaLED"],
> +                                      ["CurrentChannel", "hotcue_18_activate", "StantonSCS3d.BsFaLED"],
> +                                      ["CurrentChannel", "hotcue_19_activate", "StantonSCS3d.BsGaLED"],
> +                                      ["CurrentChannel", "hotcue_20_activate", "StantonSCS3d.BsHaLED"],
> +                                      ["CurrentChannel", "hotcue_21_activate", "StantonSCS3d.BsIaLED"],
> +                                      ["CurrentChannel", "hotcue_22_activate", "StantonSCS3d.BsJaLED"],
> +                                      ["CurrentChannel", "hotcue_23_activate", "StantonSCS3d.BsKaLED"],
> +                                      ["CurrentChannel", "hotcue_24_activate", "StantonSCS3d.BsLaLED"] ],
>                             "trig3":[ ["CurrentChannel", "pfl", "StantonSCS3d.B11LED"],
> -                                                                         ["CurrentChannel", "hotcue_25_enabled", "StantonSCS3d.BsALED"],
> -                                                                         ["CurrentChannel", "hotcue_26_enabled", "StantonSCS3d.BsBLED"],
> -                                                                         ["CurrentChannel", "hotcue_27_enabled", "StantonSCS3d.BsCLED"],
> -                                                                         ["CurrentChannel", "hotcue_28_enabled", "StantonSCS3d.BsDLED"],
> -                                                                         ["CurrentChannel", "hotcue_29_enabled", "StantonSCS3d.BsELED"],
> -                                                                         ["CurrentChannel", "hotcue_30_enabled", "StantonSCS3d.BsFLED"],
> -                                                                         ["CurrentChannel", "hotcue_31_enabled", "StantonSCS3d.BsGLED"],
> -                                                                         ["CurrentChannel", "hotcue_25_activate", "StantonSCS3d.BsAaLED"],
> -                                                                         ["CurrentChannel", "hotcue_26_activate", "StantonSCS3d.BsBaLED"],
> -                                                                         ["CurrentChannel", "hotcue_27_activate", "StantonSCS3d.BsCaLED"],
> -                                                                         ["CurrentChannel", "hotcue_28_activate", "StantonSCS3d.BsDaLED"],
> -                                                                         ["CurrentChannel", "hotcue_29_activate", "StantonSCS3d.BsEaLED"],
> -                                                                         ["CurrentChannel", "hotcue_30_activate", "StantonSCS3d.BsFaLED"],
> -                                                                         ["CurrentChannel", "hotcue_31_activate", "StantonSCS3d.BsGaLED"] ],
> +                                      ["CurrentChannel", "hotcue_25_enabled", "StantonSCS3d.BsALED"],
> +                                      ["CurrentChannel", "hotcue_26_enabled", "StantonSCS3d.BsBLED"],
> +                                      ["CurrentChannel", "hotcue_27_enabled", "StantonSCS3d.BsCLED"],
> +                                      ["CurrentChannel", "hotcue_28_enabled", "StantonSCS3d.BsDLED"],
> +                                      ["CurrentChannel", "hotcue_29_enabled", "StantonSCS3d.BsELED"],
> +                                      ["CurrentChannel", "hotcue_30_enabled", "StantonSCS3d.BsFLED"],
> +                                      ["CurrentChannel", "hotcue_31_enabled", "StantonSCS3d.BsGLED"],
> +                                      ["CurrentChannel", "hotcue_25_activate", "StantonSCS3d.BsAaLED"],
> +                                      ["CurrentChannel", "hotcue_26_activate", "StantonSCS3d.BsBaLED"],
> +                                      ["CurrentChannel", "hotcue_27_activate", "StantonSCS3d.BsCaLED"],
> +                                      ["CurrentChannel", "hotcue_28_activate", "StantonSCS3d.BsDaLED"],
> +                                      ["CurrentChannel", "hotcue_29_activate", "StantonSCS3d.BsEaLED"],
> +                                      ["CurrentChannel", "hotcue_30_activate", "StantonSCS3d.BsFaLED"],
> +                                      ["CurrentChannel", "hotcue_31_activate", "StantonSCS3d.BsGaLED"] ],
>                             "vinyl":[ ["CurrentChannel", "pfl", "StantonSCS3d.B11LED"],
>                                       ["CurrentChannel", "VuMeter", "StantonSCS3d.VUMeterLEDs"] ],
>                             "vinyl2":[["CurrentChannel", "pfl", "StantonSCS3d.B11LED"],
>                                       ["CurrentChannel", "VuMeter", "StantonSCS3d.VUMeterLEDs"] ],
>                             "vinyl3":[],
>                             "deck":[  ["[Master]","balance","StantonSCS3d.pitchLEDs"],
> -                                                                         ["[Master]","volume","StantonSCS3d.MasterVolumeLEDs"],
> +                                      ["[Master]","volume","StantonSCS3d.MasterVolumeLEDs"],
>                                       ["[Master]","headMix","StantonSCS3d.headMixLEDs"],
>                                       ["[Master]","headVolume","StantonSCS3d.headVolLEDs"],
>                                       ["[Master]","crossfader","StantonSCS3d.crossFaderLEDs"] ],
> @@ -399,7 +399,7 @@
>  }
>
>  StantonSCS3d.gain = function (channel, control, value) {
> -        var currentMode = StantonSCS3d.mode_store["[Channel"+StantonSCS3d.deck+"]"];
> +    var currentMode = StantonSCS3d.mode_store["[Channel"+StantonSCS3d.deck+"]"];
>     // Ignore if in DECK mode
>     if (currentMode == "deck") return;
>     if (StantonSCS3d.modifier[currentMode]==1) return;
> @@ -765,9 +765,9 @@
>                     midi.sendShortMsg(byte1,i,0x40); // Set center slider to black
>             break;
>         case "deck":
> -                       StantonSCS3d.state["forceGain"]=true;
> -                       StantonSCS3d.gainLEDs(engine.getValue("[Channel"+StantonSCS3d.deck+"]","volume"));  // Restore Gain LEDs
> -                       StantonSCS3d.state["forceGain"]=false;
> +            StantonSCS3d.state["forceGain"]=true;
> +            StantonSCS3d.gainLEDs(engine.getValue("[Channel"+StantonSCS3d.deck+"]","volume"));  // Restore Gain LEDs
> +            StantonSCS3d.state["forceGain"]=false;
>             StantonSCS3d.connectDeckSignals(channel,false,"common");    // Connect static common signals
>             break;
>     }
> @@ -1096,12 +1096,12 @@
>             var button = 0x5C-2*Math.floor(value/32);
>             if (StantonSCS3d.triggerS4==button) return; // prevent retriggering before releasing the button
>
> -                       // Deactivate any previous hot cue in the same slider touch
> +            // Deactivate any previous hot cue in the same slider touch
>             if (StantonSCS3d.state["S4hotcue"] != -1) {
> -                               engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.state["S4hotcue"]+"_activate",0);
> -                               StantonSCS3d.state["S4hotcue"]=-1;
> -                       }
> -
> +                engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.state["S4hotcue"]+"_activate",0);
> +                StantonSCS3d.state["S4hotcue"]=-1;
> +            }
> +
>             StantonSCS3d.triggerS4 = button;
>
>             var index = currentMode.charAt(currentMode.length-1);
> @@ -1117,7 +1117,7 @@
>             // If hotcue X is set, seeks the player to hotcue X's position.
>             // If hotcue X is not set, sets hotcue X to the current play position.
>             engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.hotCues[index][button]+"_activate",1);
> -            StantonSCS3d.state["S4hotcue"]=StantonSCS3d.hotCues[index][button];        // so it can be deactivated later
> +            StantonSCS3d.state["S4hotcue"]=StantonSCS3d.hotCues[index][button];    // so it can be deactivated later
>             break;
>     }
>  }
> @@ -1265,16 +1265,16 @@
>             break;
>         case "loop2":
>         case "loop3":
> -                       StantonSCS3d.S4buttonLights(channel,false,StantonSCS3d.triggerS4);
> +            StantonSCS3d.S4buttonLights(channel,false,StantonSCS3d.triggerS4);
>             StantonSCS3d.triggerS4 = 0xFF;
>             break;
>         case "trig":
>         case "trig2":
>         case "trig3":
> -                       if (StantonSCS3d.state["S4hotcue"] != -1) {
> -                               engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.state["S4hotcue"]+"_activate",0);
> -                               StantonSCS3d.state["S4hotcue"]=-1;
> -                       }
> +            if (StantonSCS3d.state["S4hotcue"] != -1) {
> +                engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.state["S4hotcue"]+"_activate",0);
> +                StantonSCS3d.state["S4hotcue"]=-1;
> +            }
>             //StantonSCS3d.S4buttonLights(channel,false,StantonSCS3d.triggerS4);
>             StantonSCS3d.triggerS4 = 0xFF;
>             break;
> @@ -1416,7 +1416,7 @@
>         switch (currentMode) {
>             case "loop2":
>             case "loop3":
> -                               midi.sendShortMsg(byte1,buttonLight,0x01); // Turn on button light
> +                midi.sendShortMsg(byte1,buttonLight,0x01); // Turn on button light
>                 // Multiple pitch points
>  //                 if (StantonSCS3d.modifier[currentMode]==1) {    // Delete a pitch point if the mode button is held
>  //                     StantonSCS3d.pitchPoints[index][button] = -0.1;
> @@ -1440,7 +1440,7 @@
>
>                 if (StantonSCS3d.modifier[currentMode]==1) { // Delete a cue point
>                     engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.hotCues[index][control]+"_clear",1);
> -                                       engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.hotCues[index][control]+"_clear",0);
> +                    engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.hotCues[index][control]+"_clear",0);
>                     break;
>                 }
>                 // If hotcue X is set, seeks the player to hotcue X's position.
> @@ -1451,16 +1451,16 @@
>         return;
>     }
>
> -       switch (currentMode) {
> -               case "trig":
> +    switch (currentMode) {
> +        case "trig":
>         case "trig2":
> -               case "trig3":
> -                       engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.hotCues[index][control]+"_activate",0);
> -                       break;
> -               default:
> -                       midi.sendShortMsg(byte1,buttonLight,0x00); // Turn off activated button LED
> -                       break;
> -       }
> +        case "trig3":
> +            engine.setValue("[Channel"+StantonSCS3d.deck+"]","hotcue_"+StantonSCS3d.hotCues[index][control]+"_activate",0);
> +            break;
> +        default:
> +            midi.sendShortMsg(byte1,buttonLight,0x00); // Turn off activated button LED
> +            break;
> +    }
>  }
>
>
> @@ -1507,18 +1507,18 @@
>  // ---- Hot cues ----
>
>  StantonSCS3d.BsLED = function (value, button, activate) {
> -       var light = button;
> -
> -       if (activate) {
> -               // Button-pressed (hot cue activated) LED
> -               if (StantonSCS3d.markHotCues == "red") light = StantonSCS3d.buttonLEDs[button];
> -       }
> -       else {
> -               // Hot cue set LED
> -               if (StantonSCS3d.markHotCues != "red") light = StantonSCS3d.buttonLEDs[button];
> -       }
> +    var light = button;
> +
> +    if (activate) {
> +        // Button-pressed (hot cue activated) LED
> +        if (StantonSCS3d.markHotCues == "red") light = StantonSCS3d.buttonLEDs[button];
> +    }
> +    else {
> +        // Hot cue set LED
> +        if (StantonSCS3d.markHotCues != "red") light = StantonSCS3d.buttonLEDs[button];
> +    }
>
> -       var byte1 = 0x90 + StantonSCS3d.channel;
> +    var byte1 = 0x90 + StantonSCS3d.channel;
>     if (value != 0) midi.sendShortMsg(byte1,light,0x01);
>     else midi.sendShortMsg(byte1,light,0x00);
>  }
>
> === modified file 'mixxx/src/midi/mididevice.cpp'
> --- mixxx/src/midi/mididevice.cpp       2010-11-15 18:04:41 +0000
> +++ mixxx/src/midi/mididevice.cpp       2010-11-17 00:57:57 +0000
> @@ -242,11 +242,17 @@
>
>     if (p) //Only pass values on to valid ControlObjects.
>     {
> -        double newValue = m_pMidiMapping->ComputeValue(mixxxControl.getMidiOption(), p->GetMidiValue(), value);
> +        double currMixxxControlValue = p->GetMidiValue();
> +        MidiOption currMidiOption = mixxxControl.getMidiOption();
> +        // FIXME: We should accept multiple options, since soft-takeover would apply in addition
> +        //  Remove this if clause and newValue=value assignment when that's done
> +        double newValue = value;
> +        if (currMidiOption!=MIDI_OPT_SOFT_TAKEOVER)
> +            newValue = m_pMidiMapping->ComputeValue(currMidiOption, currMixxxControlValue, value);
>
>         // ControlPushButton ControlObjects only accept NOTE_ON, so if the midi
>         // mapping is <button> we override the Midi 'status' appropriately.
> -        switch (mixxxControl.getMidiOption()) {
> +        switch (currMidiOption) {
>             case MIDI_OPT_BUTTON:
>             case MIDI_OPT_SWITCH: status = MIDI_STATUS_NOTE_ON; break; // Buttons and Switches are
>                                                                        // treated the same, except
> @@ -254,6 +260,31 @@
>                                                                        // computed differently.
>             default: break;
>         }
> +
> +        // Soft-takeover is processed in addition to any other options
> +        if (currMidiOption == MIDI_OPT_SOFT_TAKEOVER) {
> +            bool ignore = false;
> +            if (!m_softTakeoverTimes.contains(mixxxControl)) {
> +                uint t = QDateTime::currentDateTime().toTime_t()*1000+QDateTime::currentDateTime().toString("zzz").toUInt();
> +                m_softTakeoverTimes.insert(mixxxControl,t);
> +            }
> +
> +            // We only want to ignore the MIDI controller when all of the following are true:
> +            //  - its new value is far away from the MixxxControl
> +            //  - it's been awhile since the last MIDI message for this control affected it
> +
> +            double newDifference = currMixxxControlValue - newValue;
> +            uint currentTime = QDateTime::currentDateTime().toTime_t()*1000+QDateTime::currentDateTime().toString("zzz").toUInt();
> +            if (fabs(newDifference)>3
> +                && (currentTime - m_softTakeoverTimes.value(mixxxControl)) > 50) {
> +                ignore = true;
> +            }
> +            if (ignore) return;
> +            //  Update the time only if the value is not ignored
> +            //qint64 t = QDateTime::currentDateTime().toMSecsSinceEpoch();  // Requires Qt 4.7
> +            uint t = QDateTime::currentDateTime().toTime_t()*1000+QDateTime::currentDateTime().toString("zzz").toUInt();
> +            m_softTakeoverTimes.insert(mixxxControl,t); // Replaces any previous value for this MixxxControl
> +        }
>
>         ControlObject::sync();
>
> @@ -270,7 +301,7 @@
>     QMutexLocker locker(&m_mutex); //Lots of returns in this function. Keeps things simple.
>
>     QString message = m_strDeviceName+": [";
> -    for(int i=0; i<length; i++) {
> +    for(uint i=0; i<length; i++) {
>         message += QString("%1%2")
>                     .arg(data[i], 2, 16, QChar('0')).toUpper()
>                     .arg((i<(length-1))?' ':']');
>
> === modified file 'mixxx/src/midi/mididevice.h'
> --- mixxx/src/midi/mididevice.h 2010-11-15 18:04:41 +0000
> +++ mixxx/src/midi/mididevice.h 2010-11-17 00:57:57 +0000
> @@ -33,6 +33,7 @@
>
>  #include <QtCore>
>  #include "midimessage.h"
> +#include "mixxxcontrol.h"
>
>  //Forward declarations
>  class MidiMapping;
> @@ -113,6 +114,7 @@
>             MIDI message while we're trying to shut it down. See
>             MidiDevice::shutdown() for more details. */
>         bool m_bSendInhibit;
> +        QHash<MixxxControl,uint> m_softTakeoverTimes;
>  };
>
>  #endif
>
> === modified file 'mixxx/src/midi/midimapping.cpp'
> --- mixxx/src/midi/midimapping.cpp      2010-11-15 18:04:41 +0000
> +++ mixxx/src/midi/midimapping.cpp      2010-11-17 00:57:57 +0000
> @@ -705,7 +705,7 @@
>  */
>  void MidiMapping::savePreset(QString path) {
>     qDebug() << "Writing MIDI preset file" << path;
> -    m_mappingLock.lock();
> +    QMutexLocker lock(&m_mappingLock);
>     QFile output(path);
>     if (!output.open(QIODevice::WriteOnly | QIODevice::Truncate)) return;
>     QTextStream outputstream(&output);
> @@ -714,7 +714,6 @@
>     // Save the DOM to the XML file
>     docBindings.save(outputstream, 4);
>     output.close();
> -    m_mappingLock.unlock();
>  }
>
>  /* applyPreset()
>
> === modified file 'mixxx/src/midi/midioptiondelegate.cpp'
> --- mixxx/src/midi/midioptiondelegate.cpp       2010-11-15 18:04:41 +0000
> +++ mixxx/src/midi/midioptiondelegate.cpp       2010-11-17 00:57:57 +0000
> @@ -21,6 +21,7 @@
>  #define MIDI_OPT_STRING_HERC_JOG    "HercJog"    // Generic hercules wierd range correction
>  #define MIDI_OPT_STRING_SPREAD64    "Spread64"   // Accelerated difference from 64
>  #define MIDI_OPT_STRING_SELECTKNOB  "SelectKnob" // Relative knob which can be turned forever and outputs a signed value.
> +#define MIDI_OPT_STRING_SOFT_TAKEOVER   "SoftTakeover"  // Prevents sudden changes when hardware & software MixxxControl value differ
>  #define MIDI_OPT_STRING_SCRIPT      "Script"     // Maps a MIDI control to a custom MixxxScript function
>
>  MidiOptionDelegate::MidiOptionDelegate(QObject *parent)
> @@ -50,7 +51,8 @@
>            case MIDI_OPT_HERC_JOG: text = MIDI_OPT_STRING_HERC_JOG; break;
>            case MIDI_OPT_SPREAD64: text = MIDI_OPT_STRING_SPREAD64; break;
>            case MIDI_OPT_SELECTKNOB: text = MIDI_OPT_STRING_SELECTKNOB; break;
> -           case MIDI_OPT_SCRIPT: text = MIDI_OPT_STRING_SCRIPT; break;
> +           case MIDI_OPT_SOFT_TAKEOVER: text = MIDI_OPT_STRING_SOFT_TAKEOVER; break;
> +           case MIDI_OPT_SCRIPT: text = MIDI_OPT_STRING_SCRIPT; break;
>            default: text = QString("%1").arg(midioption); break;
>         }
>
> @@ -77,6 +79,7 @@
>     editor->addItem(MIDI_OPT_STRING_HERC_JOG);
>     editor->addItem(MIDI_OPT_STRING_SPREAD64);
>     editor->addItem(MIDI_OPT_STRING_SELECTKNOB);
> +    editor->addItem(MIDI_OPT_STRING_SOFT_TAKEOVER);
>     editor->addItem(MIDI_OPT_STRING_SCRIPT);
>
>     return editor;
> @@ -102,7 +105,8 @@
>         case MIDI_OPT_HERC_JOG:     comboIdx = 8; break;
>         case MIDI_OPT_SPREAD64:     comboIdx = 9; break;
>         case MIDI_OPT_SELECTKNOB:   comboIdx = 10; break;
> -        case MIDI_OPT_SCRIPT:       comboIdx = 11; break;
> +        case MIDI_OPT_SOFT_TAKEOVER:comboIdx = 11; break;
> +        case MIDI_OPT_SCRIPT:       comboIdx = 12; break;
>         default: comboIdx = 0; break;
>     }
>     comboBox->setCurrentIndex(comboIdx);
> @@ -116,6 +120,7 @@
>     //comboBox->interpretText();
>     //Get the text from the combobox and turn it into a MidiMessage integer.
>     QString text = comboBox->currentText();
> +    // Sigh, it sucks that we can't switch() on a string...
>     if (text == MIDI_OPT_STRING_NORMAL) //These come from the MidiOption enum (mixxxcontrol.h)
>         midioption = MIDI_OPT_NORMAL;
>     else if (text == MIDI_OPT_STRING_INVERT)
> @@ -138,8 +143,10 @@
>         midioption = MIDI_OPT_SPREAD64;
>     else if (text == MIDI_OPT_STRING_SELECTKNOB)
>         midioption = MIDI_OPT_SELECTKNOB;
> +    else if (text == MIDI_OPT_STRING_SOFT_TAKEOVER)
> +        midioption = MIDI_OPT_SOFT_TAKEOVER;
>     else if (text == MIDI_OPT_STRING_SCRIPT)
> -        midioption = MIDI_OPT_SCRIPT;
> +        midioption = MIDI_OPT_SCRIPT;
>     model->setData(index, midioption, Qt::EditRole);
>  }
>
>
> === modified file 'mixxx/src/midi/midioutputmapping.h'
> --- mixxx/src/midi/midioutputmapping.h  2010-11-15 18:04:41 +0000
> +++ mixxx/src/midi/midioutputmapping.h  2010-11-17 00:57:57 +0000
> @@ -1,7 +1,7 @@
>
>  #include <QtCore>
>
> -class MidiOutputMapping : public QHash<MixxxControl, MidiMessage>
> +class MidiOutputMapping : public QMultiHash<MixxxControl, MidiMessage>
>  {
>        public:
>                MidiOutputMapping() {};
>
> === modified file 'mixxx/src/midi/midiscriptengine.cpp'
> --- mixxx/src/midi/midiscriptengine.cpp 2010-11-07 05:50:26 +0000
> +++ mixxx/src/midi/midiscriptengine.cpp 2010-11-17 00:57:57 +0000
> @@ -696,7 +696,31 @@
>     ControlObjectThread *cot = getControlObjectThread(group, name);
>
>     if(cot != NULL) {
> -        cot->slotSet(newValue);
> +        // Soft-takeover
> +        bool ignore = false;
> +        MixxxControl mixxxControl = MixxxControl(group,name);
> +        if (m_softTakeoverTimes.contains(mixxxControl)) {
> +            // We only want to ignore the MIDI controller when all of the following are true:
> +            //  - its new value is far away from the MixxxControl
> +            //  - it's been awhile since the last MIDI message for this control affected it
> +
> +            double difference = getValue(group,name) - newValue;
> +            uint currentTime = QDateTime::currentDateTime().toTime_t()*1000+QDateTime::currentDateTime().toString("zzz").toUInt();
> +            // FIXME: Argh, need a way to find out the max & min values of the
> +            //  ControlObject in question instead of hard-coding the difference threshold
> +            if (fabs(difference)>0.05
> +                && (currentTime - m_softTakeoverTimes.value(mixxxControl)) > 50) {
> +                ignore = true;
> +            }
> +            if (!ignore) {
> +                //  Update the time only if the value is not ignored
> +                //qint64 t = QDateTime::currentDateTime().toMSecsSinceEpoch();  // Requires Qt 4.7
> +                uint t = QDateTime::currentDateTime().toTime_t()*1000+QDateTime::currentDateTime().toString("zzz").toUInt();
> +                m_softTakeoverTimes.insert(mixxxControl,t); // Replaces any previous value for this MixxxControl
> +            }
> +        }
> +
> +        if (!ignore) cot->slotSet(newValue);
>     }
>
>  }
> @@ -1220,3 +1244,19 @@
>
>     m_ramp[deck] = true;    // Activate the ramping in scratchProcess()
>  }
> +
> +/*  -------- ------------------------------------------------------
> +    Purpose: [En/dis]ables soft-takeover status for a particular MixxxControl
> +    Input:   MixxxControl group and key values,
> +                whether to set the soft-takeover status or not
> +    Output:  -
> +    -------- ------------------------------------------------------ */
> +void MidiScriptEngine::softTakeover(QString group, QString name, bool set) {
> +    MixxxControl mc = MixxxControl(group,name);
> +    if (set) {
> +        //qint64 t = QDateTime::currentDateTime().toMSecsSinceEpoch();  // Requires Qt 4.7
> +        uint t = QDateTime::currentDateTime().toTime_t()*1000+QDateTime::currentDateTime().toString("zzz").toUInt();
> +        m_softTakeoverTimes.insert(mc,t);
> +    }
> +    else m_softTakeoverTimes.remove(mc);
> +}
> \ No newline at end of file
>
> === modified file 'mixxx/src/midi/midiscriptengine.h'
> --- mixxx/src/midi/midiscriptengine.h   2010-11-07 05:15:18 +0000
> +++ mixxx/src/midi/midiscriptengine.h   2010-11-17 00:57:57 +0000
> @@ -23,6 +23,7 @@
>  #include "configobject.h"
>  #include "midimessage.h"
>  #include "pitchfilter.h"
> +#include "mixxxcontrol.h"
>  class MidiDevice;
>
>  //Forward declaration(s)
> @@ -56,6 +57,7 @@
>     Q_INVOKABLE void scratchEnable(int deck, int intervalsPerRev, float rpm, float alpha, float beta);
>     Q_INVOKABLE void scratchTick(int deck, int interval);
>     Q_INVOKABLE void scratchDisable(int deck);
> +    Q_INVOKABLE void softTakeover(QString group, QString name, bool set);
>
>  public slots:
>     void slotValueChanged(double value);
> @@ -112,6 +114,7 @@
>     QMutex m_scriptEngineLock;
>     QHash<ConfigKey, ControlObjectThread*> m_controlCache;
>     QHash<int, QPair<QString, bool> > m_timers;
> +    QHash<MixxxControl,uint> m_softTakeoverTimes;
>
>     // Scratching functions & variables
>     void scratchProcess(int timerId);
>
> === modified file 'mixxx/src/mixxxcontrol.cpp'
> --- mixxx/src/mixxxcontrol.cpp  2010-11-15 18:04:41 +0000
> +++ mixxx/src/mixxxcontrol.cpp  2010-11-17 00:57:57 +0000
> @@ -35,7 +35,7 @@
>         strMidiOption = "normal";
>     }
>
> -    if (strMidiOption == "normal")
> +    if (strMidiOption == "normal")  // can't switch() on a string
>         m_midiOption = MIDI_OPT_NORMAL;
>     else if (strMidiOption == "invert")
>         m_midiOption = MIDI_OPT_INVERT;
> @@ -57,6 +57,8 @@
>         m_midiOption = MIDI_OPT_SPREAD64;
>     else if (strMidiOption == "selectknob")
>         m_midiOption = MIDI_OPT_SELECTKNOB;
> +    else if (strMidiOption == "soft-takeover")
> +        m_midiOption = MIDI_OPT_SOFT_TAKEOVER;
>     else if (strMidiOption == "script-binding")
>         m_midiOption = MIDI_OPT_SCRIPT;
>     else {
> @@ -114,7 +116,7 @@
>     QDomElement optionsNode = nodeMaker.createElement("options");
>     QString strMidiOption;
>     int iMidiOption = this->getMidiOption();
> -    if (iMidiOption == MIDI_OPT_NORMAL)
> +    if (iMidiOption == MIDI_OPT_NORMAL) // can't switch() on a string
>         strMidiOption = "normal";
>     else if (iMidiOption == MIDI_OPT_INVERT)
>         strMidiOption = "invert";
> @@ -136,6 +138,8 @@
>         strMidiOption = "spread64";
>     else if (iMidiOption == MIDI_OPT_SELECTKNOB)
>         strMidiOption = "selectknob";
> +    else if (iMidiOption == MIDI_OPT_SOFT_TAKEOVER)
> +        strMidiOption = "soft-takeover";
>     else if (iMidiOption == MIDI_OPT_SCRIPT)
>         strMidiOption = "script-binding";
>     else {
>
> === modified file 'mixxx/src/mixxxcontrol.h'
> --- mixxx/src/mixxxcontrol.h    2010-11-15 18:04:41 +0000
> +++ mixxx/src/mixxxcontrol.h    2010-11-17 00:57:57 +0000
> @@ -16,6 +16,8 @@
>     MIDI_OPT_HERC_JOG         = 8, // Generic hercules wierd range correction
>     MIDI_OPT_SPREAD64         = 9, // Accelerated difference from 64
>     MIDI_OPT_SELECTKNOB       = 10,// Relative knob which can be turned forever and outputs a signed value.
> +
> +    MIDI_OPT_SOFT_TAKEOVER    = 40,// Prevents sudden changes when hardware position differs from software value
>
>     MIDI_OPT_SCRIPT           = 50,// Maps a MIDI control to a custom MixxxScript function
>  } MidiOption;
>
>
>