Merging from trunk
[ardour.git] / gtk2_ardour / regionview.cc
index be3fbf8d8d323fa73d7ce57e549de39da81db5be..114aaf5da37f2232d66c512c404bb47f3fddda87 100644 (file)
@@ -27,8 +27,8 @@
 
 #include <ardour/playlist.h>
 #include <ardour/audioregion.h>
-#include <ardour/sndfilesource.h>
-#include <ardour/diskstream.h>
+#include <ardour/audiosource.h>
+#include <ardour/audio_diskstream.h>
 
 #include "streamview.h"
 #include "regionview.h"
@@ -49,6 +49,7 @@
 
 using namespace sigc;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace Editing;
 using namespace ArdourCanvas;
 
@@ -56,19 +57,25 @@ static const int32_t sync_mark_width = 9;
 
 sigc::signal<void,AudioRegionView*> AudioRegionView::AudioRegionViewGoingAway;
 
-AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, AudioTimeAxisView &tv, 
-                                 AudioRegion& r, 
-                                 double spu, 
-                                 double amplitude_above_axis,
-                                 Gdk::Color& basic_color,
-                                 bool wfw)
-
+AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, AudioTimeAxisView &tv, AudioRegion& r, double spu,
+                                 Gdk::Color& basic_color)
        : TimeAxisViewItem (r.name(), *parent, tv, spu, basic_color, r.position(), r.length(),
                            TimeAxisViewItem::Visibility (TimeAxisViewItem::ShowNameText|
                                                          TimeAxisViewItem::ShowNameHighlight|
                                                          TimeAxisViewItem::ShowFrame)),
+         region (r)
+{
+}
 
+AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, AudioTimeAxisView &tv, AudioRegion& r, double spu, 
+                                 Gdk::Color& basic_color, TimeAxisViewItem::Visibility visibility)
+       : TimeAxisViewItem (r.name(), *parent, tv, spu, basic_color, r.position(), r.length(), visibility),
          region (r)
+{
+}
+
+void
+AudioRegionView::init (double amplitude_above_axis, Gdk::Color& basic_color, bool wfw)
 {
         ArdourCanvas::Points shape;
        XMLNode *node;
@@ -98,8 +105,8 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, AudioTimeAxisView
 
        create_waves ();
 
-       gtk_object_set_data (GTK_OBJECT(name_highlight), "regionview", this);
-       gtk_object_set_data (GTK_OBJECT(name_text), "regionview", this);
+       name_highlight->set_data ("regionview", this);
+       name_text->set_data ("regionview", this);
 
        //      shape = new ArdourCanvas::Points ();
 
@@ -111,49 +118,46 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, AudioTimeAxisView
        shape.push_back (Gnome::Art::Point (-((sync_mark_width-1)/2), 1));
 
        sync_mark =  new ArdourCanvas::Polygon (*group);
-       sync_mark->property_points().set_value(shape);
-       sync_mark->set_property ("fill_color_rgba", fill_color);
+       sync_mark->property_points() = shape;
+       sync_mark->property_fill_color_rgba() = fill_color;
        sync_mark->hide();
 
        fade_in_shape = new ArdourCanvas::Polygon (*group);
-       fade_in_shape->set_property ("fill_color_rgba", fade_color);
+       fade_in_shape->property_fill_color_rgba() = fade_color;
        fade_in_shape->set_data ("regionview", this);
        
        fade_out_shape = new ArdourCanvas::Polygon (*group);
-       fade_out_shape->set_property ("fill_color_rgba", fade_color);
+       fade_out_shape->property_fill_color_rgba() = fade_color;
        fade_out_shape->set_data ("regionview", this);
-       
 
 
        {
-                       uint32_t r,g,b,a;
-                       UINT_TO_RGBA(fill_color,&r,&g,&b,&a);
-       
-
-       fade_in_handle = new ArdourCanvas::SimpleRect (*group);
-       fade_in_handle->set_property ("fill_color_rgba", RGBA_TO_UINT(r,g,b,0));
-       fade_in_handle->set_property ("outline_pixels", 0);
-       fade_in_handle->set_property ("y1", 2.0);
-       fade_in_handle->set_property ("y2", 7.0);
-
-       fade_in_handle->set_data ("regionview", this);
+               uint32_t r,g,b,a;
+               UINT_TO_RGBA(fill_color,&r,&g,&b,&a);
        
-       fade_out_handle = new ArdourCanvas::SimpleRect (*group);
-       fade_out_handle->set_property ("fill_color_rgba", RGBA_TO_UINT(r,g,b,0));
-       fade_out_handle->set_property ("outline_pixels", 0);
-       fade_out_handle->set_property ("y1", 2.0);
-       fade_out_handle->set_property ("y2", 7.0);
 
-       gtk_object_set_data (GTK_OBJECT(fade_out_handle), "regionview", this);
+               fade_in_handle = new ArdourCanvas::SimpleRect (*group);
+               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,0);
+               fade_in_handle->property_outline_pixels() = 0;
+               fade_in_handle->property_y1() = 2.0;
+               fade_in_handle->property_y2() = 7.0;
+               
+               fade_in_handle->set_data ("regionview", this);
+               
+               fade_out_handle = new ArdourCanvas::SimpleRect (*group);
+               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,0);
+               fade_out_handle->property_outline_pixels() = 0;
+               fade_out_handle->property_y1() = 2.0;
+               fade_out_handle->property_y2() = 7.0;
+               
+               fade_out_handle->set_data ("regionview", this);
        }
 
        string foo = region.name();
        foo += ':';
        foo += "gain";
 
