enough with umpteen "i18n.h" files. Consolidate on pbd/i18n.h

[ardour.git] / gtk2_ardour / region_editor.cc

diff --git a/gtk2_ardour/region_editor.cc b/gtk2_ardour/region_editor.cc
index 51e4deddc88a88de66d6eb8c792ed9109da23fa5..596f6009aad5a6ded0263eec68e772aae707e583 100644 (file)
--- a/gtk2_ardour/region_editor.cc
+++ b/gtk2_ardour/region_editor.cc
@@ -33,14 +33,16 @@
 #include "main_clock.h"
 #include "gui_thread.h"
 #include "region_editor.h"
-#include "utils.h"
+#include "public_editor.h"
+#include "tooltips.h"
 
-#include "i18n.h"
+#include "pbd/i18n.h"
 
 using namespace ARDOUR;
 using namespace PBD;
 using namespace std;
 using namespace Gtkmm2ext;
+using namespace ARDOUR_UI_UTILS;
 
 RegionEditor::RegionEditor (Session* s, boost::shared_ptr<Region> r)
        : ArdourDialog (_("Region"))
@@ -76,7 +78,7 @@ RegionEditor::RegionEditor (Session* s, boost::shared_ptr<Region> r)
        sync_offset_absolute_clock.set_session (_session);
        start_clock.set_session (_session);
 
-       ARDOUR_UI::instance()->set_tip (audition_button, _("audition this region"));
+       set_tooltip (audition_button, _("audition this region"));
 
        audition_button.unset_flags (Gtk::CAN_FOCUS);
 
@@ -161,7 +163,6 @@ RegionEditor::RegionEditor (Session* s, boost::shared_ptr<Region> r)
        set_name ("RegionEditorWindow");
        add_events (Gdk::KEY_PRESS_MASK|Gdk::KEY_RELEASE_MASK);
 
-       signal_delete_event().connect (sigc::bind (sigc::ptr_fun (just_hide_it), static_cast<Window *> (this)));
        signal_response().connect (sigc::mem_fun (*this, &RegionEditor::handle_response));
 
        set_title (string_compose (_("Region '%1'"), _region->name()));
@@ -271,33 +272,41 @@ RegionEditor::connect_editor_events ()
 void
 RegionEditor::position_clock_changed ()
 {
-       _session->begin_reversible_command (_("change region start position"));
-
+       bool in_command = false;
        boost::shared_ptr<Playlist> pl = _region->playlist();
 
        if (pl) {
+               PublicEditor::instance().begin_reversible_command (_("change region start position"));
+               in_command = true;
+
                _region->clear_changes ();
                _region->set_position (position_clock.current_time());
                _session->add_command(new StatefulDiffCommand (_region));
        }
 
-       _session->commit_reversible_command ();
+       if (in_command) {
+               PublicEditor::instance().commit_reversible_command ();
+       }
 }
 
 void
 RegionEditor::end_clock_changed ()
 {
-       _session->begin_reversible_command (_("change region end position"));
-
+       bool in_command = false;
        boost::shared_ptr<Playlist> pl = _region->playlist();
 
        if (pl) {
+               PublicEditor::instance().begin_reversible_command (_("change region end position"));
+               in_command = true;
+
                 _region->clear_changes ();
                _region->trim_end (end_clock.current_time());
                _session->add_command(new StatefulDiffCommand (_region));
        }
 
-       _session->commit_reversible_command ();
+       if (in_command) {
+               PublicEditor::instance().commit_reversible_command ();
+       }
 
        end_clock.set (_region->position() + _region->length() - 1, true);
 }
@@ -306,18 +315,21 @@ void
 RegionEditor::length_clock_changed ()
 {
        framecnt_t frames = length_clock.current_time();
-
-       _session->begin_reversible_command (_("change region length"));
-
+       bool in_command = false;
        boost::shared_ptr<Playlist> pl = _region->playlist();
 
        if (pl) {
-                _region->clear_changes ();
+               PublicEditor::instance().begin_reversible_command (_("change region length"));
+               in_command = true;
+
+               _region->clear_changes ();
                _region->trim_end (_region->position() + frames - 1);
                _session->add_command(new StatefulDiffCommand (_region));
        }
 
-       _session->commit_reversible_command ();
+       if (in_command) {
+               PublicEditor::instance().commit_reversible_command ();
+       }
 
        length_clock.set (_region->length());
 }
@@ -401,25 +413,25 @@ RegionEditor::audition_state_changed (bool yn)
 void
 RegionEditor::sync_offset_absolute_clock_changed ()
 {
-       _session->begin_reversible_command (_("change region sync point"));
+       PublicEditor::instance().begin_reversible_command (_("change region sync point"));
 
         _region->clear_changes ();
        _region->set_sync_position (sync_offset_absolute_clock.current_time());
        _session->add_command (new StatefulDiffCommand (_region));
 
-       _session->commit_reversible_command ();
+       PublicEditor::instance().commit_reversible_command ();
 }
 
 void
 RegionEditor::sync_offset_relative_clock_changed ()
 {
-       _session->begin_reversible_command (_("change region sync point"));
+       PublicEditor::instance().begin_reversible_command (_("change region sync point"));
 
         _region->clear_changes ();
        _region->set_sync_position (sync_offset_relative_clock.current_time() + _region->position ());
        _session->add_command (new StatefulDiffCommand (_region));
 
-       _session->commit_reversible_command ();
+       PublicEditor::instance().commit_reversible_command ();
 }
 
 bool