Fix unused parameter warnings since GCC apparently doesn't feel like listening to...
authorDavid Robillard <d@drobilla.net>
Wed, 21 Oct 2009 00:15:42 +0000 (00:15 +0000)
committerDavid Robillard <d@drobilla.net>
Wed, 21 Oct 2009 00:15:42 +0000 (00:15 +0000)
git-svn-id: svn://localhost/ardour2/branches/3.0@5835 d708f5d6-7413-0410-9779-e7cbd77b26cf

33 files changed:
gtk2_ardour/editor.cc
gtk2_ardour/keyboard.cc
gtk2_ardour/time_axis_view.cc
gtk2_ardour/ui_config.cc
libs/ardour/audio_diskstream.cc
libs/ardour/audio_track.cc
libs/ardour/audiosource.cc
libs/ardour/control_protocol_manager.cc
libs/ardour/crossfade.cc
libs/ardour/file_source.cc
libs/ardour/io.cc
libs/ardour/location.cc
libs/ardour/lv2_plugin.cc
libs/ardour/midi_diskstream.cc
libs/ardour/midi_model.cc
libs/ardour/midi_source.cc
libs/ardour/midi_track.cc
libs/ardour/mute_master.cc
libs/ardour/panner.cc
libs/ardour/processor.cc
libs/ardour/rc_configuration.cc
libs/ardour/region.cc
libs/ardour/route_group.cc
libs/ardour/session_command.cc
libs/ardour/session_configuration.cc
libs/ardour/session_metadata.cc
libs/ardour/source.cc
libs/ardour/tempo.cc
libs/ardour/user_bundle.cc
libs/midi++2/midnam_patch.cc
libs/pbd/controllable.cc
libs/surfaces/generic_midi/midicontrollable.cc
libs/surfaces/mackie/mackie_control_protocol.cc

index 05bae8a9e575b03a8c296ace2b43649fe9f8658d..6d55327e45b253d44c96e0d87fa3852c1f9c0889 100644 (file)
@@ -2228,7 +2228,7 @@ Editor::set_edit_point_preference (EditPoint ep, bool force)
 }
 
 int
-Editor::set_state (const XMLNode& node, int version)
+Editor::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        XMLNode* geometry;
index 635ac8137c6df8fd30012446340456df092e952e..99154a993329df06f5aa1a09a140a3be94a78e0b 100644 (file)
@@ -157,8 +157,8 @@ Keyboard::get_state (void)
        return *node;
 }
 
-int 
-Keyboard::set_state (const XMLNode& node, int version)
+int
+Keyboard::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
 
index 21c5bd9f1111938372a7daa454b9feaeeb94ae13..56f23a0ac2daacc2f94660c51516c5a8411ba2a8 100644 (file)
@@ -1001,7 +1001,7 @@ TimeAxisView::get_state ()
 }
 
 int
-TimeAxisView::set_state (const XMLNode& node, int version)
+TimeAxisView::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty *prop;
 
index d0878073b7f8b08cdc3e3490b8b4210cf0da6005..1ba298c2ec0c9d8d3073ba87f0baa8bdc815a8c1 100644 (file)
@@ -221,7 +221,7 @@ UIConfiguration::get_variables (std::string which_node)
 }
 
 int
-UIConfiguration::set_state (const XMLNode& root, int version)
+UIConfiguration::set_state (const XMLNode& root, int /*version*/)
 {
        if (root.name() != "Ardour") {
                return -1;
index 59fe0aecdfaa9cef7adb7d40f71a29419eca99a8..aa002130f88b5609bb50ff88511d1424c5960797 100644 (file)
@@ -1768,7 +1768,7 @@ AudioDiskstream::get_state ()
 }
 
 int
-AudioDiskstream::set_state (const XMLNode& node, int version)
+AudioDiskstream::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        XMLNodeList nlist = node.children();
index c1862fb5b0a0e21fb3ce480a84fe76d98f05d9fa..2c38e37dee808b138133c4ea19620046f45230c9 100644 (file)
@@ -56,7 +56,7 @@ AudioTrack::AudioTrack (Session& sess, string name, Route::Flag flag, TrackMode
        use_new_diskstream ();
 }
 
-AudioTrack::AudioTrack (Session& sess, const XMLNode& node, int version)
+AudioTrack::AudioTrack (Session& sess, const XMLNode& node, int /*version*/)
        : Track (sess, node)
 {
        _set_state (node, Stateful::loading_state_version, false);
index 93e201b8c2843171a86208fd77338b0d0de793a1..4b9e53b66551262fb22dd6ba59e68a687b407229 100644 (file)
@@ -118,7 +118,7 @@ AudioSource::get_state ()
 }
 
 int
-AudioSource::set_state (const XMLNode& node, int version)
+AudioSource::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
 
index 0e687988d2d480b37827dcddfd67fcf5fbf8825b..724d60c389614108c1ab699bd454947c674512f0 100644 (file)
@@ -288,7 +288,7 @@ ControlProtocolManager::cpi_by_name (string name)
 }
 
 int
-ControlProtocolManager::set_state (const XMLNode& node, int version)
+ControlProtocolManager::set_state (const XMLNode& node, int /*version*/)
 {
        XMLNodeList clist;
        XMLNodeConstIterator citer;
index c69eeea21da2aecb0c54dc4d5b3f761c834c5bd4..7bbad9a1d1d5bf0f069845d487648c7ef619c353 100644 (file)
@@ -720,7 +720,7 @@ Crossfade::get_state ()
 }
 
 int
-Crossfade::set_state (const XMLNode& node, int version)
+Crossfade::set_state (const XMLNode& node, int /*version*/)
 {
        XMLNodeConstIterator i;
        XMLNodeList children;
index 22e9061d9c6d9652a1d8a53ce94941f17e33ae6d..e5232f23849c361cb9ebaffc46c52e05e35447c6 100644 (file)
@@ -104,7 +104,7 @@ FileSource::init (const ustring& pathstr, bool must_exist)
 }
 
 int
-FileSource::set_state (const XMLNode& node, int version)
+FileSource::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
 
index a56f2fd08305e196cdaa400400bb62688eeff191..a56f81781483951c8ef64c39c97c3f7adf7f3585 100644 (file)
@@ -739,7 +739,7 @@ IO::find_possible_bundle (const string &desired_name)
 }
 
 int