-       gain_line = new AudioRegionGainLine (foo, tv.session(), *this, *group, region.envelope(),
-                                            PublicEditor::canvas_control_point_event,
-                                            PublicEditor::canvas_line_event);
+       gain_line = new AudioRegionGainLine (foo, trackview.session(), *this, *group, region.envelope());
 
        if (!(_flags & EnvelopeVisible)) {
                gain_line->hide ();
@@ -178,18 +182,18 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, AudioTimeAxisView
 
        region.StateChanged.connect (mem_fun(*this, &AudioRegionView::region_changed));
 
-       group->signal_event().connect (bind (mem_fun (editor, &PublicEditor::UNC (PublicEditor::canvas_region_view_event)), group, this));
-       region_view_name_highlight->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_region_view_name_highlight_event), region_view_name_highlight, this));
-       fade_in_shape->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_fade_in_event), fade_in_shape, this));
-
-       fade_in_handle->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_fade_in_handle_event), fade_in_handle, this));
-       fade_out_shape->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_fade_out_event), fade_out_shape, this));
-       fade_out_handle->signal_event().connect (bind (mem_fun (editor, &PublicEditor::canvas_fade_out_handle_event), fade_out_handle, this));
+       group->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_event), group, this));
+       name_highlight->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_region_view_name_highlight_event), name_highlight, this));
+       fade_in_shape->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_event), fade_in_shape, this));
+       fade_in_handle->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_in_handle_event), fade_in_handle, this));
+       fade_out_shape->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_event), fade_out_shape, this));
+       fade_out_handle->signal_event().connect (bind (mem_fun (PublicEditor::instance(), &PublicEditor::canvas_fade_out_handle_event), fade_out_handle, this));
 
        set_colors ();
 
-       /* XXX sync mark drag? */
+       ColorChanged.connect (mem_fun (*this, &AudioRegionView::color_handler));
 
+       /* XXX sync mark drag? */
 }
 
 AudioRegionView::~AudioRegionView ()
@@ -202,7 +206,7 @@ AudioRegionView::~AudioRegionView ()
                gnome_canvas_waveview_cache_destroy (*cache);
        }
 
-       /* all waveviews will be destroyed when the group is destroyed */
+       /* all waveviews etc will be destroyed when the group is destroyed */
 
        for (vector<GhostRegion*>::iterator g = ghosts.begin(); g != ghosts.end(); ++g) {
                delete *g;
@@ -212,7 +216,9 @@ AudioRegionView::~AudioRegionView ()
                delete editor;
        }
 
-       delete gain_line;
+       if (gain_line) {
+               delete gain_line;
+       }
 }
 
 gint
@@ -315,14 +321,14 @@ AudioRegionView::fade_in_active_changed ()
 
        if (region.fade_in_active()) {
                col = RGBA_TO_UINT(r,g,b,120);
-               fade_in_shape->set_property ("fill_color_rgba", col);
-               fade_in_shape->set_property ("width_pixels", 0);
-               fade_in_shape->set_property ("outline_color_rgba", RGBA_TO_UINT(r,g,b,0));
+               fade_in_shape->property_fill_color_rgba() = col;
+               fade_in_shape->property_width_pixels() = 0;
+               fade_in_shape->property_outline_color_rgba() = RGBA_TO_UINT(r,g,b,0);
        } else { 
                col = RGBA_TO_UINT(r,g,b,0);
-               fade_in_shape->set_property ("fill_color_rgba", col);
-               fade_in_shape->set_property ("width_pixels", 1);
-               fade_in_shape->set_property ("outline_color_rgba", RGBA_TO_UINT(r,g,b,255));
+               fade_in_shape->property_fill_color_rgba() = col;
+               fade_in_shape->property_width_pixels() = 1;
+               fade_in_shape->property_outline_color_rgba() = RGBA_TO_UINT(r,g,b,255);
        }
 }
 
