Fixed bug where embedded sources did not correctly obey the BWF time stamp and would...
[ardour.git] / gtk2_ardour / imageframe_time_axis.cc
index 018fae68959e169c849bd65171cd050d64cfaa09..b680dbda3a9f37d1de5bb76e3fc4c3df138173df 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2003 Paul Davis 
+    Copyright (C) 2003 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
 #include <string>
 #include <algorithm>
 
-#include <pbd/error.h>
+#include "pbd/error.h"
 
 #include <gtkmm/menu.h>
 
 #include <gtkmm2ext/utils.h>
 #include <gtkmm2ext/gtk_ui.h>
 
-#include <ardour/session.h>
-#include <ardour/utils.h>
+#include "ardour/session.h"
+#include "ardour/utils.h"
 
 #include "public_editor.h"
 #include "imageframe_time_axis.h"
@@ -47,7 +47,6 @@
 
 using namespace ARDOUR;
 using namespace PBD;
-using namespace sigc;
 using namespace Gtk;
 
 /**
@@ -58,12 +57,12 @@ using namespace Gtk;
  * @param sess the current session
  * @param canvas the parent canvas item
  */
-ImageFrameTimeAxis::ImageFrameTimeAxis(const string & track_id, PublicEditor& ed, ARDOUR::Session& sess, ArdourCanvas::Canvas& canvas)
+ImageFrameTimeAxis::ImageFrameTimeAxis(const string & track_id, PublicEditor& ed, ARDOUR::Session* sess, ArdourCanvas::Canvas& canvas)
        : AxisView(sess),
          VisualTimeAxis(track_id, ed, sess, canvas)
 {
        _color = unique_random_color() ;
-       
+
        selection_group = new ArdourCanvas::Group (*canvas_display);
        selection_group->hide();
 
@@ -73,15 +72,17 @@ ImageFrameTimeAxis::ImageFrameTimeAxis(const string & track_id, PublicEditor& ed
 
        /* create our new image frame view */
        view = new ImageFrameTimeAxisView(*this) ;
-       
+
        /* create the Image Frame Edit Menu */
        create_imageframe_menu() ;
-       
+
        // set the initial time axis text label
        label_view() ;
-               
+
        // set the initial height of this time axis
-       set_height(Normal) ;
+       set_height(hNormal) ;
+
+       TimeAxisView::CatchDeletion.connect (*this, ui_bind (&ImageFrameTimeAxis::remove_time_axis_view, this, _1), gui_context());
 }
 
 /**
@@ -90,41 +91,32 @@ ImageFrameTimeAxis::ImageFrameTimeAxis(const string & track_id, PublicEditor& ed
  */
 ImageFrameTimeAxis::~ImageFrameTimeAxis ()
 {
-        GoingAway() ; /* EMIT_SIGNAL */
-       
+       CatchDeletion (this);
+
        // Destroy all the marker views we may have associaited with this TimeAxis
        for(MarkerTimeAxisList::iterator iter = marker_time_axis_list.begin(); iter != marker_time_axis_list.end(); ++iter)
        {
                MarkerTimeAxis* mta = *iter ;
                MarkerTimeAxisList::iterator next = iter ;
                next++ ;
-               
+
                marker_time_axis_list.erase(iter) ;
 
                delete mta ;
                mta = 0 ;
-               
+
                iter = next ;
        }
-       
-       if(image_action_menu)
-       {
-               delete image_action_menu ;
-               image_action_menu = 0 ;
-       }
-       
-       if (selection_group)
-       {
-               delete selection_group;
-               selection_group = 0 ;
-       }
-       
+
+       delete image_action_menu ;
+       image_action_menu = 0 ;
+
+       delete selection_group;
+       selection_group = 0 ;
+
        // Destroy our Axis View helper
-       if(view)
-       {
-               delete view ;
-               view = 0 ;
-       }
+       delete view ;
+       view = 0 ;
 }
 
 //---------------------------------------------------------------------------------------//
@@ -133,19 +125,19 @@ ImageFrameTimeAxis::~ImageFrameTimeAxis ()
 /**
  * Sets the height of this TrackView to one of ths TrackHeghts
  *
- * @param h the TrackHeight value to set
+ * @param h
  */
 void
-ImageFrameTimeAxis::set_height (TrackHeight h)
+ImageFrameTimeAxis::set_height (uint32_t h)
 {
        VisualTimeAxis::set_height(h) ;
-       
+
        // tell out view helper of the change too
        if(view != 0)
        {
                view->set_height((double) height) ;
        }
-       
+
        // tell those interested that we have had our height changed
         gui_changed("track_height",(void*)0); /* EMIT_SIGNAL */
 }
@@ -203,7 +195,7 @@ ImageFrameTimeAxis::popup_imageframe_edit_menu(int button, int32_t time, ImageFr
        {
                imageframe_item_menu->set_sensitive(false) ;
        }
-       
+
        imageframe_menu->popup(button,time) ;
 }
 
@@ -244,18 +236,18 @@ ImageFrameTimeAxis::build_display_menu()
 
        MenuList& items = display_menu->items();
 
-       items.push_back (MenuElem (_("Rename"), mem_fun(*this, &ImageFrameTimeAxis::start_time_axis_rename)));
+       items.push_back (MenuElem (_("Rename"), sigc::mem_fun(*this, &ImageFrameTimeAxis::start_time_axis_rename)));
 
        image_action_menu = new Menu() ;
        image_action_menu->set_name ("ArdourContextMenu");
        MenuList image_items = image_action_menu->items() ;
-       
+
        items.push_back (SeparatorElem());
        items.push_back (MenuElem (_("Height"), *size_menu));
-       items.push_back (MenuElem (_("Color"), mem_fun(*this, &ImageFrameTimeAxis::select_track_color)));
+       items.push_back (MenuElem (_("Color"), sigc::mem_fun(*this, &ImageFrameTimeAxis::select_track_color)));
 
        items.push_back (SeparatorElem());
-       items.push_back (MenuElem (_("Remove"), bind(mem_fun(*this, &VisualTimeAxis::remove_this_time_axis), (void*)this))) ;
+       items.push_back (MenuElem (_("Remove"), sigc::bind(sigc::mem_fun(*this, &VisualTimeAxis::remove_this_time_axis), (void*)this))) ;
 }
 
 /**
@@ -270,7 +262,7 @@ ImageFrameTimeAxis::create_imageframe_menu()
        imageframe_menu = manage(new Menu) ;
        imageframe_menu->set_name ("ArdourContextMenu");
        MenuList& items = imageframe_menu->items();
-       
+
        imageframe_item_menu = manage(new Menu) ;
        imageframe_item_menu->set_name ("ArdourContextMenu");
        MenuList& imageframe_sub_items = imageframe_item_menu->items() ;
@@ -282,14 +274,14 @@ ImageFrameTimeAxis::create_imageframe_menu()
 
        if(view)
        {
-               duration_items.push_back(MenuElem (_("0.5 seconds"), bind (mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 0.5))) ;
-               duration_items.push_back(MenuElem (_("1 seconds"), bind (mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 1.0))) ;
-               duration_items.push_back(MenuElem (_("1.5 seconds"), bind (mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 1.5))) ;
-               duration_items.push_back(MenuElem (_("2 seconds"), bind (mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 2.0))) ;
-               duration_items.push_back(MenuElem (_("2.5 seconds"), bind (mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 2.5))) ;
-               duration_items.push_back(MenuElem (_("3 seconds"), bind (mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 3.0))) ;
+               duration_items.push_back(MenuElem (_("0.5 seconds"), sigc::bind (sigc::mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 0.5))) ;
+               duration_items.push_back(MenuElem (_("1 seconds"), sigc::bind (sigc::mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 1.0))) ;
+               duration_items.push_back(MenuElem (_("1.5 seconds"), sigc::bind (sigc::mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 1.5))) ;
+               duration_items.push_back(MenuElem (_("2 seconds"), sigc::bind (sigc::mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 2.0))) ;
+               duration_items.push_back(MenuElem (_("2.5 seconds"), sigc::bind (sigc::mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 2.5))) ;
+               duration_items.push_back(MenuElem (_("3 seconds"), sigc::bind (sigc::mem_fun (view, &ImageFrameTimeAxisView::set_imageframe_duration_sec), 3.0))) ;
                //duration_items.push_back(SeparatorElem()) ;
-               //duration_items.push_back(MenuElem (_("custom"), mem_fun(*this, &ImageFrameTimeAxis::set_imageframe_duration_custom))) ;
+               //duration_items.push_back(MenuElem (_("custom"), sigc::mem_fun(*this, &ImageFrameTimeAxis::set_imageframe_duration_custom))) ;
        }
 
        imageframe_sub_items.push_back(MenuElem(_("Duration (sec)"), *duration_menu)) ;
@@ -297,11 +289,11 @@ ImageFrameTimeAxis::create_imageframe_menu()
        imageframe_sub_items.push_back(SeparatorElem()) ;
        if(view)
        {
-               imageframe_sub_items.push_back(MenuElem (_("Remove Frame"), bind(mem_fun (view, &ImageFrameTimeAxisView::remove_selected_imageframe_item), (void*)this))) ;
+               imageframe_sub_items.push_back(MenuElem (_("Remove Frame"), sigc::bind(sigc::mem_fun (view, &ImageFrameTimeAxisView::remove_selected_imageframe_item), (void*)this))) ;
        }
-       
+
        items.push_back(MenuElem(_("Image Frame"), *imageframe_item_menu)) ;
-       items.push_back(MenuElem (_("Rename Track"), mem_fun(*this,&ImageFrameTimeAxis::start_time_axis_rename))) ;
+       items.push_back(MenuElem (_("Rename Track"), sigc::mem_fun(*this,&ImageFrameTimeAxis::start_time_axis_rename))) ;
 
        imageframe_menu->show_all() ;
 }
@@ -324,7 +316,7 @@ bool
 ImageFrameTimeAxis::add_marker_time_axis(MarkerTimeAxis* marker_track, void* src)
 {
        bool ret = false ;
-       
+
        if(get_named_marker_time_axis(marker_track->name()) != 0)
        {
                ret = false ;
@@ -332,12 +324,10 @@ ImageFrameTimeAxis::add_marker_time_axis(MarkerTimeAxis* marker_track, void* src
        else
        {
                marker_time_axis_list.push_back(marker_track) ;
-               marker_track->GoingAway.connect(bind(mem_fun(*this, &ImageFrameTimeAxis::remove_time_axis_view), marker_track, (void*)this));
-       
                 MarkerTimeAxisAdded(marker_track, src) ; /* EMIT_SIGNAL */
                ret = true ;
        }