-IO::get_port_counts_2X (XMLNode const & node, int version, ChanCount& n, boost::shared_ptr<Bundle>& c)
+IO::get_port_counts_2X (XMLNode const & node, int /*version*/, ChanCount& n, boost::shared_ptr<Bundle>& /*c*/)
 {
        XMLProperty const * prop;
        XMLNodeList children = node.children ();
@@ -892,7 +892,7 @@ IO::make_connections (const XMLNode& node, int version, bool in)
 
 
 int
-IO::make_connections_2X (const XMLNode& node, int version, bool in)
+IO::make_connections_2X (const XMLNode& node, int /*version*/, bool in)
 {
        const XMLProperty* prop;
 
index 7b9e706c36bc9de7b07e93344dbb00f0141bd9a1..02d355d915dfb3d08abc3feb0d9b82199542f545 100644 (file)
@@ -347,7 +347,7 @@ Location::get_state (void)
 }
 
 int
-Location::set_state (const XMLNode& node, int version)
+Location::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty *prop;
 
@@ -674,7 +674,7 @@ Locations::get_state ()
 }
 
 int
-Locations::set_state (const XMLNode& node, int version)
+Locations::set_state (const XMLNode& node, int /*version*/)
 {
        XMLNodeList nlist;
        XMLNodeConstIterator niter;
index 38774104afa9bd90b042b32d6dbafad63b784ecf..678dfd47159e354bc6c0aedad6d132e5f9d8a8fc 100644 (file)
@@ -354,7 +354,7 @@ LV2Plugin::has_editor() const
 }
 
 int
-LV2Plugin::set_state(const XMLNode& node, int version)
+LV2Plugin::set_state(const XMLNode& node, int /*version*/)
 {
        XMLNodeList nodes;
        XMLProperty *prop;
index 2e9ac51059cca6bb109ac810c97465ea0989cf31..07cfbf215ce59e3f1d1226faf59bc21aff8710bd 100644 (file)
@@ -1232,7 +1232,7 @@ MidiDiskstream::get_state ()
 }
 
 int
-MidiDiskstream::set_state (const XMLNode& node, int version)
+MidiDiskstream::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        XMLNodeList nlist = node.children();
index 851ee49b46551c7a51d69118390ca9b1cb1da4cc..e35c5c6d9838b89ae66f98cfce5df4a69391a580 100644 (file)
@@ -260,7 +260,7 @@ MidiModel::DeltaCommand::unmarshal_note(XMLNode *xml_note)
 #define DELTA_COMMAND_ELEMENT "DeltaCommand"
 
 int
-MidiModel::DeltaCommand::set_state (const XMLNode& delta_command, int version)
+MidiModel::DeltaCommand::set_state (const XMLNode& delta_command, int /*version*/)
 {
        if (delta_command.name() != string(DELTA_COMMAND_ELEMENT)) {
                return 1;
@@ -646,7 +646,7 @@ MidiModel::DiffCommand::unmarshal_change(XMLNode *xml_change)
 }
 
 int
-MidiModel::DiffCommand::set_state(const XMLNode& diff_command, int version)
+MidiModel::DiffCommand::set_state(const XMLNode& diff_command, int /*version*/)
 {
        if (diff_command.name() != string(DIFF_COMMAND_ELEMENT)) {
                return 1;
index 20119e20b69bcc7c080f41104ebecf9536d76aa9..b925c810dad48251f2b34e2738e3ef65c8b75495 100644 (file)
@@ -95,7 +95,7 @@ MidiSource::get_state ()
 }
 
 int
-MidiSource::set_state (const XMLNode& node, int version)
+MidiSource::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
 
index 6b27d1b0099e700f623a1863a39fbc2871b65d73..d3f3638b6b6705810fd58db6fd2160128af4b4d7 100644 (file)
@@ -66,7 +66,7 @@ MidiTrack::MidiTrack (Session& sess, string name, Route::Flag flag, TrackMode mo
        _mode = mode;
 }
 
-MidiTrack::MidiTrack (Session& sess, const XMLNode& node, int version)
+MidiTrack::MidiTrack (Session& sess, const XMLNode& node, int /*version*/)
        : Track (sess, node, DataType::MIDI)
        , _immediate_events(1024) // FIXME: size?
        , _step_edit_ring_buffer(64) // FIXME: size?
index dd507aacebeed4f2f28b45013ab35584de2f6453..224eccdd515b55dc2975f6136a7b8550234ac905 100644 (file)
@@ -103,7 +103,7 @@ MuteMaster::get_value () const
 }
 
 int
-MuteMaster::set_state (const XMLNode& node, int version)
+MuteMaster::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
 
index 6c8dc153bbb8e2ad83b9ad61135743ef09cd4b54..c1292e67920e2bebf959e59afb5b0a03d72a9ef0 100644 (file)
@@ -172,7 +172,7 @@ StreamPanner::set_position (float xpos, float ypos, float zpos, bool link_call)
 }
 
 int
-StreamPanner::set_state (const XMLNode& node, int version)
+StreamPanner::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        XMLNodeConstIterator iter;
@@ -730,7 +730,7 @@ Multi2dPanner::state (bool /*full_state*/)
 }
 
 int
-Multi2dPanner::set_state (const XMLNode& node, int version)
+Multi2dPanner::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        float newx,newy;
index a52db49224163ab6eb08da969f1734a461387912..b46cc661e49c7dbb57f4175fbfdb9450bb9cb63e 100644 (file)
@@ -145,7 +145,7 @@ Processor::state (bool full_state)
 }
 
 int
-Processor::set_state_2X (const XMLNode & node, int version)
+Processor::set_state_2X (const XMLNode & node, int /*version*/)
 {
        XMLProperty const * prop;
 
index 725225b8d0a462fb2af2bb056af1c444573d48bd..c7d39c5b995b6e659c676d4ce1f665f66c41c7c1 100644 (file)
@@ -240,7 +240,7 @@ RCConfiguration::get_variables ()
 }
 
 int
