Merging from trunk
[ardour.git] / gtk2_ardour / regionview.cc
index 9ec42bd62b0f85aab9540a8f14dc65c9b445217b..114aaf5da37f2232d66c512c404bb47f3fddda87 100644 (file)
 
 #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"
 #include "audio_time_axis.h"
-#include "canvas-simplerect.h"
-#include "canvas-simpleline.h"
-#include "canvas-waveview.h"
+#include "simplerect.h"
+#include "simpleline.h"
+#include "waveview.h"
 #include "public_editor.h"
 #include "region_editor.h"
 #include "region_gain_line.h"
 
 using namespace sigc;
 using namespace ARDOUR;
+using namespace PBD;
 using namespace Editing;
+using namespace ArdourCanvas;
 
 static const int32_t sync_mark_width = 9;
 
 sigc::signal<void,AudioRegionView*> AudioRegionView::AudioRegionViewGoingAway;
 
-AudioRegionView::AudioRegionView (GtkCanvasGroup *parent, AudioTimeAxisView &tv, 
-                                 AudioRegion& r, 
-                                 double spu, 
-                                 double amplitude_above_axis,
-                                 GdkColor& basic_color,
-                                 bool wfw)
-
-       : TimeAxisViewItem (r.name(), parent, tv, spu, basic_color, r.position(), r.length(),
+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)
 {
-       GtkCanvasPoints *shape;
+}
+
+void
+AudioRegionView::init (double amplitude_above_axis, Gdk::Color& basic_color, bool wfw)
+{
+        ArdourCanvas::Points shape;
        XMLNode *node;
 
        editor = 0;
@@ -97,80 +105,59 @@ AudioRegionView::AudioRegionView (GtkCanvasGroup *parent, AudioTimeAxisView &tv,
 
        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 = gtk_canvas_points_new (4);
+       //      shape = new ArdourCanvas::Points ();
 
        /* an equilateral triangle */
 
-       shape->coords[0] = -((sync_mark_width-1)/2);
-       shape->coords[1] = 1;
-
-       shape->coords[2] = (sync_mark_width - 1)/2;
-       shape->coords[3] = 1;
-
-       shape->coords[4] = 0;
-       shape->coords[5] = sync_mark_width - 1;
+       shape.push_back (Gnome::Art::Point (-((sync_mark_width-1)/2), 1));
+       shape.push_back (Gnome::Art::Point ((sync_mark_width - 1)/2, 1));
+       shape.push_back (Gnome::Art::Point (0, sync_mark_width - 1));
+       shape.push_back (Gnome::Art::Point (-((sync_mark_width-1)/2), 1));
 
-       shape->coords[6] = -((sync_mark_width-1)/2);
-       shape->coords[7] = 1;
-
-       // cerr << "set sync mark al points, nc = " << shape->num_points << endl;
-       sync_mark = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                        gtk_canvas_polygon_get_type(),
-                                        "points", shape,
-                                        "fill_color_rgba", fill_color,
-                                        NULL);
-       gtk_canvas_item_hide (sync_mark);
-       gtk_canvas_points_unref (shape);
+       sync_mark =  new ArdourCanvas::Polygon (*group);
+       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->property_fill_color_rgba() = fade_color;
+       fade_in_shape->set_data ("regionview", this);
        
-       fade_in_shape = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                            gtk_canvas_polygon_get_type(),
-                                            "fill_color_rgba", fade_color,
-                                            NULL);
-       gtk_object_set_data (GTK_OBJECT(fade_in_shape), "regionview", this);
-
-       fade_out_shape = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                             gtk_canvas_polygon_get_type(),
-                                             "fill_color_rgba", fade_color,
-                                             NULL);
-       gtk_object_set_data (GTK_OBJECT(fade_out_shape), "regionview", this);
-
+       fade_out_shape = new ArdourCanvas::Polygon (*group);
+       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);
+               uint32_t r,g,b,a;
+               UINT_TO_RGBA(fill_color,&r,&g,&b,&a);
        
 
-       fade_in_handle = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                             gtk_canvas_simplerect_get_type(),
-                                             "fill_color_rgba", RGBA_TO_UINT(r,g,b,0),
-                                             "outline_pixels", 0,
-                                             "y1", 2.0,
-                                             "y2", 7.0,
-                                             NULL);
-       gtk_object_set_data (GTK_OBJECT(fade_in_handle), "regionview", this);
-       
-       fade_out_handle = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                              gtk_canvas_simplerect_get_type(),
-                                              "fill_color_rgba", RGBA_TO_UINT(r,g,b,0),
-                                              "outline_pixels", 0,
-                                              "y1", 2.0,
-                                              "y2", 7.0,
-                                              NULL);
-       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 ();
@@ -195,37 +182,18 @@ AudioRegionView::AudioRegionView (GtkCanvasGroup *parent, AudioTimeAxisView &tv,
 
        region.StateChanged.connect (mem_fun(*this, &AudioRegionView::region_changed));
 
-       gtk_signal_connect (GTK_OBJECT(group), "event",
-                           (GtkSignalFunc) PublicEditor::canvas_region_view_event,
-                           this);
-       gtk_signal_connect (GTK_OBJECT(name_highlight), "event",
-                           (GtkSignalFunc) PublicEditor::canvas_region_view_name_highlight_event,
-                           this);
-
-       gtk_signal_connect (GTK_OBJECT(name_text), "event",
-                           (GtkSignalFunc) PublicEditor::canvas_region_view_name_event,
-                           this);
-
-       gtk_signal_connect (GTK_OBJECT(fade_in_shape), "event",
-                           (GtkSignalFunc) PublicEditor::canvas_fade_in_event,
-                           this);
-
-       gtk_signal_connect (GTK_OBJECT(fade_in_handle), "event",
-                           (GtkSignalFunc) PublicEditor::canvas_fade_in_handle_event,
-                           this);
-
-       gtk_signal_connect (GTK_OBJECT(fade_out_shape), "event",
-                           (GtkSignalFunc) PublicEditor::canvas_fade_out_event,
-                           this);
-
-       gtk_signal_connect (GTK_OBJECT(fade_out_handle), "event",
-                           (GtkSignalFunc) PublicEditor::canvas_fade_out_handle_event,
-                           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 ()
@@ -234,11 +202,11 @@ AudioRegionView::~AudioRegionView ()
 
        AudioRegionViewGoingAway (this); /* EMIT_SIGNAL */
 
-       for (vector<GtkCanvasWaveViewCache *>::iterator cache = wave_caches.begin(); cache != wave_caches.end() ; ++cache) {
-               gtk_canvas_waveview_cache_destroy (*cache);
+       for (vector<GnomeCanvasWaveViewCache *>::iterator cache = wave_caches.begin(); cache != wave_caches.end() ; ++cache) {
+               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;
@@ -248,11 +216,13 @@ AudioRegionView::~AudioRegionView ()
                delete editor;
        }
 
-       delete gain_line;
+       if (gain_line) {
+               delete gain_line;
+       }
 }
 
 gint
-AudioRegionView::_lock_toggle (GtkCanvasItem* item, GdkEvent* ev, void* arg)
+AudioRegionView::_lock_toggle (ArdourCanvas::Item* item, GdkEvent* ev, void* arg)
 {
        switch (ev->type) {
        case GDK_BUTTON_RELEASE:
@@ -351,18 +321,14 @@ AudioRegionView::fade_in_active_changed ()
 
        if (region.fade_in_active()) {
                col = RGBA_TO_UINT(r,g,b,120);
-               gtk_canvas_item_set (fade_in_shape, 
-                                    "fill_color_rgba", col,
-                                    "width_pixels", 0,
-                                    "outline_color_rgba", RGBA_TO_UINT(r,g,b,0),
-                                    NULL);
+               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);
-               gtk_canvas_item_set (fade_in_shape, 
-                                    "fill_color_rgba", col,
-                                    "width_pixels", 1,
-                                    "outline_color_rgba", RGBA_TO_UINT(r,g,b,255),
-                                    NULL);
+               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);
        }
 }
 
@@ -375,18 +341,14 @@ AudioRegionView::fade_out_active_changed ()
 
        if (region.fade_out_active()) {
                col = RGBA_TO_UINT(r,g,b,120);
-               gtk_canvas_item_set (fade_out_shape, 
-                                    "fill_color_rgba", col,
-                                    "width_pixels", 0,
-                                    "outline_color_rgba", RGBA_TO_UINT(r,g,b,0),
-                                    NULL);
+               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);
-               gtk_canvas_item_set (fade_out_shape, 
-                                    "fill_color_rgba", col,
-                                    "width_pixels", 1,
-                                    "outline_color_rgba", RGBA_TO_UINT(r,g,b,255),
-                                    NULL);
+               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);
        }
 }
 