@@ -335,14 +341,14 @@ AudioRegionView::fade_out_active_changed ()
 
        if (region.fade_out_active()) {
                col = RGBA_TO_UINT(r,g,b,120);
-               fade_out_shape->set_property ("fill_color_rgba", col);
-               fade_out_shape->set_property ("width_pixels", 0);
-               fade_out_shape->set_property ("outline_color_rgba", RGBA_TO_UINT(r,g,b,0));
+               fade_out_shape->property_fill_color_rgba() = col;
+               fade_out_shape->property_width_pixels() = 0;
+               fade_out_shape->property_outline_color_rgba() = RGBA_TO_UINT(r,g,b,0);
        } else { 
                col = RGBA_TO_UINT(r,g,b,0);
-               fade_out_shape->set_property ("fill_color_rgba", col);
-               fade_out_shape->set_property ("width_pixels", 1);
-               fade_out_shape->set_property ("outline_color_rgba", RGBA_TO_UINT(r,g,b,255));
+               fade_out_shape->property_fill_color_rgba() = col;
+               fade_out_shape->property_width_pixels() = 1;
+               fade_out_shape->property_outline_color_rgba() = RGBA_TO_UINT(r,g,b,255);
        }
 }
 
@@ -354,7 +360,7 @@ AudioRegionView::region_scale_amplitude_changed ()
 
        for (uint32_t n = 0; n < waves.size(); ++n) {
                // force a reload of the cache
-               waves[n]->property_data_src().set_value(&region);
+               waves[n]->property_data_src() = &region;
        }
 }
 
@@ -383,7 +389,7 @@ AudioRegionView::region_resized (Change what_changed)
                reset_width_dependent_items (unit_length);
                
                for (uint32_t n = 0; n < waves.size(); ++n) {
-                       waves[n]->property_region_start().set_value(region.start());
+                       waves[n]->property_region_start() = region.start();
                }
                
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
@@ -391,7 +397,7 @@ AudioRegionView::region_resized (Change what_changed)
                        (*i)->set_duration (unit_length);
 
                        for (vector<WaveView*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
-                               (*w)->property_region_start().set_value(region.start());
+                               (*w)->property_region_start() = region.start();
                        }
                }
        }
@@ -404,19 +410,21 @@ AudioRegionView::reset_width_dependent_items (double pixel_width)
        _pixel_width = pixel_width;
 
        if (zero_line) {
-               zero_line->set_property ("x2", pixel_width - 1.0);
+               zero_line->property_x2() = pixel_width - 1.0;
        }
 
-       if (pixel_width <= 6.0) {
-               fade_in_handle->hide();
-               fade_out_handle->hide();
-       } else {
-               if (_height < 5.0) {
+       if (fade_in_handle) {
+               if (pixel_width <= 6.0) {
                        fade_in_handle->hide();
                        fade_out_handle->hide();
                } else {
-                       fade_in_handle->show();
-                       fade_out_handle->show();
+                       if (_height < 5.0) {
+                               fade_in_handle->hide();
+                               fade_out_handle->hide();
+                       } else {
+                               fade_in_handle->show();
+                               fade_out_handle->show();
+                       }
                }
        }
 
@@ -438,9 +446,9 @@ AudioRegionView::region_muted ()
 
        for (uint32_t n=0; n < waves.size(); ++n) {
                if (region.muted()) {
-                       waves[n]->property_wave_color().set_value(color_map[cMutedWaveForm]);
+                       waves[n]->property_wave_color() = color_map[cMutedWaveForm];
                } else {
-                       waves[n]->property_wave_color().set_value(color_map[cWaveForm]);
+                       waves[n]->property_wave_color() = color_map[cWaveForm];
                }
        }
 }
@@ -518,23 +526,27 @@ AudioRegionView::set_height (gdouble height)
                
                gdouble yoff = n * (ht+1);
                
-               waves[n]->property_height().set_value(ht);
-               waves[n]->property_y().set_value(yoff + 2);
+               waves[n]->property_height() = ht;
+               waves[n]->property_y() = yoff + 2;
        }
 