-RCConfiguration::set_state (const XMLNode& root, int version)
+RCConfiguration::set_state (const XMLNode& root, int /*version*/)
 {
        if (root.name() != "Ardour") {
                return -1;
index e037131ae8d8e4bc0b5a98d963cbb0b46fbce32f..a881d9f1ed3c35a6f7f3ef20cee8856a2a07c660 100644 (file)
@@ -1174,7 +1174,7 @@ Region::get_state ()
 }
 
 int
-Region::set_live_state (const XMLNode& node, int version, Change& what_changed, bool send)
+Region::set_live_state (const XMLNode& node, int /*version*/, Change& what_changed, bool send)
 {
        const XMLNodeList& nlist = node.children();
        const XMLProperty *prop;
index 3d7ac3ef70df961afcc1205bc5b9b8e88fb924ff..792bcfc3e50da996062b4f7b0e0605641942e41f 100644 (file)
@@ -136,7 +136,7 @@ RouteGroup::get_state (void)
        return *node;
 }
 
-int 
+int
 RouteGroup::set_state (const XMLNode& node, int version)
 {
        if (version < 3000) {
@@ -161,14 +161,14 @@ RouteGroup::set_state (const XMLNode& node, int version)
 }
 
 int
-RouteGroup::set_state_2X (const XMLNode& node, int version)
+RouteGroup::set_state_2X (const XMLNode& node, int /*version*/)
 {
        XMLProperty const * prop;
-       
+
        if ((prop = node.property ("name")) != 0) {
                _name = prop->value();
        }
-       
+
        if ((prop = node.property ("flags")) != 0) {
                _flags = Flag (string_2_enum (prop->value(), _flags));
        }
index e7c9819d20dc538db153b2f5cc1b5796f0753bbb..d5de62b14260fc11e535468f964ea9e8728f3cb6 100644 (file)
@@ -170,7 +170,7 @@ Session::GlobalRouteStateCommand::GlobalRouteStateCommand (Session& s, const XML
 }
 
 int
-Session::GlobalRouteStateCommand::set_state (const XMLNode& node, int version)
+Session::GlobalRouteStateCommand::set_state (const XMLNode& node, int /*version*/)
 {
        GlobalRouteBooleanState states;
        XMLNodeList nlist;
@@ -483,7 +483,7 @@ Session::GlobalMeteringStateCommand::get_state()
 }
 
 int
-Session::GlobalMeteringStateCommand::set_state (const XMLNode& node, int version)
+Session::GlobalMeteringStateCommand::set_state (const XMLNode& node, int /*version*/)
 {
        GlobalRouteBooleanState states;
        XMLNodeList nlist;
index c35da623689b1d00b0c9aa2e41fbdeade71fb351..0f00dae1615b512340316ef4131ca39d1755af1a 100644 (file)
@@ -76,7 +76,7 @@ SessionConfiguration::get_variables ()
 
 
 int
-SessionConfiguration::set_state (XMLNode const& root, int version)
+SessionConfiguration::set_state (XMLNode const& root, int /*version*/)
 {
        if (root.name() != "Ardour") {
                return -1;
index 446d944f8a56f7089cefc649ecc95dde693e8736..10c39983df93fbd4b30c50e1e0a682808b1b850f 100644 (file)
@@ -163,7 +163,7 @@ SessionMetadata::get_state ()
 }
 
 int
-SessionMetadata::set_state (const XMLNode & state, int version)
+SessionMetadata::set_state (const XMLNode & state, int /*version*/)
 {
        const XMLNodeList & children = state.children();
        ustring name;
index fe8a8a1e0aecf02d419aa683cb7035d0ddf4401a..4efd154dcdf82c0702a6ad24461c0894c640311b 100644 (file)
@@ -110,7 +110,7 @@ Source::get_state ()
 }
 
 int
-Source::set_state (const XMLNode& node, int version)
+Source::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
 
index c66e0f6f24fda712e123b9696b7716ff0e156398..b59719867bc7b70981b3d2e1e1fff38f24fd3f0f 100644 (file)
@@ -1500,7 +1500,7 @@ TempoMap::get_state ()
 }
 
 int
-TempoMap::set_state (const XMLNode& node, int version)
+TempoMap::set_state (const XMLNode& node, int /*version*/)
 {
        {
                Glib::RWLock::WriterLock lm (lock);
index 16ecb88c8da5a3e48a103babf39cfbf72cff0ad3..4c39c5f5fff21ca5e73822e9670ae6a252b3248d 100644 (file)
@@ -24,7 +24,7 @@ ARDOUR::UserBundle::UserBundle (XMLNode const & node, bool i)
 }
 
 int
-ARDOUR::UserBundle::set_state (XMLNode const & node, int version)
+ARDOUR::UserBundle::set_state (XMLNode const & node, int /*version*/)
 {
        XMLProperty const * name;
 
index d80e11d81ad63b93d4d0e1f9283ebddc719e1eb7..b9a810ba588a7b03f0edd47a7140cbca909db099 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2008 Hans Baier 
+    Copyright (C) 2008 Hans Baier
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -61,7 +61,7 @@ void initialize_primary_key_from_commands (PatchPrimaryKey& id, const XMLNode* n
                        assert(control != "");
                        string value = node->property("Value")->value();
                        assert(value != "");
-                       
+
                        if (control == "0") {
                                id.msb = PBD::atoi(value);
                        } else if (control == "32") {
@@ -77,13 +77,13 @@ void initialize_primary_key_from_commands (PatchPrimaryKey& id, const XMLNode* n
 
 
 int
-Patch::set_state (const XMLNode& node, int version)
+Patch::set_state (const XMLNode& node, int /*version*/)
 {
        assert(node.name() == "Patch");
        _number = node.property("Number")->value();
        _name   = node.property("Name")->value();
        XMLNode* commands = node.child("PatchMIDICommands");
-       
+
        if (commands) {
                initialize_primary_key_from_commands(_id, commands);
        } else {
@@ -101,7 +101,7 @@ Patch::set_state (const XMLNode& node, int version)
        cerr << "deserialized Patch: name: " <<  _name << " msb: " << _id.msb << " lsb: " << _id.lsb << " program " << _id.program_number << endl;
        // TODO: handle that more gracefully
        assert(_id.is_sane());
-       
+
        return 0;
 }
 
@@ -116,7 +116,7 @@ Note::get_state (void)
 }
 
 int
-Note::set_state (const XMLNode& node, int version)
+Note::set_state (const XMLNode& node, int /*version*/)
 {
        assert(node.name() == "Note");
        _number = node.property("Number")->value();
@@ -147,7 +147,7 @@ NoteNameList::set_state (const XMLNode& node, int version)
                note->set_state(*(*i), version);
                _notes.push_back(note);
        }
-       
+
        return 0;
 }
 
@@ -178,7 +178,7 @@ PatchBank::set_state (const XMLNode& node, int version)
                _id = new PatchPrimaryKey();
                initialize_primary_key_from_commands(*_id, commands);
        }
-       
+
        XMLNode* patch_name_list = node.child("PatchNameList");
        assert(patch_name_list);
        const XMLNodeList patches = patch_name_list->children();
@@ -187,7 +187,7 @@ PatchBank::set_state (const XMLNode& node, int version)
                patch->set_state(*(*i), version);
                _patch_name_list.push_back(patch);
        }
-       
+
        return 0;
 }
 
@@ -245,7 +245,7 @@ ChannelNameSet::set_state (const XMLNode& node, int version)
                                // cerr << "AvailableForChannels after insert" << endl;
                        }
                }
-               
+
                // cerr << "before PatchBank" << endl;
 
                if (node->name() == "PatchBank") {
@@ -263,19 +263,19 @@ ChannelNameSet::set_state (const XMLNode& node, int version)
                        // cerr << "after PatchBank pushback" << endl;
                }
        }
-       
+
        // cerr << "ChannelnameSet done" << endl;
 
        return 0;
 }
 
 int
-CustomDeviceMode::set_state(const XMLNode& a_node, int version)
+CustomDeviceMode::set_state(const XMLNode& a_node, int /*version*/)
 {
        assert(a_node.name() == "CustomDeviceMode");
-       
+
        _name = a_node.property("Name")->value();
-       
+
        boost::shared_ptr<XMLSharedNodeList> channel_name_set_assignments =
                a_node.find("//ChannelNameSetAssign");
        for(XMLSharedNodeList::const_iterator i = channel_name_set_assignments->begin();
@@ -294,15 +294,15 @@ CustomDeviceMode::get_state(void)
 {
        XMLNode* custom_device_mode = new XMLNode("CustomDeviceMode");
        custom_device_mode->add_property("Name",   _name);
-       XMLNode* channel_name_set_assignments = 
+       XMLNode* channel_name_set_assignments =
                custom_device_mode->add_child("ChannelNameSetAssignments");
        for (int i = 0; i < 15 && !_channel_name_set_assignments[i].empty(); i++) {
-               XMLNode* channel_name_set_assign = 
+               XMLNode* channel_name_set_assign =
                        channel_name_set_assignments->add_child("ChannelNameSetAssign");
                channel_name_set_assign->add_property("Channel", i + 1);
                channel_name_set_assign->add_property("NameSet", _channel_name_set_assignments[i]);
        }
-       
+
        return *custom_device_mode;
 }
 
@@ -337,7 +337,7 @@ MasterDeviceNames::set_state(const XMLNode& a_node, int version)
             ++i) {
                boost::shared_ptr<CustomDeviceMode> custom_device_mode(new CustomDeviceMode());
                custom_device_mode->set_state(*(*i), version);
-               
+
                _custom_device_modes[custom_device_mode->name()] = custom_device_mode;
                _custom_device_mode_names.push_back(custom_device_mode->name());
        }
@@ -376,9 +376,9 @@ MasterDeviceNames::get_state(void)
 }
 
 MIDINameDocument::MIDINameDocument (const string& filename)
-       : _document(XMLTree(filename)) 
-{ 
-       set_state(*_document.root(), 0); 
+       : _document(XMLTree(filename))
+{
+       set_state(*_document.root(), 0);
 }
 
 int
@@ -388,7 +388,7 @@ MIDINameDocument::set_state(const XMLNode& a_node, int version)
        boost::shared_ptr<XMLSharedNodeList> author = a_node.find("//Author");
        assert(author->size() == 1);
        _author = author->front()->content();
-       
+
        // cerr << "MIDINameDocument::set_state befor masterdevicenames" << endl;
        // MasterDeviceNames
        boost::shared_ptr<XMLSharedNodeList> master_device_names_list = a_node.find("//MasterDeviceNames");
@@ -399,7 +399,7 @@ MIDINameDocument::set_state(const XMLNode& a_node, int version)
                // cerr << "MIDINameDocument::set_state before masterdevicenames->set_state" << endl;
                master_device_names->set_state(*(*i), version);
                // cerr << "MIDINameDocument::set_state after masterdevicenames->set_state" << endl;
-               
+
                for (MasterDeviceNames::Models::const_iterator model = master_device_names->models().begin();
                     model != master_device_names->models().end();
                     ++model) {
@@ -407,11 +407,11 @@ MIDINameDocument::set_state(const XMLNode& a_node, int version)
                                _master_device_names_list.insert(
                                                std::pair<std::string, boost::shared_ptr<MasterDeviceNames> >
                                                         (*model,      master_device_names));
-                               
+
                                _all_models.push_back(*model);
                }
        }
-       
+
        return 0;
 }
 