@@ -398,7 +360,7 @@ AudioRegionView::region_scale_amplitude_changed ()
 
        for (uint32_t n = 0; n < waves.size(); ++n) {
                // force a reload of the cache
-               gtk_canvas_item_set (waves[n], "data_src", &region, NULL);
+               waves[n]->property_data_src() = &region;
        }
 }
 
@@ -427,15 +389,15 @@ AudioRegionView::region_resized (Change what_changed)
                reset_width_dependent_items (unit_length);
                
                for (uint32_t n = 0; n < waves.size(); ++n) {
-                       gtk_canvas_item_set (waves[n], "region_start", (guint32) region.start(), NULL);
+                       waves[n]->property_region_start() = region.start();
                }
                
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
 
                        (*i)->set_duration (unit_length);
 
-                       for (vector<GtkCanvasItem*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
-                               gtk_canvas_item_set ((*w), "region_start", region.start(), NULL);
+                       for (vector<WaveView*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
+                               (*w)->property_region_start() = region.start();
                        }
                }
        }
@@ -448,19 +410,21 @@ AudioRegionView::reset_width_dependent_items (double pixel_width)
        _pixel_width = pixel_width;
 
        if (zero_line) {
-               gtk_canvas_item_set (zero_line, "x2", pixel_width - 1.0, NULL);
+               zero_line->property_x2() = pixel_width - 1.0;
        }
 
