(merge from 2.0-ongoing -r1911:1912) fix audio clock handling of key press; fix crash...
[ardour.git] / gtk2_ardour / editor_region_list.cc
index bbcfb09a2902f2121aada484b5a531827c46d78b..8a50332193400448173cfba7c0b5392dae996abe 100644 (file)
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <cstdlib>
 #include <cmath>
 #include <algorithm>
 #include <string>
+#include <sstream>
 
 #include <pbd/basename.h>
 
 #include <ardour/audioregion.h>
+#include <ardour/audiofilesource.h>
+#include <ardour/silentfilesource.h>
 #include <ardour/session_region.h>
 
 #include <gtkmm2ext/stop_signal.h>
 
 #include "editor.h"
 #include "editing.h"
+#include "keyboard.h"
 #include "ardour_ui.h"
 #include "gui_thread.h"
 #include "actions.h"
 
 using namespace sigc;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace Gtk;
 using namespace Glib;
 using namespace Editing;
 
 void
-Editor::handle_audio_region_removed (AudioRegion* region)
+Editor::handle_region_removed (boost::weak_ptr<Region> wregion)
 {
-       ENSURE_GUI_THREAD (bind (mem_fun (*this, &Editor::handle_audio_region_removed), region));
+       ENSURE_GUI_THREAD (mem_fun (*this, &Editor::redisplay_regions));
        redisplay_regions ();
 }
 
 void
-Editor::handle_new_audio_region (AudioRegion *region)
+Editor::handle_new_region (boost::weak_ptr<Region> wregion)
 {
-       ENSURE_GUI_THREAD (bind (mem_fun (*this, &Editor::handle_new_audio_region), region));
+       ENSURE_GUI_THREAD (bind (mem_fun (*this, &Editor::handle_new_region), wregion));
 
        /* don't copy region - the one we are being notified
           about belongs to the session, and so it will
           never be edited.
        */
-       add_audio_region_to_region_display (region);
+
+       boost::shared_ptr<Region> region (wregion.lock());
+       
+       if (region) {
+               add_region_to_region_display (region);
+       }
 }
 
 void
-Editor::region_hidden (Region* r)
+Editor::region_hidden (boost::shared_ptr<Region> r)
 {
        ENSURE_GUI_THREAD(bind (mem_fun(*this, &Editor::region_hidden), r));    
 
@@ -73,11 +82,14 @@ Editor::region_hidden (Region* r)
 }
 
 void
