Prevent spinlock copy construction
[ardour.git] / gtk2_ardour / generic_pluginui.cc
index 8be90cfa3ff37704d2d874f69f8e8e0e027e8d2e..6e51d605dd0703213954fd516d329ca715373e79 100644 (file)
 #include "gtkmm2ext/doi.h"
 
 #include "widgets/ardour_knob.h"
-#include "widgets/click_box.h"
 #include "widgets/fastmeter.h"
 #include "widgets/slider_controller.h"
 #include "widgets/tooltips.h"
 
 #include "plugin_ui.h"
+#include "plugin_presets_ui.h"
 #include "plugin_display.h"
 #include "gui_thread.h"
 #include "automation_controller.h"
@@ -157,6 +157,11 @@ GenericPluginUI::GenericPluginUI (boost::shared_ptr<PluginInsert> pi, bool scrol
        } else {
                pack_end (plugin_analysis_expander, false, false);
        }
+
+       if (insert->provides_stats ()) {
+               pack_end (cpuload_expander, false, false);
+       }
+
        if (!plugin->get_docs().empty()) {
                pack_end (description_expander, false, false);
        }
@@ -195,7 +200,7 @@ GenericPluginUI::GenericPluginUI (boost::shared_ptr<PluginInsert> pi, bool scrol
 
        main_contents.pack_start (scroller, true, true);
 
-       prefheight = 0;
+       prefheight = -1;
        build ();
 
        if (insert->plugin()->has_midnam() && insert->plugin()->knows_bank_patch()) {
@@ -207,6 +212,7 @@ GenericPluginUI::GenericPluginUI (boost::shared_ptr<PluginInsert> pi, bool scrol
                scroller.set_name ("PluginEditor");
        } else {
                scroller.signal_size_request().connect (sigc::mem_fun(*this, &GenericPluginUI::scroller_size_request));
+               scroller.signal_realize().connect (sigc::mem_fun(scroller, &Widget::queue_resize));
                scroller.set_policy (Gtk::POLICY_AUTOMATIC, Gtk::POLICY_NEVER);
        }
 
@@ -218,28 +224,34 @@ GenericPluginUI::~GenericPluginUI ()
        if (output_controls.size() > 0) {
                screen_update_connection.disconnect();
        }
+       delete automation_menu;
        delete _pianomm;
 }
 
 void
 GenericPluginUI::scroller_size_request (Gtk::Requisition* a)
 {
-       Glib::RefPtr<Gdk::Screen> screen = get_screen();
-       if (!screen)
-               screen = Gdk::Screen::get_default();
-
-       int maximum_width;
-       {
-               Gdk::Rectangle monitor;
-               const int monitor_num = screen->get_monitor_at_window (get_window ());
-               screen->get_monitor_geometry (
-                               (monitor_num < 0) ? 0 : monitor_num,
-                               monitor);
-
-               maximum_width = monitor.get_width() * 0.9;
+       GtkRequisition request = hpacker.size_request();
+
+       Glib::RefPtr<Gdk::Window> window (scroller.get_window());
+       Glib::RefPtr<Gdk::Screen> screen;
+
+       if (window) {
+               screen = window->get_screen();
        }
 
-       GtkRequisition request = hpacker.size_request();
+       if (!screen) {
+               a->width = min(1024, request.width);
+               return;
+       }
+
+       Gdk::Rectangle monitor;
+       const int monitor_num = screen->get_monitor_at_window (window);
+       screen->get_monitor_geometry (
+                       (monitor_num < 0) ? 0 : monitor_num,
+                       monitor);
+
+       const int maximum_width = monitor.get_width() * 0.9;
 
        if (request.width > maximum_width) {
                for (vector<ControlUI*>::const_iterator cuip = input_controls.begin();
@@ -293,8 +305,9 @@ std::size_t s1pos, s2pos, n = 0;
        s1pos = s1.length();
        s2pos = s2.length();
        while (s1pos-- > 0 && s2pos-- > 0) {
-               if (!match_or_digit(s1[s1pos], s2[s2pos])       )
+               if (!match_or_digit(s1[s1pos], s2[s2pos])) {
                        break;
+               }
                n++;
        }
        return n;
@@ -307,7 +320,8 @@ void
 GenericPluginUI::build ()
 {
        std::vector<ControlUI *> control_uis;
-       bool grid = plugin->parameter_count() > 0;
+       bool grid_avail = false;
+       bool grid_veto = false;
 
        // Build a ControlUI for each control port
        for (size_t i = 0; i < plugin->parameter_count(); ++i) {
@@ -330,8 +344,10 @@ GenericPluginUI::build ()
                        ControlUI* cui;
                        Plugin::UILayoutHint hint;
 
-                       if (!plugin->get_layout(i, hint)) {
-                               grid = false;
+                       if (plugin->get_layout(i, hint)) {
+                               grid_avail = true;
+                       } else {
+                               grid_veto = true;
                        }
 
                        boost::shared_ptr<ARDOUR::AutomationControl> c
@@ -345,7 +361,7 @@ GenericPluginUI::build ()
                                continue;
                        }
 
-                       if (grid) {
+                       if (grid_avail && !grid_veto) {
                                cui->x0 = hint.x0;
                                cui->x1 = hint.x1;
                                cui->y0 = hint.y0;
@@ -361,6 +377,8 @@ GenericPluginUI::build ()
                }
        }
 
+       bool grid = grid_avail && !grid_veto;
+
        // Build a ControlUI for each property
        const Plugin::PropertyDescriptors& descs = plugin->get_supported_properties();
        for (Plugin::PropertyDescriptors::const_iterator d = descs.begin(); d != descs.end(); ++d) {
@@ -397,7 +415,20 @@ GenericPluginUI::build ()
                plugin->announce_property_values();
        }
 
-       if (grid) {
+       if (control_uis.empty ()) {
+               std::vector<Plugin::PresetRecord> presets = insert->plugin()->get_presets();
+               bool show_preset_browser = false;
+               for (std::vector<Plugin::PresetRecord>::const_iterator i = presets.begin(); i != presets.end(); ++i) {
+                       if (i->valid && !i->description.empty()) {
+                               show_preset_browser = true;
+                               break;
+                       }
+               }
+               if (show_preset_browser) {
+                       preset_gui = new PluginPresetsUI (insert);
+                       hpacker.pack_start (*preset_gui, true, true);
+               }
+       } else if (grid) {
                custom_layout (control_uis);
        } else {
                automatic_layout (control_uis);
@@ -495,7 +526,7 @@ GenericPluginUI::automatic_layout (const std::vector<ControlUI*>& control_uis)
                                             FILL|EXPAND, FILL);
                        button_row++;
 
-               } else if (cui->controller || cui->clickbox || cui->combo) {
+               } else if (cui->controller || cui->combo) {
                        // Get all of the controls into a list, so that
                        // we can lay them out a bit more nicely later.
                        cui_controls_list.push_back(cui);
@@ -595,7 +626,7 @@ GenericPluginUI::automatic_layout (const std::vector<ControlUI*>& control_uis)
                box->pack_start (*cui, false, false);
        }
 
-       if (is_scrollable) {
+       if (is_scrollable && i > 0) {
                prefheight = 30 * i;
        }
 
@@ -663,9 +694,9 @@ GenericPluginUI::build_midi_table ()
        Frame* frame = manage (new Frame);
        frame->set_name ("BaseFrame");
        if (dynamic_cast<MidiTrack*> (insert->owner())) {
-               frame->set_label (_("MIDI Progams (sent to track)"));
+               frame->set_label (_("MIDI Programs (sent to track)"));
        } else {
-               frame->set_label (_("MIDI Progams (volatile)"));
+               frame->set_label (_("MIDI Programs (volatile)"));
        }
        frame->add (*pgm_table);
        hpacker.pack_start (*frame, false, false);
@@ -725,7 +756,6 @@ GenericPluginUI::midi_refill_patches ()
                                for (MIDI::Name::PatchNameList::const_iterator j = patches.begin(); j != patches.end(); ++j) {
                                        const std::string pgm = (*j)->name ();
                                        MIDI::Name::PatchPrimaryKey const& key = (*j)->patch_primary_key ();
-                                       assert ((*i)->number () == key.bank());
                                        const uint32_t bp = (key.bank() << 7) | key.program();
                                        midi_pgmsel[chn]->AddMenuElem (MenuElemNoMnemonic (pgm, sigc::bind (sigc::mem_fun (*this, &GenericPluginUI::midi_bank_patch_select), chn, bp)));
                                        pgm_names[bp] = pgm;
@@ -792,7 +822,6 @@ GenericPluginUI::ControlUI::ControlUI (const Evoral::Parameter& p)
        : param(p)
        , automate_button (X_("")) // force creation of a label
        , combo (0)
-       , clickbox (0)
        , file_button (0)
        , spin_box (0)
        , display (0)
@@ -874,32 +903,6 @@ GenericPluginUI::automation_state_changed (ControlUI* cui)
        }
 }
 
-bool
-GenericPluginUI::integer_printer (char buf[32], Adjustment &adj, ControlUI* cui)
-{
-       float const        v   = cui->control->interface_to_internal(adj.get_value ());
-       const std::string& str = ARDOUR::value_as_string(cui->control->desc(), Variant(v));
-       const size_t       len = str.copy(buf, 31);
-       buf[len] = '\0';
-       return true;
-}
-
-bool
-GenericPluginUI::midinote_printer (char buf[32], Adjustment &adj, ControlUI* cui)
-{
-       float const        v   = cui->control->interface_to_internal(adj.get_value ());
-       const std::string& str = ARDOUR::value_as_string(cui->control->desc(), Variant(v));
-       const size_t       len = str.copy(buf, 31);
-       buf[len] = '\0';
-       return true;
-}
-
-void
-GenericPluginUI::print_parameter (char *buf, uint32_t len, uint32_t param)
-{
-       plugin->print_parameter (param, buf, len);
-}
-
 /** Build a ControlUI for a parameter/property.
  * Note that mcontrol may be NULL for outputs.
  */
@@ -934,6 +937,7 @@ GenericPluginUI::build_control_ui (const Evoral::Parameter&             param,
                        // Create/add controller
                        control_ui->file_button = manage(new Gtk::FileChooserButton(Gtk::FILE_CHOOSER_ACTION_OPEN));
                        control_ui->file_button->set_title(desc.label);
+                       Gtkmm2ext::add_volume_shortcuts (*control_ui->file_button);
 
                        if (use_knob) {
                                control_ui->knobtable = manage (new Table());
@@ -1009,16 +1013,7 @@ GenericPluginUI::build_control_ui (const Evoral::Parameter&             param,
 
                        Adjustment* adj = control_ui->controller->adjustment();
 
-                       if (desc.integer_step && !desc.toggled) {
-                               control_ui->clickbox = new ArdourWidgets::ClickBox (adj, "PluginUIClickBox", true);
-                               Gtkmm2ext::set_size_request_to_display_given_text (*control_ui->clickbox, "g9999999", 2, 2);
-                               if (desc.unit == ParameterDescriptor::MIDI_NOTE) {
-                                       control_ui->clickbox->set_printer (sigc::bind (sigc::mem_fun (*this, &GenericPluginUI::midinote_printer), control_ui));
-                               } else {
-                                       control_ui->clickbox->set_printer (sigc::bind (sigc::mem_fun (*this, &GenericPluginUI::integer_printer), control_ui));
-                               }
-                               control_ui->clickbox->set_controllable (mcontrol);
-                       } else if (desc.toggled) {
+                       if (desc.toggled) {
                                ArdourButton* but = dynamic_cast<ArdourButton*> (control_ui->controller->widget());
                                assert(but);
                                but->set_tweaks(ArdourButton::Square);
@@ -1029,6 +1024,11 @@ GenericPluginUI::build_control_ui (const Evoral::Parameter&             param,
                        } else {
                                control_ui->controller->set_size_request (200, -1);
                                control_ui->controller->set_name (X_("ProcessorControlSlider"));
+                               if (desc.integer_step) {
+                                       AutomationBarController* abc = dynamic_cast <AutomationBarController*> (control_ui->controller->widget ());
+                                       assert (abc);
+                                       abc->set_digits (0);
+                               }
                        }
 
                        if (!desc.integer_step && !desc.toggled && use_knob) {
@@ -1049,10 +1049,6 @@ GenericPluginUI::build_control_ui (const Evoral::Parameter&             param,
                        if (control_ui->combo) {
                                control_ui->knobtable->attach (control_ui->label, 0, 1, 0, 1);
                                control_ui->knobtable->attach (*control_ui->combo, 0, 1, 1, 2);
-                       } else if (control_ui->clickbox) {
-                               control_ui->knobtable->attach (*control_ui->clickbox, 0, 2, 0, 1);
-                               control_ui->knobtable->attach (control_ui->label, 0, 1, 1, 2, FILL, SHRINK);
-                               control_ui->knobtable->attach (control_ui->automate_button, 1, 2, 1, 2, SHRINK, SHRINK, 2, 0);
                        } else if (control_ui->spin_box) {
                                ArdourKnob* knob = dynamic_cast<ArdourKnob*>(control_ui->controller->widget ());
                                knob->set_tooltip_prefix (desc.label + ": ");
@@ -1078,8 +1074,6 @@ GenericPluginUI::build_control_ui (const Evoral::Parameter&             param,
                        control_ui->pack_start (control_ui->label, true, true);
                        if (control_ui->combo) {
                                control_ui->pack_start(*control_ui->combo, false, true);
-                       } else if (control_ui->clickbox) {
-                               control_ui->pack_start (*control_ui->clickbox, false, false);
                        } else if (control_ui->spin_box) {
                                control_ui->pack_start (*control_ui->spin_box, false, false);
                                control_ui->pack_start (*control_ui->controller, false, false);