-       if (pixel_width <= 6.0) {
-               gtk_canvas_item_hide (fade_in_handle);
-               gtk_canvas_item_hide (fade_out_handle);
-       } else {
-               if (_height < 5.0) {
-                       gtk_canvas_item_hide (fade_in_handle);
-                       gtk_canvas_item_hide (fade_out_handle);
+       if (fade_in_handle) {
+               if (pixel_width <= 6.0) {
+                       fade_in_handle->hide();
+                       fade_out_handle->hide();
                } else {
-                       gtk_canvas_item_show (fade_in_handle);
-                       gtk_canvas_item_show (fade_out_handle);
+                       if (_height < 5.0) {
+                               fade_in_handle->hide();
+                               fade_out_handle->hide();
+                       } else {
+                               fade_in_handle->show();
+                               fade_out_handle->show();
+                       }
                }
        }
 
@@ -482,9 +446,9 @@ AudioRegionView::region_muted ()
 
        for (uint32_t n=0; n < waves.size(); ++n) {
                if (region.muted()) {
-                       gtk_canvas_item_set (waves[n], "wave_color", color_map[cMutedWaveForm], NULL);
+                       waves[n]->property_wave_color() = color_map[cMutedWaveForm];
                } else {
-                       gtk_canvas_item_set (waves[n], "wave_color", color_map[cWaveForm], NULL);
+                       waves[n]->property_wave_color() = color_map[cWaveForm];
                }
        }
 }
@@ -535,7 +499,7 @@ AudioRegionView::set_position (jack_nframes_t pos, void* src, double* ignored)
 
        if (delta) {
                for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-                       gtk_canvas_item_move ((*i)->group, delta, 0.0);
+                       (*i)->group->move (delta, 0.0);
                }
        }
 
@@ -562,23 +526,27 @@ AudioRegionView::set_height (gdouble height)
                
                gdouble yoff = n * (ht+1);
                