-       if ((height/wcnt) < NAME_HIGHLIGHT_SIZE) {
-               gain_line->hide ();
-       } else {
-               if (_flags & EnvelopeVisible) {
-                       gain_line->show ();
+       if (gain_line) {
+               if ((height/wcnt) < NAME_HIGHLIGHT_SIZE) {
+                       gain_line->hide ();
+               } else {
+                       if (_flags & EnvelopeVisible) {
+                               gain_line->show ();
+                       }
                }
+               gain_line->set_height ((uint32_t) rint (height - NAME_HIGHLIGHT_SIZE));
        }
 
        manage_zero_line ();
-       gain_line->set_height ((uint32_t) rint (height - NAME_HIGHLIGHT_SIZE));
        reset_fade_shapes ();
-
-       name_text->raise_to_top();
+       
+       if (name_text) {
+               name_text->raise_to_top();
+       }
 }
 
 void
@@ -546,8 +558,8 @@ AudioRegionView::manage_zero_line ()
 
        if (_height >= 100) {
                gdouble wave_midpoint = (_height - NAME_HIGHLIGHT_SIZE) / 2.0;
-               zero_line->set_property ("y1", wave_midpoint);
-               zero_line->set_property ("y2", wave_midpoint);
+               zero_line->property_y1() = wave_midpoint;
+               zero_line->property_y2() = wave_midpoint;
                zero_line->show();
        } else {
                zero_line->hide();
@@ -570,6 +582,10 @@ AudioRegionView::reset_fade_in_shape ()
 void
 AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
 {
+       if (fade_in_handle == 0) {
+               return;
+       }
+
        /* smallest size for a fade is 64 frames */
 
        width = std::max ((jack_nframes_t) 64, width);
@@ -594,8 +610,8 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
                handle_center = 3.0;
        }
        
-       fade_in_handle->set_property ("x1",  handle_center - 3.0);
-       fade_in_handle->set_property ("x2",  handle_center + 3.0);
+       fade_in_handle->property_x1() =  handle_center - 3.0;
+       fade_in_handle->property_x2() =  handle_center + 3.0;
        
        if (pwidth < 5) {
                fade_in_shape->hide();
@@ -637,7 +653,7 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
 
        (*points)[pi] = (*points)[0];
        
-       fade_in_shape->property_points().set_value(*points);
+       fade_in_shape->property_points() = *points;
        delete points;
 }
 
@@ -650,6 +666,10 @@ AudioRegionView::reset_fade_out_shape ()
 void
 AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
 {      
+       if (fade_out_handle == 0) {
+               return;
+       }
+
        /* smallest size for a fade is 64 frames */
 
        width = std::max ((jack_nframes_t) 64, width);
@@ -674,8 +694,8 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
                handle_center = 3.0;
        }
        
-       fade_out_handle->set_property ("x1",  handle_center - 3.0);
-       fade_out_handle->set_property ("x2",  handle_center + 3.0);
+       fade_out_handle->property_x1() =  handle_center - 3.0;
+       fade_out_handle->property_x2() =  handle_center + 3.0;
 
        /* don't show shape if its too small */
        
@@ -703,23 +723,23 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
        double xdelta = pwidth/npoints;
 
        for (pi = 0, pc = 0; pc < npoints; ++pc) {
-               (*points)[pi] = _pixel_width - 1 - pwidth + (pc*xdelta);
-               (*points)[pi++] = 2 + (h - (curve[pc] * h));
+               (*points)[pi].set_x(_pixel_width - 1 - pwidth + (pc*xdelta));
+               (*points)[pi++].set_y(2 + (h - (curve[pc] * h)));
        }
        
        /* fold back */
 
-       (*points)[pi] = _pixel_width;
-       (*points)[pi++] = h;
+       (*points)[pi].set_x(_pixel_width);
+       (*points)[pi++].set_y(h);
 
-       (*points)[pi] = _pixel_width;
-       (*points)[pi++] = 2;
+       (*points)[pi].set_x(_pixel_width);
+       (*points)[pi++].set_y(2);
 
        /* connect the dots ... */
 
        (*points)[pi] = (*points)[0];
 
-       fade_out_shape->set_property ("points", *points);
+       fade_out_shape->property_points() = *points;
        delete points;
 }
 
@@ -729,7 +749,7 @@ AudioRegionView::set_samples_per_unit (gdouble spu)
        TimeAxisViewItem::set_samples_per_unit (spu);
 
        for (uint32_t n=0; n < waves.size(); ++n) {
-               waves[n]->property_samples_per_unit().set_value(spu);
+               waves[n]->property_samples_per_unit() = spu;
        }
 
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
@@ -737,7 +757,9 @@ AudioRegionView::set_samples_per_unit (gdouble spu)
                (*i)->set_duration (region.length() / samples_per_unit);
        }
 