-Editor::add_audio_region_to_region_display (AudioRegion *region)
+Editor::add_region_to_region_display (boost::shared_ptr<Region> region)
 {
        string str;
        TreeModel::Row row;
        Gdk::Color c;
+       bool missing_source;
+
+       missing_source = boost::dynamic_pointer_cast<SilentFileSource>(region->source());
 
        if (!show_automatic_regions_in_region_list && region->automatic()) {
                return;
@@ -94,7 +106,8 @@ Editor::add_audio_region_to_region_display (AudioRegion *region)
                        parent = *(region_list_model->append());
                        
                        parent[region_list_columns.name] = _("Hidden");
-                       parent[region_list_columns.region] = 0;
+                       boost::shared_ptr<Region> proxy = parent[region_list_columns.region];
+                       proxy.reset ();
 
                } else {
 
@@ -102,7 +115,8 @@ Editor::add_audio_region_to_region_display (AudioRegion *region)
 
                                parent = *(region_list_model->insert(iter));
                                parent[region_list_columns.name] = _("Hidden");
-                               parent[region_list_columns.region] = 0;
+                               boost::shared_ptr<Region> proxy = parent[region_list_columns.region];
+                               proxy.reset ();
 
                        } else {
                                parent = *iter;
@@ -115,15 +129,27 @@ Editor::add_audio_region_to_region_display (AudioRegion *region)
        } else if (region->whole_file()) {
 
                row = *(region_list_model->append());
-               set_color(c, rgba_from_style ("RegionListWholeFile", 0xff, 0, 0, 0, "fg", Gtk::STATE_NORMAL, false ));
+               if (missing_source) {
+                       c.set_rgb(65535,0,0);     // FIXME: error color from style
+               } else {
+                       set_color(c, rgba_from_style ("RegionListWholeFile", 0xff, 0, 0, 0, "fg", Gtk::STATE_NORMAL, false ));
+               }
                row[region_list_columns.color_] = c;
 
-               if (region->source().name()[0] == '/') { // external file
+               if (region->source()->name()[0] == '/') { // external file
 
                        if (region->whole_file()) {
+
+                               boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(region->source());
+
                                str = ".../";
-                               str += PBD::basename_nosuffix (region->source().name());
-                               
+
+                               if (afs) {
+                                       str = region_name_from_path (afs->path(), region->n_channels() > 1);
+                               } else {
+                                       str += region->source()->name();
+                               }
+
                        } else {
                                str = region->name();
                        }
@@ -134,6 +160,18 @@ Editor::add_audio_region_to_region_display (AudioRegion *region)
 
                }
 
+               if (region->n_channels() > 1) {
+                       std::stringstream foo;
+                       foo << region->n_channels ();
+                       str += " [";
+                       str += foo.str();
+                       str += ']';
+               }
+
+               if (missing_source) {
+                       str += _(" (MISSING)");
+               }
+
                row[region_list_columns.name] = str;
                row[region_list_columns.region] = region;
 
@@ -149,11 +187,11 @@ Editor::add_audio_region_to_region_display (AudioRegion *region)
 
                for (i = rows.begin(); i != rows.end(); ++i) {
 
-                       Region* rr = (*i)[region_list_columns.region];
-                       AudioRegion* r = dynamic_cast<AudioRegion*>(rr);
+                       boost::shared_ptr<Region> rr = (*i)[region_list_columns.region];
+                       boost::shared_ptr<AudioRegion> r = boost::dynamic_pointer_cast<AudioRegion>(rr);
 
                        if (r && r->whole_file()) {
-                               if (region->source_equivalent (*r)) {
+                               if (region->source_equivalent (r)) {
                                        row = *(region_list_model->append ((*i).children()));
                                        found_parent = true;
                                        break;
@@ -193,25 +231,32 @@ Editor::region_list_selection_changed()
                TreeView::Selection::ListHandle_Path::iterator i = rows.begin();
                TreeIter iter;
 
-               /* just set the first selected region (in fact, the selection model might be SINGLE, which
-                  means there can only be one.
-               */
-               
                if ((iter = region_list_model->get_iter (*i))) {
-                       set_selected_regionview_from_region_list (*((*iter)[region_list_columns.region]), Selection::Set);
+                       boost::shared_ptr<Region> r = (*iter)[region_list_columns.region];
+                       
+                       /* they could have clicked on a row that is just a placeholder, like "Hidden" */
+                       
+                       if (r) {
+                               
+                               /* just set the first selected region (in fact, the selection model might be SINGLE, which
+                                  means there can only be one.
+                               */
+                               
+                               set_selected_regionview_from_region_list (r, Selection::Set);
+                       }
                }
        }
 }
 
 void
-Editor::insert_into_tmp_audio_regionlist(AudioRegion* region)
+Editor::insert_into_tmp_regionlist(boost::shared_ptr<Region> region)
 {
        /* keep all whole files at the beginning */
        
        if (region->whole_file()) {
-               tmp_audio_region_list.push_front (region);
+               tmp_region_list.push_front (region);
        } else {
-               tmp_audio_region_list.push_back (region);
+               tmp_region_list.push_back (region);
        }
 }
 
@@ -227,23 +272,18 @@ Editor::redisplay_regions ()
                   sorting.
                */
                
-               tmp_audio_region_list.clear();
-               session->foreach_audio_region (this, &Editor::insert_into_tmp_audio_regionlist);
+               tmp_region_list.clear();
+               session->foreach_region (this, &Editor::insert_into_tmp_regionlist);
 
-               for (list<AudioRegion*>::iterator r = tmp_audio_region_list.begin(); r != tmp_audio_region_list.end(); ++r) {
-                       add_audio_region_to_region_display (*r);
+               for (list<boost::shared_ptr<Region> >::iterator r = tmp_region_list.begin(); r != tmp_region_list.end(); ++r) {
+                       add_region_to_region_display (*r);
                }
+               tmp_region_list.clear();
                
                region_list_display.set_model (region_list_model);
        }
 }
 
-void
-Editor::region_list_clear ()
-{
-       region_list_model->clear();
-}
-
 void
 Editor::build_region_list_menu ()
 {
@@ -321,7 +361,7 @@ Editor::region_list_display_key_release (GdkEventKey* ev)
 bool
 Editor::region_list_display_button_press (GdkEventButton *ev)
 {
-       Region* region;
+       boost::shared_ptr<Region> region;
        TreeIter iter;
        TreeModel::Path path;
        TreeViewColumn* column;
@@ -334,25 +374,20 @@ Editor::region_list_display_button_press (GdkEventButton *ev)
                }
        }
 
-       if (region == 0) {
-               return false;
-       }
-
-       if (Keyboard::is_delete_event (ev)) {
-               session->remove_region_from_region_list (*region);
-               return true;
-       }
-
        if (Keyboard::is_context_menu_event (ev)) {
                show_region_list_display_context_menu (ev->button, ev->time);
                return true;
        }
 
+       if (region == 0) {
+               return false;
+       }
+
        switch (ev->button) {
        case 1:
                /* audition on double click */
                if (ev->type == GDK_2BUTTON_PRESS) {
-                       consider_auditioning (*region);
+                       consider_auditioning (region);
                        return true;
                }
                return false;
@@ -360,7 +395,7 @@ Editor::region_list_display_button_press (GdkEventButton *ev)
 
        case 2:
                if (!Keyboard::modifier_state_equals (ev->state, Keyboard::Control)) {
-                       consider_auditioning (*region);
+                       consider_auditioning (region);
                }
                return true;
                break;
@@ -380,7 +415,7 @@ Editor::region_list_display_button_release (GdkEventButton *ev)
        TreeViewColumn* column;
        int cellx;
        int celly;
-       Region* region = 0;
+       boost::shared_ptr<Region> region;
 
        if (region_list_display.get_path_at_pos ((int)ev->x, (int)ev->y, path, column, cellx, celly)) {
                if ((iter = region_list_model->get_iter (path))) {
@@ -389,7 +424,7 @@ Editor::region_list_display_button_release (GdkEventButton *ev)
        }
 
        if (region && Keyboard::is_delete_event (ev)) {
-               session->remove_region_from_region_list (*region);
+               session->remove_region_from_region_list (region);
                return true;
        }
 
@@ -397,9 +432,9 @@ Editor::region_list_display_button_release (GdkEventButton *ev)
 }
 
 void
-Editor::consider_auditioning (Region& region)
+Editor::consider_auditioning (boost::shared_ptr<Region> region)
 {
-       AudioRegion* r = dynamic_cast<AudioRegion*> (&region);
+       boost::shared_ptr<AudioRegion> r = boost::dynamic_pointer_cast<AudioRegion> (region);
 
        if (r == 0) {
                session->cancel_audition ();
@@ -413,7 +448,7 @@ Editor::consider_auditioning (Region& region)
                }
        }
 
-       session->audition_region (*r);
+       session->audition_region (r);
        last_audition_region = r;
 }
 
@@ -422,8 +457,8 @@ Editor::region_list_sorter (TreeModel::iterator a, TreeModel::iterator b)
 {
        int cmp = 0;
 
-       Region* r1 = (*a)[region_list_columns.region];
-       Region* r2 = (*b)[region_list_columns.region];
+       boost::shared_ptr<Region> r1 = (*a)[region_list_columns.region];
+       boost::shared_ptr<Region> r2 = (*b)[region_list_columns.region];
 
        /* handle rows without regions, like "Hidden" */
 
@@ -435,8 +470,8 @@ Editor::region_list_sorter (TreeModel::iterator a, TreeModel::iterator b)
                return 1;
        }
 
-       AudioRegion* region1 = dynamic_cast<AudioRegion*> (r1);
-       AudioRegion* region2 = dynamic_cast<AudioRegion*> (r2);
+       boost::shared_ptr<AudioRegion> region1 = boost::dynamic_pointer_cast<AudioRegion> (r1);
+       boost::shared_ptr<AudioRegion> region2 = boost::dynamic_pointer_cast<AudioRegion> (r2);
 
        if (region1 == 0 || region2 == 0) {
                Glib::ustring s1;
@@ -465,7 +500,7 @@ Editor::region_list_sorter (TreeModel::iterator a, TreeModel::iterator b)
                break;
                
        case ByTimestamp:
-               cmp = region1->source().timestamp() - region2->source().timestamp();
+               cmp = region1->source()->timestamp() - region2->source()->timestamp();
                break;
        
        case ByStartInFile:
@@ -477,22 +512,22 @@ Editor::region_list_sorter (TreeModel::iterator a, TreeModel::iterator b)
                break;
                
        case BySourceFileName:
-               cmp = strcasecmp (region1->source().name().c_str(), region2->source().name().c_str());
+               cmp = strcasecmp (region1->source()->name().c_str(), region2->source()->name().c_str());
                break;
 
        case BySourceFileLength:
-               cmp = region1->source().length() - region2->source().length();
+               cmp = region1->source()->length() - region2->source()->length();
                break;
                
        case BySourceFileCreationDate:
-               cmp = region1->source().timestamp() - region2->source().timestamp();
+               cmp = region1->source()->timestamp() - region2->source()->timestamp();
                break;
 
        case BySourceFileFS:
-               if (region1->source().name() == region2->source().name()) {
+               if (region1->source()->name() == region2->source()->name()) {
                        cmp = strcasecmp (region1->name().c_str(),  region2->name().c_str());
                } else {
-                       cmp = strcasecmp (region1->source().name().c_str(),  region2->source().name().c_str());
+                       cmp = strcasecmp (region1->source()->name().c_str(),  region2->source()->name().c_str());
                }
                break;
        }
@@ -522,7 +557,7 @@ Editor::reset_region_list_sort_direction (bool up)
 }
 
 void
-Editor::region_list_selection_mapover (slot<void,Region&> sl)
+Editor::region_list_selection_mapover (slot<void,boost::shared_ptr<Region> > sl)
 {
        Glib::RefPtr<TreeSelection> selection = region_list_display.get_selection();
        TreeView::Selection::ListHandle_Path rows = selection->get_selected_rows ();
@@ -536,20 +571,30 @@ Editor::region_list_selection_mapover (slot<void,Region&> sl)
                TreeIter iter;
 
                if ((iter = region_list_model->get_iter (*i))) {
-                       sl (*((*iter)[region_list_columns.region]));
+
+                       /* some rows don't have a region associated with them, but can still be
+                          selected (XXX maybe prevent them from being selected)
+                       */
+
+                       boost::shared_ptr<Region> r = (*iter)[region_list_columns.region];
+
+                       if (r) {
+                               sl (r);
+                       }
                }
        }
 }
 
 void
-Editor::hide_a_region (Region& r)
+Editor::hide_a_region (boost::shared_ptr<Region> r)
 {
-       r.set_hidden (true);
+       r->set_hidden (true);
 }
 
 void
-Editor::remove_a_region (Region& r)
+Editor::remove_a_region (boost::shared_ptr<Region> r)
 {
+       cerr << "remove " << r->name();
        session->remove_region_from_region_list (r);
 }
 
@@ -568,6 +613,7 @@ Editor::hide_region_from_region_list ()
 void
 Editor::remove_region_from_region_list ()
 {
+       cerr << "Mapping remove over region selection\n";
        region_list_selection_mapover (mem_fun (*this, &Editor::remove_a_region));
 }
 
@@ -580,7 +626,7 @@ Editor::region_list_display_drag_data_received (const RefPtr<Gdk::DragContext>&
        vector<ustring> paths;
 
        if (convert_drop_to_paths (paths, context, x, y, data, info, time) == 0) {
-               jack_nframes_t pos = 0;
+               nframes_t pos = 0;
                do_embed (paths, false, ImportAsRegion, 0, pos, true);
                context->drag_finish (true, false, time);
        }
@@ -591,5 +637,7 @@ Editor::region_list_selection_filter (const RefPtr<TreeModel>& model, const Tree
 {
        /* not possible to select rows that do not represent regions, like "Hidden" */
 
-       return (*(model->get_iter (path)))[region_list_columns.region] != 0;
+       /// XXXX FIXME boost::shared_ptr<Region> r = ((model->get_iter (path)))[region_list_columns.region];
+       /// return r != 0;
+       return true;
 }