-               gtk_canvas_item_set (waves[n], "height", ht, NULL);
-               gtk_canvas_item_set (waves[n], "y", yoff + 2, NULL);
+               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 ();
-
-       gtk_canvas_item_raise_to_top (name_text) ;
+       
+       if (name_text) {
+               name_text->raise_to_top();
+       }
 }
 
 void
@@ -590,13 +558,11 @@ AudioRegionView::manage_zero_line ()
 
        if (_height >= 100) {
                gdouble wave_midpoint = (_height - NAME_HIGHLIGHT_SIZE) / 2.0;
-               gtk_canvas_item_set (zero_line, 
-                                    "y1", wave_midpoint, 
-                                    "y2", wave_midpoint, 
-                                    NULL);
-               gtk_canvas_item_show (zero_line);
+               zero_line->property_y1() = wave_midpoint;
+               zero_line->property_y2() = wave_midpoint;
+               zero_line->show();
        } else {
-               gtk_canvas_item_hide (zero_line);
+               zero_line->hide();
        }
 }
 
@@ -616,18 +582,22 @@ 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);
 
-       GtkCanvasPoints* points;
+       Points* points;
        double pwidth = width / samples_per_unit;
        uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
        double h; 
        
        if (_height < 5) {
-               gtk_canvas_item_hide (fade_in_shape);
-               gtk_canvas_item_hide (fade_in_handle);
+               fade_in_shape->hide();
+               fade_in_handle->hide();
                return;
        }
 
@@ -640,17 +610,15 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
                handle_center = 3.0;
        }
        
-       gtk_canvas_item_set (fade_in_handle, 
-                            "x1",  handle_center - 3.0,
-                            "x2",  handle_center + 3.0,
-                            NULL);
+       fade_in_handle->property_x1() =  handle_center - 3.0;
+       fade_in_handle->property_x2() =  handle_center + 3.0;
        
        if (pwidth < 5) {
-               gtk_canvas_item_hide (fade_in_shape);
+               fade_in_shape->hide();
                return;
        }
 
-       gtk_canvas_item_show (fade_in_shape);
+       fade_in_shape->show();
 
        float curve[npoints];
        region.fade_in().get_vector (0, region.fade_in().back()->when, curve, npoints);
@@ -669,25 +637,24 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
        double xdelta = pwidth/npoints;
 
        for (pi = 0, pc = 0; pc < npoints; ++pc) {
-               points->coords[pi++] = 1 + (pc * xdelta);
-               points->coords[pi++] = 2 + (h - (curve[pc] * h));
+               (*points)[pi].set_x(1 + (pc * xdelta));
+               (*points)[pi++].set_y(2 + (h - (curve[pc] * h)));
        }
        
        /* fold back */
 
-       points->coords[pi++] = pwidth;
-       points->coords[pi++] = 2;
+       (*points)[pi].set_x(pwidth);
+       (*points)[pi++].set_y(2);
 
-       points->coords[pi++] = 1;
-       points->coords[pi++] = 2;
+       (*points)[pi].set_x(1);
+       (*points)[pi++].set_y(2);
 
        /* connect the dots ... */
 
-       points->coords[pi++] = points->coords[0];
-       points->coords[pi] = points->coords[1];
+       (*points)[pi] = (*points)[0];
        
-       gtk_canvas_item_set (fade_in_shape, "points", points, NULL);
-       gtk_canvas_points_unref (points);
+       fade_in_shape->property_points() = *points;
+       delete points;
 }
 
 void
@@ -699,18 +666,22 @@ 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);
 
-       GtkCanvasPoints* points;
+       Points* points;
        double pwidth = width / samples_per_unit;
        uint32_t npoints = std::min (gdk_screen_width(), (int) pwidth);
        double h;
 
        if (_height < 5) {
-               gtk_canvas_item_hide (fade_out_shape);
-               gtk_canvas_item_hide (fade_out_handle);
+               fade_out_shape->hide();
+               fade_out_handle->hide();
                return;
        }
 
@@ -723,19 +694,17 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
                handle_center = 3.0;
        }
        