index ea6ad364a8a4fef13ae77ba25f36e34671920b2c..a1cd6505400f084845b0b8f2cff46b6ce0a1a1f2 100644 (file)
@@ -105,7 +105,7 @@ Controllable::get_state ()
 }
 
 int
-Controllable::set_state (const XMLNode& node, int version)
+Controllable::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop = node.property (X_("id"));
 
index 8275d0849c465f16e5f716cac354576d2c2f43b2..c16a0bdefd04bf7a195ca7cad20ff4780b90087a 100644 (file)
@@ -1,6 +1,6 @@
 /*
     Copyright (C) 1998-2006 Paul Davis
+
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
     the Free Software Foundation; either version 2 of the License, or
@@ -42,7 +42,7 @@ MIDIControllable::MIDIControllable (Port& p, Controllable& c, bool is_bistate)
        control_additional = (byte) -1;
        connections = 0;
        feedback = true; // for now
-       
+
        /* use channel 0 ("1") as the initial channel */
 
        midi_rebind (0);
@@ -59,18 +59,18 @@ MIDIControllable::midi_forget ()
        /* stop listening for incoming messages, but retain
           our existing event + type information.
        */
-       
+
        if (connections > 0) {
                midi_sense_connection[0].disconnect ();
-       } 
-       
+       }
+
        if (connections > 1) {
                midi_sense_connection[1].disconnect ();
        }