-       
+
        return(ret) ;
 }
 
@@ -351,7 +341,7 @@ MarkerTimeAxis*
 ImageFrameTimeAxis::get_named_marker_time_axis(const string & track_id)
 {
        MarkerTimeAxis* mta =  0 ;
-       
+
        for (MarkerTimeAxisList::iterator i = marker_time_axis_list.begin(); i != marker_time_axis_list.end(); ++i)
        {
                if (((MarkerTimeAxis*)*i)->name() == track_id)
@@ -374,21 +364,21 @@ MarkerTimeAxis*
 ImageFrameTimeAxis::remove_named_marker_time_axis(const string & track_id, void* src)
 {
        MarkerTimeAxis* mta = 0 ;
-       
+
        for(MarkerTimeAxisList::iterator i = marker_time_axis_list.begin(); i != marker_time_axis_list.end(); ++i)
        {
                if (((MarkerTimeAxis*)*i)->name() == track_id)
                {
                        mta = ((MarkerTimeAxis*)*i) ;
-                       
+
                        // the iterator is invalid after this call, so we can no longer use it as is.
                        marker_time_axis_list.erase(i) ;
-                       
+
                         MarkerTimeAxisRemoved(mta->name(), src) ; /* EMIT_SIGNAL */
                        break ;
                }
        }
-       
+
        return(mta) ;
 }
 
@@ -400,17 +390,20 @@ ImageFrameTimeAxis::remove_named_marker_time_axis(const string & track_id, void*
  * @param src the identity of the object that initiated the change
  */
 void
-ImageFrameTimeAxis::remove_time_axis_view(MarkerTimeAxis* mta, void* src)
+ImageFrameTimeAxis::remove_time_axis_view (TimeAxisView* tav)
 {
-       ENSURE_GUI_THREAD(bind (mem_fun(*this, &ImageFrameTimeAxis::remove_time_axis_view), mta, src));
-       
+       MarkerTimeAxisView* mtav = dynamic_cast<MarkerTimeAxisView*> (tav);
+
+       if (!mtav) {
+               return;
+       }
+
        MarkerTimeAxisList::iterator i;
-       if((i = find (marker_time_axis_list.begin(), marker_time_axis_list.end(), mta)) != marker_time_axis_list.end())
-       {
+
+       if ((i = find (marker_time_axis_list.begin(), marker_time_axis_list.end(), mta)) != marker_time_axis_list.end())  {
                // note that we dont delete the object itself, we just remove it from our list
                marker_time_axis_list.erase(i) ;
-
-                MarkerTimeAxisRemoved(mta->name(), src) ; /* EMIT_SIGNAL */
+               MarkerTimeAxisRemoved (mta->name(), src) ; /* EMIT_SIGNAL */
        }
 }