substantial overhaul of MCU code - no more separate thread, just connect to signals...

[ardour.git] / libs / surfaces / mackie / interface.cc

diff --git a/libs/surfaces/mackie/interface.cc b/libs/surfaces/mackie/interface.cc
index 500854fe7193befa2d8f54641fd36ec24f6c613a..067941484c703779859a91ca96355caaf7a55ff0 100644 (file)
--- a/libs/surfaces/mackie/interface.cc
+++ b/libs/surfaces/mackie/interface.cc
@@ -15,40 +15,75 @@
        along with this program; if not, write to the Free Software
        Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 */
-#include <control_protocol/control_protocol.h>
+#include "control_protocol/control_protocol.h"
 #include "mackie_control_protocol.h"
 
+#include "ardour/rc_configuration.h"
+
+#include "pbd/error.h"
+
 #include <stdexcept>
 
 using namespace ARDOUR;
+using namespace PBD;
 using namespace std;
 
 ControlProtocol*
-new_mackie_protocol (ControlProtocolDescriptor* descriptor, Session* s)
+new_mackie_protocol (ControlProtocolDescriptor*, Session* s)
 {
-       MackieControlProtocol * mcp = 0;
-       try
+       if ( Config->get_mmc_port_name().substr(0,3) == "mcu" )
+       {
+               error << "mcu already used as mmc port" << endmsg;
+       }
+       else if ( Config->get_mtc_port_name().substr(0,3) == "mcu" )
+       {
+               error << "mcu already used as mtc port" << endmsg;
+       }
+       else if ( Config->get_midi_port_name().substr(0,3) == "mcu" )
        {
-               mcp = new MackieControlProtocol (*s);
-               mcp->set_active( true );
+               error << "mcu already used as midi port" << endmsg;
        }
-       catch( exception & e )
+       else
        {
-               cout << "Error instantiating MackieControlProtocol: " << e.what() << endl;
-               delete mcp;
-               mcp = 0;
+               // no one else is using the port, so try instantiate the object
+               MackieControlProtocol * mcp = 0;
+               try
+               {
+                       mcp = new MackieControlProtocol (*s);
+                       mcp->set_active( true );
+               }
+               catch( exception & e )
+               {
+                       error << "Error instantiating MackieControlProtocol: " << e.what() << endmsg;
+                       delete mcp;
+                       mcp = 0;
+               }
+               return mcp;
        }
-       return mcp;
+       return 0;
 }
 
 void
-delete_mackie_protocol (ControlProtocolDescriptor* descriptor, ControlProtocol* cp)
+delete_mackie_protocol (ControlProtocolDescriptor*, ControlProtocol* cp)
 {
-       delete cp;
+       try
+       {
+               delete cp;
+       }
+       catch ( exception & e )
+       {
+               cout << "Exception caught trying to destroy MackieControlProtocol: " << e.what() << endl;
+       }
 }
 
+/**
+       This is called on startup to check whether the lib should be loaded.
+
+       So anything that can be changed in the UI should not be used here to
+       prevent loading of the lib.
+*/
 bool
-probe_mackie_protocol (ControlProtocolDescriptor* descriptor)
+probe_mackie_protocol (ControlProtocolDescriptor*)
 {
        return MackieControlProtocol::probe();
 }
@@ -59,7 +94,11 @@ static ControlProtocolDescriptor mackie_descriptor = {
        ptr : 0,
        module : 0,
        mandatory : 0,
-       supports_feedback : true,
+       // actually, the surface does support feedback, but all this
+       // flag does is show a submenu on the UI, which is useless for the mackie
+       // because feedback is always on. In any case, who'd want to use the
+       // mcu without the motorised sliders doing their thing?
+       supports_feedback : false,
        probe : probe_mackie_protocol,
        initialize : new_mackie_protocol,
        destroy : delete_mackie_protocol