-       
+
        connections = 0;
        midi_learn_connection.disconnect ();
-       
+
 }
 
 void
@@ -101,7 +101,7 @@ MIDIControllable::drop_external_control ()
 {
        if (connections > 0) {
                midi_sense_connection[0].disconnect ();
-       } 
+       }
        if (connections > 1) {
                midi_sense_connection[1].disconnect ();
        }
@@ -147,14 +147,14 @@ MIDIControllable::midi_to_control(float val)
        return val / midi_range * control_range + control_min;
 }
 
-void 
-MIDIControllable::midi_sense_note_on (Parser &p, EventTwoBytes *tb) 
+void
+MIDIControllable::midi_sense_note_on (Parser &p, EventTwoBytes *tb)
 {
        midi_sense_note (p, tb, true);
 }
 
-void 
-MIDIControllable::midi_sense_note_off (Parser &p, EventTwoBytes *tb) 
+void
+MIDIControllable::midi_sense_note_off (Parser &p, EventTwoBytes *tb)
 {
        midi_sense_note (p, tb, false);
 }
@@ -168,7 +168,7 @@ MIDIControllable::midi_sense_note (Parser &, EventTwoBytes *msg, bool is_on)
 
                /* Note: parser handles the use of zero velocity to
                   mean note off. if we get called with is_on=true, then we
-                  got a *real* note on.  
+                  got a *real* note on.
                */
 
                if (msg->note_number == control_additional) {
@@ -209,7 +209,7 @@ MIDIControllable::midi_sense_program_change (Parser &, byte msg)
        if (!bistate) {
                controllable.set_value (msg/127.0);
                last_value = (MIDI::byte) (controllable.get_value() * 127.0); // to prevent feedback fights
-       } 
+       }
 }
 
 void
@@ -221,7 +221,7 @@ MIDIControllable::midi_sense_pitchbend (Parser &, pitchbend_t pb)
 
        controllable.set_value ((pb/(float) SHRT_MAX));
        last_value = (MIDI::byte) (controllable.get_value() * 127.0); // to prevent feedback fights
-}                      
+}
 
 void
 MIDIControllable::midi_receiver (Parser &, byte *msg, size_t /*len*/)
@@ -257,7 +257,7 @@ MIDIControllable::bind_midi (channel_t chn, eventType ev, MIDI::byte additional)
        if (_port.input() == 0) {
                return;
        }
-       
+
        Parser& p = *_port.input();
 
        int chn_i = chn;
