diff --git a/gtk2_ardour/midi_time_axis.cc b/gtk2_ardour/midi_time_axis.cc index 19d2c892d5..7c7ef7b350 100644 --- a/gtk2_ardour/midi_time_axis.cc +++ b/gtk2_ardour/midi_time_axis.cc @@ -263,8 +263,7 @@ MidiTimeAxisView::set_route (boost::shared_ptr rt) if (gui_property (X_("midnam-custom-device-mode")).empty()) { boost::shared_ptr device_names = get_device_names(); if (device_names) { - set_gui_property (X_("midnam-custom-device-mode"), - *device_names->custom_device_mode_names().begin()); + set_gui_property (X_("midnam-custom-device-mode"), *device_names->custom_device_mode_names().begin()); } } @@ -486,12 +485,8 @@ MidiTimeAxisView::model_changed(const std::string& model) } _midnam_custom_device_mode_selector.clear_items(); - for (std::list::const_iterator i = device_modes.begin(); - i != device_modes.end(); ++i) { - _midnam_custom_device_mode_selector.AddMenuElem( - Gtk::Menu_Helpers::MenuElem( - *i, sigc::bind(sigc::mem_fun(*this, &MidiTimeAxisView::custom_device_mode_changed), - *i))); + for (std::list::const_iterator i = device_modes.begin(); i != device_modes.end(); ++i) { + _midnam_custom_device_mode_selector.AddMenuElem (Gtk::Menu_Helpers::MenuElem(*i, sigc::bind(sigc::mem_fun(*this, &MidiTimeAxisView::custom_device_mode_changed), *i))); } if (!device_modes.empty()) { @@ -920,12 +915,12 @@ MidiTimeAxisView::get_device_mode() using namespace MIDI::Name; boost::shared_ptr device_names = get_device_names(); + if (!device_names) { return boost::shared_ptr(); } - return device_names->custom_device_mode_by_name( - gui_property (X_("midnam-custom-device-mode"))); + return device_names->custom_device_mode_by_name(gui_property (X_("midnam-custom-device-mode"))); } boost::shared_ptr