-       gain_line->reset ();
+       if (gain_line) {
+               gain_line->reset ();
+       }
        reset_fade_shapes ();
        region_sync_changed ();
 }
@@ -760,7 +782,7 @@ void
 AudioRegionView::set_amplitude_above_axis (gdouble spp)
 {
        for (uint32_t n=0; n < waves.size(); ++n) {
-               waves[n]->property_amplitude_above_axis().set_value(spp);
+               waves[n]->property_amplitude_above_axis() = spp;
        }
 }
 
@@ -781,14 +803,19 @@ AudioRegionView::set_colors ()
 {
        TimeAxisViewItem::set_colors ();
        
-       gain_line->set_line_color (region.envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
-       sync_mark->set_property ("fill_color_rgba", fill_color);
+       if (gain_line) {
+               gain_line->set_line_color (region.envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
+       }
+
+       if (sync_mark) {
+               sync_mark->property_fill_color_rgba() = fill_color;
+       }
 
        for (uint32_t n=0; n < waves.size(); ++n) {
                if (region.muted()) {
-                       waves[n]->property_wave_color().set_value(color_map[cMutedWaveForm]);
+                       waves[n]->property_wave_color() = color_map[cMutedWaveForm];
                } else {
-                       waves[n]->property_wave_color().set_value(color_map[cWaveForm]);
+                       waves[n]->property_wave_color() = color_map[cWaveForm];
                }
        }
 }
@@ -840,17 +867,25 @@ AudioRegionView::region_renamed ()
                str = region.name();
        }
 
+       if (region.speed_mismatch (trackview.session().frame_rate())) {
+               str = string ("*") + str;
+       }
+
        if (region.muted()) {
                str = string ("!") + str;
        }
 
-       set_item_name (region.name(), this);
+       set_item_name (str, this);
        set_name_text (str);
 }
 
 void
 AudioRegionView::region_sync_changed ()
 {
+       if (sync_mark == 0) {
+               return;
+       }
+
        int sync_dir;
        jack_nframes_t sync_offset;
 
@@ -880,24 +915,15 @@ AudioRegionView::region_sync_changed ()
 
                        Points points;
                        
-                       points = sync_mark->property_points().get_value();
+                       //points = sync_mark->property_points().get_value();
                        
                        double offset = sync_offset / samples_per_unit;
-                       
-                       points[0].set_x(offset - ((sync_mark_width-1)/2));
-                       points[0].set_y(1);
-                       
-                       points[1].set_x(offset + (sync_mark_width-1)/2);
-                       points[1].set_y(1);
-                       
-                       points[2].set_x(offset);
-                       points[2].set_y(sync_mark_width - 1);
-                       
-                       points[3].set_x(offset - ((sync_mark_width-1)/2));
-                       points[3].set_y(1);
-                       
+                       points.push_back (Gnome::Art::Point (offset - ((sync_mark_width-1)/2), 1));
+                       points.push_back (Gnome::Art::Point (offset + ((sync_mark_width-1)/2), 1));
+                       points.push_back (Gnome::Art::Point (offset, sync_mark_width - 1));
+                       points.push_back (Gnome::Art::Point (offset - ((sync_mark_width-1)/2), 1));     
+                       sync_mark->property_points().set_value (points);
                        sync_mark->show();
-                       sync_mark->property_points().set_value(points);
 
                }
        }