@@ -284,7 +284,7 @@ MIDIControllable::bind_midi (channel_t chn, eventType ev, MIDI::byte additional)
                midi_sense_connection[0] = p.channel_note_on[chn_i].connect
                        (mem_fun (*this, &MIDIControllable::midi_sense_note_on));
                if (bistate) {
-                       midi_sense_connection[1] = p.channel_note_off[chn_i].connect 
+                       midi_sense_connection[1] = p.channel_note_off[chn_i].connect
                                (mem_fun (*this, &MIDIControllable::midi_sense_note_off));
                        connections = 2;
                } else {
@@ -294,7 +294,7 @@ MIDIControllable::bind_midi (channel_t chn, eventType ev, MIDI::byte additional)
                break;
 
        case MIDI::controller:
-               midi_sense_connection[0] = p.channel_controller[chn_i].connect 
+               midi_sense_connection[0] = p.channel_controller[chn_i].connect
                        (mem_fun (*this, &MIDIControllable::midi_sense_controller));
                connections = 1;
                snprintf (buf, sizeof (buf), "MIDI control: Controller %d", control_additional);
@@ -304,7 +304,7 @@ MIDIControllable::bind_midi (channel_t chn, eventType ev, MIDI::byte additional)
        case MIDI::program:
                if (!bistate) {
                        midi_sense_connection[0] = p.channel_program_change[chn_i].connect
-                               (mem_fun (*this, 
+                               (mem_fun (*this,
                                       &MIDIControllable::midi_sense_program_change));
                        connections = 1;
                        _control_description = "MIDI control: ProgramChange";
@@ -334,7 +334,7 @@ MIDIControllable::send_feedback ()
                return;
        }
 
-       msg[0] = (control_type & 0xF0) | (control_channel & 0xF); 
+       msg[0] = (control_type & 0xF0) | (control_channel & 0xF);
        msg[1] = control_additional;
        msg[2] = (byte) (control_to_midi(controllable.get_value()));
 
@@ -345,9 +345,9 @@ MIDI::byte*
 MIDIControllable::write_feedback (MIDI::byte* buf, int32_t& bufsize, bool /*force*/)
 {
        if (control_type != none && feedback && bufsize > 2) {
-               
+
                MIDI::byte gm = (MIDI::byte) (control_to_midi(controllable.get_value()));
-               
+
                if (gm != last_value) {
                        *buf++ = (0xF0 & control_type) | (0xF & control_channel);
                        *buf++ = control_additional; /* controller number */
@@ -356,12 +356,12 @@ MIDIControllable::write_feedback (MIDI::byte* buf, int32_t& bufsize, bool /*forc
                        bufsize -= 3;
                }
        }
-       
+
        return buf;
 }
 
-int 
-MIDIControllable::set_state (const XMLNode& node, int version)
+int
+MIDIControllable::set_state (const XMLNode& node, int /*version*/)
 {
        const XMLProperty* prop;
        int xx;
@@ -394,7 +394,7 @@ MIDIControllable::set_state (const XMLNode& node, int version)
        }
 
        bind_midi (control_channel, control_type, control_additional);
-       
+
        return 0;
 }
 
index 5fef10fac49cc5baffd26713a327ef01a54a7e82..cc33585b62b4a5393f4333142da6c844e563f558 100644 (file)
@@ -187,7 +187,7 @@ MackiePort & MackieControlProtocol::port_for_id( uint32_t index )
                current_max += (*it)->strips();
                if ( index < current_max ) return **it;
        }
-       
+
        // oops - no matching port
        ostringstream os;
        os << "No port for index " << index;
@@ -216,15 +216,15 @@ struct RouteByRemoteId
 MackieControlProtocol::Sorted MackieControlProtocol::get_sorted_routes()
 {
        Sorted sorted;
-       
+
        // fetch all routes
        boost::shared_ptr<RouteList> routes = session->get_routes();
        set<uint32_t> remote_ids;
-       
+
        // routes with remote_id 0 should never be added
        // TODO verify this with ardour devs
        // remote_ids.insert( 0 );
-       
+
        // sort in remote_id order, and exclude master, control and hidden routes
        // and any routes that are already set.
        for (RouteList::iterator it = routes->begin(); it != routes->end(); ++it )
@@ -255,7 +255,7 @@ void MackieControlProtocol::switch_banks( int initial )
 {
        // DON'T prevent bank switch if initial == _current_initial_bank
        // because then this method can't be used as a refresh
-       
+
        // sanity checking
        Sorted sorted = get_sorted_routes();
        int delta = sorted.size() - route_table.size();
@@ -267,11 +267,11 @@ void MackieControlProtocol::switch_banks( int initial )
                return;
        }
        _current_initial_bank = initial;
-       
+
        // first clear the signals from old routes
        // taken care of by the RouteSignal destructors
        clear_route_signals();
-       
+
        // now set the signals for new routes
        if ( _current_initial_bank <= sorted.size() )
        {
@@ -282,7 +282,7 @@ void MackieControlProtocol::switch_banks( int initial )
 #ifdef DEBUG
                cout << "switch to " << _current_initial_bank << ", " << end_pos << endl;
 #endif
-               
+
                // link routes to strips
                uint32_t i = 0;
                for ( ; it != end && it != sorted.end(); ++it, ++i )
@@ -298,7 +298,7 @@ void MackieControlProtocol::switch_banks( int initial )
                        // update strip from route
                        rs->notify_all();
                }
-               
+
                // create dead strips if there aren't enough routes to
                // fill a bank
                for ( ; i < route_table.size(); ++i )
@@ -309,7 +309,7 @@ void MackieControlProtocol::switch_banks( int initial )
                        port.write( builder.zero_strip( port, strip ) );
                }
        }
-       
+
        // display the current start bank.
        surface().display_bank_start( mcu_port(), builder, _current_initial_bank );
 }
@@ -317,17 +317,17 @@ void MackieControlProtocol::switch_banks( int initial )
 void MackieControlProtocol::zero_all()
 {
        // TODO turn off SMPTE displays
-       
+
        // zero all strips
        for ( Surface::Strips::iterator it = surface().strips.begin(); it != surface().strips.end(); ++it )
        {
                MackiePort & port = port_for_id( (*it)->index() );
                port.write( builder.zero_strip( port, **it ) );
        }
-       
+
        // and the master strip
        mcu_port().write( builder.zero_strip( dynamic_cast<MackiePort&>( mcu_port() ), master_strip() ) );
-       
+
        // turn off global buttons and leds
        // global buttons are only ever on mcu_port, so we don't have
        // to figure out which port.
@@ -355,29 +355,29 @@ int MackieControlProtocol::set_active( bool yn )
                        if ( yn )
                        {
                                // TODO what happens if this fails half way?
-                               
+
                                // create MackiePorts
                                {
                                        Glib::Mutex::Lock lock( update_mutex );
                                        create_ports();
                                }
-                               
+
                                // make sure the ports are being listened to
                                update_ports();
-                               
+
                                // wait until poll thread is running, with ports to poll
                                // the mutex is only there because conditions require a mutex
                                {
                                        Glib::Mutex::Lock lock( update_mutex );
                                        while ( nfds == 0 ) update_cond.wait( update_mutex );
                                }
-                               
+
                                // now initialise MackiePorts - ie exchange sysex messages
                                for( MackiePorts::iterator it = _ports.begin(); it != _ports.end(); ++it )
                                {
                                        (*it)->open();
                                }
-                               
+
                                // wait until all ports are active
                                // TODO a more sophisticated approach would
                                // allow things to start up with only an MCU, even if
@@ -386,15 +386,15 @@ int MackieControlProtocol::set_active( bool yn )
                                {
                                        (*it)->wait_for_init();
                                }
-                               
+
                                // create surface object. This depends on the ports being
                                // correctly initialised
                                initialize_surface();
                                connect_session_signals();
-                               
+
                                // yeehah!
                                _active = true;
-                               
+
                                // send current control positions to surface
                                // must come after _active = true otherwise it won't run
                                update_surface();
@@ -450,13 +450,13 @@ bool MackieControlProtocol::handle_strip_button( Control & control, ButtonState
                        //state = default_button_press( dynamic_cast<Button&>( control ) );
                }
        }
-       
+
        if ( control.name() == "fader_touch" )
        {
                state = bs == press;
                control.strip().gain().in_use( state );
        }
-       
+
        return state;
 }
 
@@ -541,7 +541,7 @@ void MackieControlProtocol::update_surface()
                // do the initial bank switch to connect signals
                // _current_initial_bank is initialised by set_state
                switch_banks( _current_initial_bank );
-               
+
                // create a RouteSignal for the master route
 
                boost::shared_ptr<Route> mr = master_route ();
@@ -550,10 +550,10 @@ void MackieControlProtocol::update_surface()
                        // update strip from route
                        master_route_signal->notify_all();
                }
-              
+
                // sometimes the jog wheel is a pot
                surface().blank_jog_ring( mcu_port(), builder );
-               
+
                // update global buttons and displays
                notify_record_state_changed();
                notify_transport_state_changed();
@@ -574,7 +574,7 @@ void MackieControlProtocol::connect_session_signals()
        session->config.ParameterChanged.connect ( ( mem_fun (*this, &MackieControlProtocol::notify_parameter_changed) ) );
        // receive rude solo changed
        connections_back = session->SoloActive.connect( ( mem_fun (*this, &MackieControlProtocol::notify_solo_active_changed) ) );
-       
+
        // make sure remote id changed signals reach here
        // see also notify_route_added
        Sorted sorted = get_sorted_routes();
@@ -603,7 +603,7 @@ void MackieControlProtocol::add_port( MIDI::Port & midi_port, int number )
        {
                MackiePort * sport = new MackiePort( *this, midi_port, number );
                _ports.push_back( sport );
-               
+
                connections_back = sport->init_event.connect(
                        sigc::bind (
                                mem_fun (*this, &MackieControlProtocol::handle_port_init)
@@ -624,7 +624,7 @@ void MackieControlProtocol::add_port( MIDI::Port & midi_port, int number )
                                , sport
                        )
                );
-               
+
                _ports_changed = true;
        }
 }