-       gtk_canvas_item_set (fade_out_handle, 
-                            "x1",  handle_center - 3.0,
-                            "x2",  handle_center + 3.0,
-                            NULL);
+       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 */
        
        if (pwidth < 5) {
-               gtk_canvas_item_hide (fade_out_shape);
+               fade_out_shape->hide();
                return;
        } 
        
-       gtk_canvas_item_show (fade_out_shape);
+       fade_out_shape->show();
 
        float curve[npoints];
        region.fade_out().get_vector (0, region.fade_out().back()->when, curve, npoints);
@@ -754,25 +723,24 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
        double xdelta = pwidth/npoints;
 
        for (pi = 0, pc = 0; pc < npoints; ++pc) {
-               points->coords[pi++] = _pixel_width - 1 - pwidth + (pc*xdelta);
-               points->coords[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->coords[pi++] = _pixel_width;
-       points->coords[pi++] = h;
+       (*points)[pi].set_x(_pixel_width);
+       (*points)[pi++].set_y(h);
 
-       points->coords[pi++] = _pixel_width;
-       points->coords[pi++] = 2;
+       (*points)[pi].set_x(_pixel_width);
+       (*points)[pi++].set_y(2);
 
        /* connect the dots ... */
 
-       points->coords[pi++] = points->coords[0];
-       points->coords[pi] = points->coords[1];
+       (*points)[pi] = (*points)[0];
 
-       gtk_canvas_item_set (fade_out_shape, "points", points, NULL);
-       gtk_canvas_points_unref (points);
+       fade_out_shape->property_points() = *points;
+       delete points;
 }
 
 void
@@ -781,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) {
-               gtk_canvas_item_set (waves[n], "samples_per_unit", spu, NULL);
+               waves[n]->property_samples_per_unit() = spu;
        }
 
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
@@ -789,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 ();
 }
@@ -812,12 +782,12 @@ void
 AudioRegionView::set_amplitude_above_axis (gdouble spp)
 {
        for (uint32_t n=0; n < waves.size(); ++n) {
-               gtk_canvas_item_set (waves[n], "amplitude_above_axis", spp, NULL);
+               waves[n]->property_amplitude_above_axis() = spp;
        }
 }
 
 void
-AudioRegionView::compute_colors (GdkColor& basic_color)
+AudioRegionView::compute_colors (Gdk::Color& basic_color)
 {
        TimeAxisViewItem::compute_colors (basic_color);
        uint32_t r, g, b, a;
@@ -833,14 +803,19 @@ AudioRegionView::set_colors ()
 {
        TimeAxisViewItem::set_colors ();
        
-       gain_line->set_line_color (region.envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
-       gtk_canvas_item_set (sync_mark, "fill_color_rgba", fill_color, NULL);
+       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()) {
-                       gtk_canvas_item_set (waves[n], "wave_color", color_map[cMutedWaveForm], NULL);
+                       waves[n]->property_wave_color() = color_map[cMutedWaveForm];
                } else {
-                       gtk_canvas_item_set (waves[n], "wave_color", color_map[cWaveForm], NULL);
+                       waves[n]->property_wave_color() = color_map[cWaveForm];
                }
        }
 }
@@ -862,12 +837,13 @@ AudioRegionView::show_region_editor ()
 {
        if (editor == 0) {
                editor = new AudioRegionEditor (trackview.session(), region, *this);
-               editor->realize ();
-               trackview.editor.ensure_float (*editor);
+               // GTK2FIX : how to ensure float without realizing
+               // editor->realize ();
+               // trackview.editor.ensure_float (*editor);
        } 
 
        editor->show_all ();
-       editor->get_window().raise();
+       editor->get_window()->raise();
 }
 
 void
@@ -891,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;
 
@@ -915,7 +899,7 @@ AudioRegionView::region_sync_changed ()
 
                /* no sync mark - its the start of the region */
 