@@ -925,13 +951,15 @@ AudioRegionView::set_waveform_visible (bool yn)
 void
 AudioRegionView::temporarily_hide_envelope ()
 {
-       gain_line->hide ();
+       if (gain_line) {
+               gain_line->hide ();
+       }
 }
 
 void
 AudioRegionView::unhide_envelope ()
 {
-       if (_flags & EnvelopeVisible) {
+       if (gain_line && (_flags & EnvelopeVisible)) {
                gain_line->show ();
        }
 }
@@ -939,7 +967,7 @@ AudioRegionView::unhide_envelope ()
 void
 AudioRegionView::set_envelope_visible (bool yn)
 {
-       if ((_flags & EnvelopeVisible) != yn) {
+       if (gain_line && ((_flags & EnvelopeVisible) != yn)) {
                if (yn) {
                        gain_line->show ();
                        _flags |= EnvelopeVisible;
@@ -964,12 +992,10 @@ AudioRegionView::create_waves ()
 
        uint32_t nchans = atv.get_diskstream()->n_channels();
        
-//     if (wait_for_waves) {
-               /* in tmp_waves, set up null pointers for each channel so the vector is allocated */
-               for (uint32_t n = 0; n < nchans; ++n) {
-                       tmp_waves.push_back (0);
-               }
-//     }
+       /* in tmp_waves, set up null pointers for each channel so the vector is allocated */
+       for (uint32_t n = 0; n < nchans; ++n) {
+               tmp_waves.push_back (0);
+       }
        
        for (uint32_t n = 0; n < nchans; ++n) {
                
@@ -980,7 +1006,7 @@ AudioRegionView::create_waves ()
                wave_caches.push_back (WaveView::create_cache ());
 
                if (wait_for_waves) {
-                       if (region.source(n).peaks_ready (bind (mem_fun(*this, &AudioRegionView::peaks_ready_handler), n))) {
+                       if (region.source(n).peaks_ready (bind (mem_fun(*this, &AudioRegionView::peaks_ready_handler), n), peaks_ready_connection)) {
                                create_one_wave (n, true);
                        } else {
                                create_zero_line = false;
@@ -991,10 +1017,10 @@ AudioRegionView::create_waves ()
        }
 
        if (create_zero_line) {
-               zero_line = new ArdourCanvas::Line (*group);
-               zero_line->set_property ("x1", (gdouble) 1.0);
-               zero_line->set_property ("x2", (gdouble) (region.length() / samples_per_unit) - 1.0);
-               zero_line->set_property ("color_rgba", (guint) color_map[cZeroLine]);
+               zero_line = new ArdourCanvas::SimpleLine (*group);
+               zero_line->property_x1() = (gdouble) 1.0;
+               zero_line->property_x2() = (gdouble) (region.length() / samples_per_unit) - 1.0;
+               zero_line->property_color_rgba() = (guint) color_map[cZeroLine];
                manage_zero_line ();
        }
 }
@@ -1006,49 +1032,33 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
        uint32_t nchans = atv.get_diskstream()->n_channels();
        uint32_t n;
        uint32_t nwaves = std::min (nchans, region.n_channels());
-       
        gdouble ht;
+
        if (trackview.height < NAME_HIGHLIGHT_SIZE) {
                ht = ((trackview.height) / (double) nchans);
        } else {
                ht = ((trackview.height - NAME_HIGHLIGHT_SIZE) / (double) nchans);
        }
+
        gdouble yoff = which * ht;
 
        WaveView *wave = new WaveView(*group);
 
-       wave->property_data_src().set_value( (gpointer) &region);
-       wave->property_cache().set_value( wave_caches[which]);
-       wave->property_cache_updater().set_value( (gboolean) true);
-       wave->property_channel().set_value( (guint32) which);
-       wave->property_length_function().set_value( (gpointer) region_length_from_c);
-       wave->property_sourcefile_length_function().set_value((gpointer) sourcefile_length_from_c);
-       wave->property_peak_function().set_value( (gpointer) region_read_peaks_from_c);
-       wave->property_x().set_value( 0.0);
-       wave->property_y().set_value( yoff);
-       wave->property_height().set_value( (double) ht);
-       wave->property_samples_per_unit().set_value( samples_per_unit);
-       wave->property_amplitude_above_axis().set_value( _amplitude_above_axis);
-       wave->property_wave_color().set_value(region.muted() ? color_map[cMutedWaveForm] : color_map[cWaveForm]);
-       wave->property_region_start().set_value(region.start());
-//     WaveView *wave = gnome_canvas_item_new (GNOME_CANVAS_GROUP(group),
-//                                                gnome_canvas_waveview_get_type (),
-//                                                "data_src", (gpointer) &region,
-//                                                "cache", wave_caches[which],
-//                                                "cache_updater", (gboolean) true,
-//                                                "channel", (guint32) which,
-//                                                "length_function", (gpointer) region_length_from_c,
-//                                                "sourcefile_length_function",(gpointer) sourcefile_length_from_c,
-//                                                "peak_function", (gpointer) region_read_peaks_from_c,
-//                                                "x", 0.0,
-//                                                "y", yoff,
-//                                                "height", (double) ht,
-//                                                "samples_per_unit", samples_per_unit,
-//                                                "amplitude_above_axis", _amplitude_above_axis,
-//                                                "wave_color", (guint32) (region.muted() ? color_map[cMutedWaveForm] : color_map[cWaveForm]),
-//                                                "region_start",(guint32) region.start(),
-//                                                NULL);
-       
+       wave->property_data_src() = (gpointer) &region;
+       wave->property_cache() =  wave_caches[which];
+       wave->property_cache_updater() = true;
+       wave->property_channel() =  which;
+       wave->property_length_function() = (gpointer) region_length_from_c;
+       wave->property_sourcefile_length_function() = (gpointer) sourcefile_length_from_c;
+       wave->property_peak_function() =  (gpointer) region_read_peaks_from_c;
+       wave->property_x() =  0.0;
+       wave->property_y() =  yoff;
+       wave->property_height() =  (double) ht;
+       wave->property_samples_per_unit() =  samples_per_unit;
+       wave->property_amplitude_above_axis() =  _amplitude_above_axis;
+       wave->property_wave_color() = region.muted() ? color_map[cMutedWaveForm] : color_map[cWaveForm];
+       wave->property_region_start() = region.start();
+
        if (!(_flags & WaveformVisible)) {
                wave->hide();
        }
@@ -1060,7 +1070,7 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
        */
 
        if (which < nchans) {
-               tmp_waves[which] = (wave);
+               tmp_waves[which] = wave;
        } else {
                /* n-channel track, >n-channel source */
        }
@@ -1073,17 +1083,18 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
                }
        }
        
-       if (n == nwaves) {
+       if (n == nwaves && waves.empty()) {
                /* all waves are ready */
                tmp_waves.resize(nwaves);
+
                waves = tmp_waves;
                tmp_waves.clear ();
-               
+
                if (!zero_line) {
-                       zero_line = new ArdourCanvas::Line (*group);
-                       zero_line->set_property ("x1", (gdouble) 1.0);
-                       zero_line->set_property ("x2", (gdouble) (region.length() / samples_per_unit) - 1.0);
-                       zero_line->set_property ("color_rgba", (guint) color_map[cZeroLine]);
+                       zero_line = new ArdourCanvas::SimpleLine (*group);
+                       zero_line->property_x1() = (gdouble) 1.0;
+                       zero_line->property_x2() = (gdouble) (region.length() / samples_per_unit) - 1.0;
+                       zero_line->property_color_rgba() = (guint) color_map[cZeroLine];
                        manage_zero_line ();
                }
        }
@@ -1093,11 +1104,20 @@ void
 AudioRegionView::peaks_ready_handler (uint32_t which)
 {
        Gtkmm2ext::UI::instance()->call_slot (bind (mem_fun(*this, &AudioRegionView::create_one_wave), which, false));
+
+       if (!waves.empty()) {
+               /* all waves created, don't hook into peaks ready anymore */
+               peaks_ready_connection.disconnect ();           
+       }
 }
 
 void
 AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 {
+       if (gain_line == 0) {
+               return;
+       }
+
        double x, y;
 
        /* don't create points that can't be seen */
@@ -1196,7 +1216,7 @@ AudioRegionView::set_waveform_shape (WaveformShape shape)
 
        if (yn != (bool) (_flags & WaveformRectified)) {
                for (vector<WaveView *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
-                       (*wave)->property_rectified().set_value(yn);
+                       (*wave)->property_rectified() = yn;
                }
 
                if (zero_line) {
@@ -1215,12 +1235,6 @@ AudioRegionView::set_waveform_shape (WaveformShape shape)
        }
 }
 
-std::string
-AudioRegionView::get_item_name ()
-{
-       return region.name();
-}
-
 void
 AudioRegionView::move (double x_delta, double y_delta)
 {
@@ -1255,35 +1269,19 @@ AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
                
                WaveView *wave = new WaveView(*ghost->group);
 
-               wave->property_data_src().set_value( &region);
-               wave->property_cache().set_value( wave_caches[n]);
-               wave->property_cache_updater().set_value(false);
-               wave->property_channel().set_value(n);
-               wave->property_length_function().set_value((gpointer)region_length_from_c);
-               wave->property_sourcefile_length_function().set_value((gpointer) sourcefile_length_from_c);
-               wave->property_peak_function().set_value( (gpointer) region_read_peaks_from_c);
-               wave->property_x().set_value( 0.0);
-               wave->property_samples_per_unit().set_value( samples_per_unit);
-               wave->property_amplitude_above_axis().set_value( _amplitude_above_axis);
-               wave->property_wave_color().set_value(color_map[cGhostTrackWave]);
-               wave->property_region_start().set_value(region.start());
-               //              WaveView *wave = gnome_canvas_item_new (GNOME_CANVAS_GROUP(ghost->group),
-               //                                                         gnome_canvas_waveview_get_type (),
-               //                                                         "data_src", (gpointer) &region,
-               //                                                         "cache", wave_caches[n],
-               //                                                         "cache_updater", (gboolean) false,
-               //                                                         "channel", (guint32) n,
-               //                                                         "length_function", (gpointer) region_length_from_c,
-               //                                                         "sourcefile_length_function",(gpointer) sourcefile_length_from_c,
-               //                                                         "peak_function", (gpointer) region_read_peaks_from_c,
-               //                                                         "x", 0.0,
-               //                                                         "samples_per_unit", samples_per_unit,
-               //                                                         "amplitude_above_axis", _amplitude_above_axis,
-               //                                                         "wave_color", color_map[cGhostTrackWave],
-               //                                                         "region_start", (guint32) region.start(),
-               //                                                         NULL);
+               wave->property_data_src() =  &region;
+               wave->property_cache() =  wave_caches[n];
+               wave->property_cache_updater() = false;
+               wave->property_channel() = n;
+               wave->property_length_function() = (gpointer)region_length_from_c;
+               wave->property_sourcefile_length_function() = (gpointer) sourcefile_length_from_c;
+               wave->property_peak_function() =  (gpointer) region_read_peaks_from_c;
+               wave->property_x() =  0.0;
+               wave->property_samples_per_unit() =  samples_per_unit;
+               wave->property_amplitude_above_axis() =  _amplitude_above_axis;
+               wave->property_wave_color() = color_map[cGhostTrackWave];
+               wave->property_region_start() = region.start();
 
-               
                ghost->waves.push_back(wave);
        }
 
@@ -1320,7 +1318,7 @@ AudioRegionView::get_fill_color ()
 void
 AudioRegionView::entered ()
 {
-       if (_flags & EnvelopeVisible) {
+       if (gain_line && _flags & EnvelopeVisible) {
                gain_line->show_all_control_points ();
        }
 
@@ -1328,27 +1326,35 @@ AudioRegionView::entered ()
        UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
        a=255;
        
-       fade_in_handle->set_property ("fill_color_rgba", RGBA_TO_UINT(r,g,b,a));
-       fade_out_handle->set_property ("fill_color_rgba", RGBA_TO_UINT(r,g,b,a));
+       if (fade_in_handle) {
+               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+       }
 }
 
 void
 AudioRegionView::exited ()
 {
-       gain_line->hide_all_but_selected_control_points ();
+       if (gain_line) {
+               gain_line->hide_all_but_selected_control_points ();
+       }
        
        uint32_t r,g,b,a;
        UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
        a=0;
        
-       fade_in_handle->set_property ("fill_color_rgba", RGBA_TO_UINT(r,g,b,a));
-       fade_out_handle->set_property ("fill_color_rgba", RGBA_TO_UINT(r,g,b,a));
+       if (fade_in_handle) {
+               fade_in_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+               fade_out_handle->property_fill_color_rgba() = RGBA_TO_UINT(r,g,b,a);
+       }
 }
 
 void
 AudioRegionView::envelope_active_changed ()
 {
-       gain_line->set_line_color (region.envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
+       if (gain_line) {
+               gain_line->set_line_color (region.envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
+       }
 }
 
 void
@@ -1359,7 +1365,7 @@ AudioRegionView::set_waveview_data_src()
 
        for (uint32_t n = 0; n < waves.size(); ++n) {
                // TODO: something else to let it know the channel
-               waves[n]->property_data_src().set_value(&region);
+               waves[n]->property_data_src() = &region;
        }
        
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
@@ -1367,10 +1373,36 @@ AudioRegionView::set_waveview_data_src()
                (*i)->set_duration (unit_length);
                
                for (vector<WaveView*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
-                       (*w)->property_data_src().set_value(&region);
+                       (*w)->property_data_src() = &region;
                }
        }
 
 }
 
+void
+AudioRegionView::color_handler (ColorID id, uint32_t val)
+{
+       switch (id) {
+       case cMutedWaveForm:
+       case cWaveForm:
+               set_colors ();
+               break;
+
+       case cGainLineInactive:
+       case cGainLine:
+               envelope_active_changed();
+               break;
+               
+       case cZeroLine:
+               if (zero_line) {
+                       zero_line->property_color_rgba() = (guint) color_map[cZeroLine];
+               }
+               break;
+
+       case cGhostTrackWave:
+               break;
 
+       default:
+               break;
+       }
+}