@@ -645,7 +645,7 @@ void MackieControlProtocol::create_ports()
                }
                add_port( *midi_port, 0 );
        }
-       
+
        // open extender ports. Up to 9. Should be enough.
        // could also use mm->get_midi_ports()
        string ext_port_base = "mcu_xt_";
@@ -676,9 +676,9 @@ void MackieControlProtocol::initialize_surface()
        {
                strips += (*it)->strips();
        }
-       
+
        set_route_table_size( strips );
-       
+
        // TODO same as code in mackie_port.cc
        string emulation = ARDOUR::Config->get_mackie_emulation();
        if ( emulation == "bcf" )
@@ -697,7 +697,7 @@ void MackieControlProtocol::initialize_surface()
        }
 
        _surface->init();
-       
+
        // Connect events. Must be after route table otherwise there will be trouble
        for( MackiePorts::iterator it = _ports.begin(); it != _ports.end(); ++it )
        {
@@ -712,10 +712,10 @@ void MackieControlProtocol::close()
        // calls methods on objects that are deleted
        _polling = false;
        pthread_join( thread, 0 );
-       
+
        // TODO disconnect port active/inactive signals
        // Or at least put a lock here
-       
+
        // disconnect global signals from Session
        // TODO Since *this is a sigc::trackable, this shouldn't be necessary
        // but it is for some reason
@@ -725,7 +725,7 @@ void MackieControlProtocol::close()
                it->disconnect();
        }
 #endif
-       
+
        if ( _surface != 0 )
        {
                // These will fail if the port has gone away.
@@ -742,7 +742,7 @@ void MackieControlProtocol::close()
                        cout << "MackieControlProtocol::close caught exception: " << e.what() << endl;
 #endif
                }
-               
+
                for( MackiePorts::iterator it = _ports.begin(); it != _ports.end(); ++it )
                {
                        try
@@ -762,21 +762,21 @@ void MackieControlProtocol::close()
 #endif
                        }
                }
-               
+
                // disconnect routes from strips
                clear_route_signals();
-               
+
                delete _surface;
                _surface = 0;
        }
-       
+
        // shut down MackiePorts
        for( MackiePorts::iterator it = _ports.begin(); it != _ports.end(); ++it )
        {
                delete *it;
        }
        _ports.clear();
-       
+
        // this is done already in monitor_work. But it's here so we know.
        delete[] pfd;
        pfd = 0;
@@ -793,26 +793,26 @@ XMLNode & MackieControlProtocol::get_state()
 #ifdef DEBUG
        cout << "MackieControlProtocol::get_state" << endl;
 #endif
-       
+
        // add name of protocol
        XMLNode* node = new XMLNode( X_("Protocol") );
        node->add_property( X_("name"), _name );
-       
+
        // add current bank
        ostringstream os;
        os << _current_initial_bank;
        node->add_property( X_("bank"), os.str() );
-       
+
        return *node;
 }
 
-int MackieControlProtocol::set_state (const XMLNode & node, int version)
+int MackieControlProtocol::set_state (const XMLNode & node, int /*version*/)
 {
 #ifdef DEBUG
        cout << "MackieControlProtocol::set_state: active " << _active << endl;
 #endif
        int retval = 0;
-       
+
        // fetch current bank
        if ( node.property( X_("bank") ) != 0 )
        {
@@ -831,7 +831,7 @@ int MackieControlProtocol::set_state (const XMLNode & node, int version)
                        return -1;
                }
        }
-       
+
        return retval;
 }
 
@@ -854,7 +854,7 @@ void MackieControlProtocol::handle_control_event( SurfacePort & port, Control &
                                cerr << "Warning: index is " << index << " which is not in the route table, size: " << route_table.size() << endl;
                }
        }
-       
+
        // This handles control element events from the surface
        // the state of the controls on the surface is usually updated
        // from UI events.
@@ -867,14 +867,14 @@ void MackieControlProtocol::handle_control_event( SurfacePort & port, Control &
                        if ( route != 0 )
                        {
                                route->gain_control()->set_value( state.pos );
-                               
+
                                // must echo bytes back to slider now, because
                                // the notifier only works if the fader is not being
                                // touched. Which it is if we're getting input.
                                port.write( builder.build_fader( (Fader&)control, state.pos ) );
                        }
                        break;
-                       
+
                case Control::type_button:
                        if ( control.group().is_strip() )
                        {
@@ -904,7 +904,7 @@ void MackieControlProtocol::handle_control_event( SurfacePort & port, Control &
                                surface().handle_button( *this, state.button_state, dynamic_cast<Button&>( control ) );
                        }
                        break;
-                       
+
                // pot (jog wheel, external control)
                case Control::type_pot:
                        if ( control.group().is_strip() )
@@ -917,14 +917,14 @@ void MackieControlProtocol::handle_control_event( SurfacePort & port, Control &
                                                // assume pan for now
                                                float xpos;
                                                route->panner()->streampanner (0).get_effective_position (xpos);
-                                               
+
                                                // calculate new value, and trim
                                                xpos += state.delta * state.sign;
                                                if ( xpos > 1.0 )
                                                        xpos = 1.0;
                                                else if ( xpos < 0.0 )
                                                        xpos = 0.0;
-                                               
+
                                                route->panner()->streampanner (0).set_position( xpos );
                                        }
                                }
@@ -946,7 +946,7 @@ void MackieControlProtocol::handle_control_event( SurfacePort & port, Control &
                                }
                        }
                        break;
-                       
+
                default:
                        cout << "Control::type not handled: " << control.type() << endl;
        }
@@ -1012,7 +1012,7 @@ void MackieControlProtocol::notify_active_changed (RouteSignal *)
                cout << e.what() << endl;
        }
 }