-               gtk_canvas_item_hide (sync_mark);
+               sync_mark->hide();
 
        } else {
 
@@ -923,38 +907,24 @@ AudioRegionView::region_sync_changed ()
 
                        /* no sync mark - its out of the bounds of the region */
 
-                       gtk_canvas_item_hide (sync_mark);
+                       sync_mark->hide();
 
                } else {
 
                        /* lets do it */
 
-                       GtkArg args[1];
-                       GtkCanvasPoints* points;
-                       
-                       args[0].name = X_("points");
+                       Points points;
                        
-                       gtk_object_getv (GTK_OBJECT(sync_mark), 1, args);
-                       points = static_cast<GtkCanvasPoints *> (GTK_VALUE_POINTER(args[0]));
+                       //points = sync_mark->property_points().get_value();
                        
                        double offset = sync_offset / samples_per_unit;
-                       
-                       points->coords[0] = offset - ((sync_mark_width-1)/2);
-                       points->coords[1] = 1;
-                       
-                       points->coords[2] = offset + (sync_mark_width-1)/2;
-                       points->coords[3] = 1;
-                       
-                       points->coords[4] = offset;
-                       points->coords[5] = sync_mark_width - 1;
-                       
-                       points->coords[6] = offset - ((sync_mark_width-1)/2);
-                       points->coords[7] = 1;
-                       
-                       gtk_canvas_item_show (sync_mark);
-                       gtk_canvas_item_set (sync_mark, "points", points, NULL);
+                       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();
 
-                       gtk_canvas_points_unref (points);
                }
        }
 }
@@ -965,12 +935,12 @@ AudioRegionView::set_waveform_visible (bool yn)
        if (((_flags & WaveformVisible) != yn)) {
                if (yn) {
                        for (uint32_t n=0; n < waves.size(); ++n) {
-                               gtk_canvas_item_show (waves[n]);
+                               waves[n]->show();
                        }
                        _flags |= WaveformVisible;
                } else {
                        for (uint32_t n=0; n < waves.size(); ++n) {
-                               gtk_canvas_item_hide (waves[n]);
+                               waves[n]->hide();
                        }
                        _flags &= ~WaveformVisible;
                }
@@ -981,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 ();
        }
 }
@@ -995,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;
@@ -1020,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) {
                
@@ -1033,10 +1003,10 @@ AudioRegionView::create_waves ()
                        break;
                }
                
-               wave_caches.push_back (gtk_canvas_waveview_cache_new ());
+               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;
@@ -1047,12 +1017,10 @@ AudioRegionView::create_waves ()
        }
 
        if (create_zero_line) {
-               zero_line = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                                gtk_canvas_simpleline_get_type(),
-                                                "x1", (gdouble) 1.0,
-                                                "x2", (gdouble) (region.length() / samples_per_unit) - 1.0,
-                                                "color_rgba", (guint) color_map[cZeroLine],
-                                                NULL);
+               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 ();
        }
 }
@@ -1064,35 +1032,35 @@ 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;
 
-       GtkCanvasItem *wave = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                                  gtk_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);
-       
+       WaveView *wave = new WaveView(*group);
+
+       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)) {
-               gtk_canvas_item_hide (wave);
+               wave->hide();
        }
 
        /* note: calling this function is serialized by the lock
@@ -1102,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 */
        }
@@ -1115,19 +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 = gtk_canvas_item_new (GTK_CANVAS_GROUP(group),
-                                                        gtk_canvas_simpleline_get_type(),
-                                                        "x1", (gdouble) 1.0,
-                                                        "x2", (gdouble) (region.length() / samples_per_unit) - 1.0,
-                                                        "color_rgba", (guint) color_map[cZeroLine],
-                                                        NULL);
+                       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 ();
                }
        }
@@ -1137,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 (GtkCanvasItem *item, GdkEvent *ev)
+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 */
@@ -1151,7 +1127,7 @@ AudioRegionView::add_gain_point_event (GtkCanvasItem *item, GdkEvent *ev)
        x = ev->button.x;
        y = ev->button.y;
 
-       gtk_canvas_item_w2i (item, &x, &y);
+       item->w2i (x, y);
 
        jack_nframes_t fx = trackview.editor.pixel_to_frame (x);
 