-       
+
 void MackieControlProtocol::notify_gain_changed( RouteSignal * route_signal, bool force_update )
 {
        try
@@ -1044,7 +1044,7 @@ void MackieControlProtocol::notify_name_changed( RouteSignal * route_signal )
                {
                        string line1;
                        string fullname = route_signal->route()->name();
-                       
+
                        if ( fullname.length() <= 6 )
                        {
                                line1 = fullname;
@@ -1053,7 +1053,7 @@ void MackieControlProtocol::notify_name_changed( RouteSignal * route_signal )
                        {
                                line1 = PBD::short_version( fullname, 6 );
                        }
-                       
+
                        SurfacePort & port = route_signal->port();
                        port.write( builder.strip_display( port, strip, 0, line1 ) );
                        port.write( builder.strip_display_blank( port, strip, 1 ) );
@@ -1075,7 +1075,7 @@ void MackieControlProtocol::notify_panner_changed( RouteSignal * route_signal, b
                {
                        float pos;
                        route_signal->route()->panner()->streampanner(0).get_effective_position( pos );
-                       
+
                        // cache the MidiByteArray here, because the mackie led control is much lower
                        // resolution than the panner control. So we save lots of byte
                        // sends in spite of more work on the comparison
@@ -1106,7 +1106,7 @@ void MackieControlProtocol::update_automation( RouteSignal & rs )
        {
                notify_gain_changed( &rs, false );
        }
-       
+
        if ( rs.route()->panner() ) {
                ARDOUR::AutoState panner_state = rs.route()->panner()->automation_state();
                if ( panner_state == Touch || panner_state == Play )
@@ -1121,14 +1121,14 @@ string MackieControlProtocol::format_bbt_timecode( nframes_t now_frame )
 {
        BBT_Time bbt_time;
        session->bbt_time( now_frame, bbt_time );
-       
+
        // According to the Logic docs
        // digits: 888/88/88/888
        // BBT mode: Bars/Beats/Subdivisions/Ticks
        ostringstream os;
        os << setw(3) << setfill('0') << bbt_time.bars;
        os << setw(2) << setfill('0') << bbt_time.beats;
-       
+
        // figure out subdivisions per beat
        const Meter & meter = session->tempo_map().meter_at( now_frame );
        int subdiv = 2;
@@ -1136,13 +1136,13 @@ string MackieControlProtocol::format_bbt_timecode( nframes_t now_frame )
        {
                subdiv = 3;
        }
-       
+
        uint32_t subdivisions = bbt_time.ticks / uint32_t( Meter::ticks_per_beat / subdiv );
        uint32_t ticks = bbt_time.ticks % uint32_t( Meter::ticks_per_beat / subdiv );
-       
+
        os << setw(2) << setfill('0') << subdivisions + 1;
        os << setw(3) << setfill('0') << ticks;
-       
+
        return os.str();
 }
 
@@ -1159,7 +1159,7 @@ string MackieControlProtocol::format_smpte_timecode( nframes_t now_frame )
        os << setw(2) << setfill('0') << smpte.minutes;
        os << setw(2) << setfill('0') << smpte.seconds;
        os << setw(3) << setfill('0') << smpte.frames;
-       
+
        return os.str();
 }
 
@@ -1170,7 +1170,7 @@ void MackieControlProtocol::update_timecode_display()
                // do assignment here so current_frame is fixed
                nframes_t current_frame = session->transport_frame();
                string timecode;
-               
+
                switch ( _timecode_type )
                {
                        case ARDOUR::AnyTime::BBT:
@@ -1183,8 +1183,8 @@ void MackieControlProtocol::update_timecode_display()
                                ostringstream os;
                                os << "Unknown timecode: " << _timecode_type;
                                throw runtime_error( os.str() );
-               }       
-               
+               }
+
                // only write the timecode string to the MCU if it's changed
                // since last time. This is to reduce midi bandwidth used.
                if ( timecode != _timecode_last )
@@ -1204,15 +1204,15 @@ void MackieControlProtocol::poll_session_data()
                {
                        update_automation( **it );
                }
-               
+
                // and the master strip
                if ( master_route_signal != 0 )
                {
                        update_automation( *master_route_signal );
                }
-               
+
                update_timecode_display();
-               
+
                _automation_last.start();
        }
 }
@@ -1225,12 +1225,12 @@ LedState MackieControlProtocol::frm_left_press (Button &)
 {
        // can use first_mark_before/after as well
        unsigned long elapsed = _frm_left_last.restart();
-       
+
        Location * loc = session->locations()->first_location_before (
                session->transport_frame()
        );
-       
-       // allow a quick double to go past a previous mark 
+
+       // allow a quick double to go past a previous mark
        if ( session->transport_rolling() && elapsed < 500 && loc != 0 )
        {
                Location * loc_two_back = session->locations()->first_location_before ( loc->start() );
@@ -1239,13 +1239,13 @@ LedState MackieControlProtocol::frm_left_press (Button &)
                        loc = loc_two_back;
                }
        }
-       
+
        // move to the location, if it's valid
        if ( loc != 0 )
        {
                session->request_locate( loc->start(), session->transport_rolling() );
        }
-       
+
        return on;
 }
 
@@ -1468,7 +1468,7 @@ void MackieControlProtocol::notify_route_added( ARDOUR::RouteList & rl )
                refresh_current_bank();
        }
        // otherwise route added, but current bank needs no updating
-       
+
        // make sure remote id changes in the new route are handled
        typedef ARDOUR::RouteList ARS;
        for ( ARS::iterator it = rl.begin(); it != rl.end(); ++it )
@@ -1486,7 +1486,7 @@ void MackieControlProtocol::notify_solo_active_changed( bool active )
 void MackieControlProtocol::notify_remote_id_changed()
 {
        Sorted sorted = get_sorted_routes();
-       
+
        // if a remote id has been moved off the end, we need to shift
        // the current bank backwards.
        if ( sorted.size() - _current_initial_bank < route_signals.size() )
@@ -1518,9 +1518,9 @@ void MackieControlProtocol::notify_transport_state_changed()
        update_global_button( "play", session->transport_rolling() );
        update_global_button( "stop", !session->transport_rolling() );
        update_global_button( "loop", session->get_play_loop() );
-       
+
        _transport_previously_rolling = session->transport_rolling();
-       
+
        // rec is special because it's tristate
        Button * rec = reinterpret_cast<Button*>( surface().controls_by_name["record"] );
        mcu_port().write( builder.build_led( *rec, record_release( *rec ) ) );
@@ -1541,7 +1541,7 @@ LedState MackieControlProtocol::left_press (Button &)
                        session->set_dirty();
                        switch_banks( new_initial );
                }
-               
+
                return on;
        }
        else
@@ -1567,7 +1567,7 @@ LedState MackieControlProtocol::right_press (Button &)
                        session->set_dirty();
                        switch_banks( _current_initial_bank + delta );
                }
-               
+
                return on;
        }
        else