@@ -1184,9 +1160,9 @@ AudioRegionView::add_gain_point_event (GtkCanvasItem *item, GdkEvent *ev)
 }
 
 void
-AudioRegionView::remove_gain_point_event (GtkCanvasItem *item, GdkEvent *ev)
+AudioRegionView::remove_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 {
-       ControlPoint *cp = reinterpret_cast<ControlPoint *> (gtk_object_get_data(GTK_OBJECT(item), "control_point"));
+        ControlPoint *cp = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"));
        region.envelope().erase (cp->model);
 }
 
@@ -1239,15 +1215,15 @@ AudioRegionView::set_waveform_shape (WaveformShape shape)
        }
 
        if (yn != (bool) (_flags & WaveformRectified)) {
-               for (vector<GtkCanvasItem *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
-                       gtk_canvas_item_set ((*wave), "rectified", (gboolean) yn, NULL);
+               for (vector<WaveView *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
+                       (*wave)->property_rectified() = yn;
                }
 
                if (zero_line) {
                        if (yn) {
-                               gtk_canvas_item_hide (zero_line);
+                               zero_line->hide();
                        } else {
-                               gtk_canvas_item_show (zero_line);
+                               zero_line->show();
                        }
                }
 
@@ -1259,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)
 {
@@ -1272,12 +1242,12 @@ AudioRegionView::move (double x_delta, double y_delta)
                return;
        }
 
-       gtk_canvas_item_move (get_canvas_group(), x_delta, y_delta);
+       get_canvas_group()->move (x_delta, y_delta);
 
        /* note: ghosts never leave their tracks so y_delta for them is always zero */
 
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
-               gtk_canvas_item_move ((*i)->group, x_delta, 0.0);
+               (*i)->group->move (x_delta, 0.0);
        }
 }
 
@@ -1297,23 +1267,21 @@ AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
                        break;
                }
                
-               GtkCanvasItem *wave = gtk_canvas_item_new (GTK_CANVAS_GROUP(ghost->group),
-                                                          gtk_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);
+               WaveView *wave = new WaveView(*ghost->group);
+
+               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);
        }
 
@@ -1350,7 +1318,7 @@ AudioRegionView::get_fill_color ()
 void
 AudioRegionView::entered ()
 {
-       if (_flags & EnvelopeVisible) {
+       if (gain_line && _flags & EnvelopeVisible) {
                gain_line->show_all_control_points ();
        }
 
@@ -1358,27 +1326,35 @@ AudioRegionView::entered ()
        UINT_TO_RGBA(fade_color,&r,&g,&b,&a);
        a=255;
        
-       gtk_canvas_item_set (fade_in_handle,  "fill_color_rgba", RGBA_TO_UINT(r,g,b,a), NULL);
-       gtk_canvas_item_set (fade_out_handle, "fill_color_rgba", RGBA_TO_UINT(r,g,b,a), NULL);
+       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;
        
-       gtk_canvas_item_set (fade_in_handle,  "fill_color_rgba", RGBA_TO_UINT(r,g,b,a), NULL);
-       gtk_canvas_item_set (fade_out_handle, "fill_color_rgba", RGBA_TO_UINT(r,g,b,a), NULL);
+       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
@@ -1389,18 +1365,44 @@ AudioRegionView::set_waveview_data_src()
 
        for (uint32_t n = 0; n < waves.size(); ++n) {
                // TODO: something else to let it know the channel
-               gtk_canvas_item_set (waves[n], "data_src", &region, NULL);
+               waves[n]->property_data_src() = &region;
        }
        
        for (vector<GhostRegion*>::iterator i = ghosts.begin(); i != ghosts.end(); ++i) {
                
                (*i)->set_duration (unit_length);
                
-               for (vector<GtkCanvasItem*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
-                       gtk_canvas_item_set ((*w), "data_src", &region, NULL);
+               for (vector<WaveView*>::iterator w = (*i)->waves.begin(); w != (*i)->waves.end(); ++w) {
+                       (*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;
+       }
+}