Make split region work across edit groups even if the region is not selected when...
[ardour.git] / gtk2_ardour / audio_region_view.cc
index 41b95b0d42d6cd60676d4e9845a234fb31483c87..dc6de8d0f6e3dbf6c9a81cfbb7a8c8bae2c26d8c 100644 (file)
@@ -29,6 +29,7 @@
 #include <ardour/audiosource.h>
 #include <ardour/audio_diskstream.h>
 #include <pbd/memento_command.h>
+#include <pbd/stacktrace.h>
 
 #include "streamview.h"
 #include "audio_region_view.h"
 #include "public_editor.h"
 #include "audio_region_editor.h"
 #include "region_gain_line.h"
+#include "control_point.h"
 #include "ghostregion.h"
 #include "audio_time_axis.h"
 #include "utils.h"
 #include "rgb_macros.h"
 #include "gui_thread.h"
+#include "ardour_ui.h"
 
 #include "i18n.h"
 
+#define MUTED_ALPHA 0x50
+
 using namespace sigc;
 using namespace ARDOUR;
 using namespace PBD;
@@ -87,14 +92,37 @@ AudioRegionView::AudioRegionView (ArdourCanvas::Group *parent, RouteTimeAxisView
 {
 }
 
+
+AudioRegionView::AudioRegionView (const AudioRegionView& other)
+       : RegionView (other)
+       , zero_line(0)
+       , fade_in_shape(0)
+       , fade_out_shape(0)
+       , fade_in_handle(0)
+       , fade_out_handle(0)
+       , gain_line(0)
+       , _amplitude_above_axis(1.0)
+       , _flags(0)
+       , fade_color(0)
+
+{
+       Gdk::Color c;
+       int r,g,b,a;
+
+       UINT_TO_RGBA (other.fill_color, &r, &g, &b, &a);
+       c.set_rgb_p (r/255.0, g/255.0, b/255.0);
+       
+       init (c, false);
+}
+
 void
 AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
 {
        // FIXME: Some redundancy here with RegionView::init.  Need to figure out
        // where order is important and where it isn't...
        
-       RegionView::init(basic_color, wfd);
-
+       RegionView::init(basic_color, false);
+       
        XMLNode *node;
 
        _amplitude_above_axis = 1.0;
@@ -133,25 +161,23 @@ AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
                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";
+       setup_fade_handle_positions ();
+
+       string line_name = _region->name();
+       line_name += ':';
+       line_name += "gain";
 
-       gain_line = new AudioRegionGainLine (foo, trackview.session(), *this, *group, audio_region()->envelope());
+       gain_line = new AudioRegionGainLine (line_name, trackview.session(), *this, *group, audio_region()->envelope());
 
        if (!(_flags & EnvelopeVisible)) {
                gain_line->hide ();
@@ -159,11 +185,9 @@ AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
                gain_line->show ();
        }
 
-       reset_width_dependent_items ((double) _region->length() / samples_per_unit);
-
        gain_line->reset ();
 
-       set_height (trackview.height);
+       set_y_position_and_height (0, trackview.height);
 
        region_muted ();
        region_sync_changed ();
@@ -174,8 +198,6 @@ AudioRegionView::init (Gdk::Color& basic_color, bool wfd)
        fade_in_active_changed ();
        fade_out_active_changed ();
 
-       _region->StateChanged.connect (mem_fun(*this, &AudioRegionView::region_changed));
-
        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));
@@ -248,19 +270,6 @@ AudioRegionView::fade_out_changed ()
 {
        reset_fade_out_shape ();
 }
-
-void
-AudioRegionView::set_fade_in_active (bool yn)
-{
-       audio_region()->set_fade_in_active (yn);
-}
-
-void
-AudioRegionView::set_fade_out_active (bool yn)
-{
-       audio_region()->set_fade_out_active (yn);
-}
-
 void
 AudioRegionView::fade_in_active_changed ()
 {
@@ -316,19 +325,8 @@ AudioRegionView::region_scale_amplitude_changed ()
 void
 AudioRegionView::region_renamed ()
 {
-       // FIXME: ugly duplication with RegionView...
+       Glib::ustring str = RegionView::make_name ();
        
-       string str;
-
-       if (_region->locked()) {
-               str += '>';
-               str += _region->name();
-               str += '<';
-       } else {
-               str = _region->name();
-       }
-
-       // ... because of this
        if (audio_region()->speed_mismatch (trackview.session().frame_rate())) {
                str = string ("*") + str;
        }
@@ -396,46 +394,49 @@ AudioRegionView::region_muted ()
 
        for (uint32_t n=0; n < waves.size(); ++n) {
                if (_region->muted()) {
-                       waves[n]->property_wave_color() = color_map[cMutedWaveForm];
+                       waves[n]->property_wave_color() = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA);
                } else {
-                       waves[n]->property_wave_color() = color_map[cWaveForm];
+                       waves[n]->property_wave_color() = ARDOUR_UI::config()->canvasvar_WaveForm.get();
                }
        }
 }
 
 void
-AudioRegionView::set_height (gdouble height)
+AudioRegionView::set_y_position_and_height (double y, double h)
 {
-       RegionView::set_height(height);
-       
-       uint32_t wcnt = waves.size();
+       RegionView::set_y_position_and_height(y, h - 1);
 
-       for (uint32_t n=0; n < wcnt; ++n) {
-               gdouble ht;
+       _y_position = y;
+       _height = h;
 
-               if ((height) <= NAME_HIGHLIGHT_THRESH) {
-                       ht = ((height-2*wcnt) / (double) wcnt);
+       uint32_t const wcnt = waves.size();
+       for (uint32_t n = 0; n < wcnt; ++n) {
+               double ht;
+
+               if (h <= NAME_HIGHLIGHT_THRESH) {
+                       ht = ((_height - 2 * wcnt) / (double) wcnt);
                } else {
-                       ht = (((height-2*wcnt) - NAME_HIGHLIGHT_SIZE) / (double) wcnt);
+                       ht = (((_height - 2 * wcnt) - NAME_HIGHLIGHT_SIZE) / (double) wcnt);
                }
                
-               gdouble yoff = n * (ht+1);
+               double const yoff = n * (ht + 1);
                
                waves[n]->property_height() = ht;
-               waves[n]->property_y() = yoff + 2;
+               waves[n]->property_y() = _y_position + yoff + 2;
        }
 
        if (gain_line) {
-               if ((height/wcnt) < NAME_HIGHLIGHT_SIZE) {
+               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));
+               gain_line->set_y_position_and_height ((uint32_t) _y_position, (uint32_t) rint (_height - NAME_HIGHLIGHT_SIZE));
        }
 
+       setup_fade_handle_positions ();
        manage_zero_line ();
        reset_fade_shapes ();
        
@@ -444,6 +445,25 @@ AudioRegionView::set_height (gdouble height)
        }
 }
 
+void
+AudioRegionView::setup_fade_handle_positions()
+{
+       /* position of fade handle offset from the top of the region view */
+       double const handle_pos = 2;
+       /* height of fade handles */
+       double const handle_height = 5;
+
+       if (fade_in_handle) {
+               fade_in_handle->property_y1() = _y_position + handle_pos;
+               fade_in_handle->property_y2() = _y_position + handle_pos + handle_height;
+       }
+       
+       if (fade_out_handle) {
+               fade_out_handle->property_y1() = _y_position + handle_pos;
+               fade_out_handle->property_y2() = _y_position + handle_pos + handle_height;
+       }
+}
+
 void
 AudioRegionView::manage_zero_line ()
 {
@@ -452,7 +472,7 @@ AudioRegionView::manage_zero_line ()
        }
 
        if (_height >= 100) {
-               gdouble wave_midpoint = (_height - NAME_HIGHLIGHT_SIZE) / 2.0;
+               double const wave_midpoint = _y_position + (_height - NAME_HIGHLIGHT_SIZE) / 2.0;
                zero_line->property_y1() = wave_midpoint;
                zero_line->property_y2() = wave_midpoint;
                zero_line->show();
@@ -471,11 +491,11 @@ AudioRegionView::reset_fade_shapes ()
 void
 AudioRegionView::reset_fade_in_shape ()
 {
-       reset_fade_in_shape_width ((jack_nframes_t) audio_region()->fade_in().back()->when);
+       reset_fade_in_shape_width ((nframes_t) audio_region()->fade_in()->back()->when);
 }
        
 void
-AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
+AudioRegionView::reset_fade_in_shape_width (nframes_t width)
 {
        if (fade_in_handle == 0) {
                return;
@@ -483,7 +503,7 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((jack_nframes_t) 64, width);
+       width = std::max ((nframes_t) 64, width);
 
        Points* points;
        double pwidth = width / samples_per_unit;
@@ -516,7 +536,7 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
        fade_in_shape->show();
 
        float curve[npoints];
-       audio_region()->fade_in().get_vector (0, audio_region()->fade_in().back()->when, curve, npoints);
+       audio_region()->fade_in()->curve().get_vector (0, audio_region()->fade_in()->back()->when, curve, npoints);
 
        points = get_canvas_points ("fade in shape", npoints+3);
 
@@ -533,16 +553,16 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
 
        for (pi = 0, pc = 0; pc < npoints; ++pc) {
                (*points)[pi].set_x(1 + (pc * xdelta));
-               (*points)[pi++].set_y(2 + (h - (curve[pc] * h)));
+               (*points)[pi++].set_y(_y_position + 2 + (h - (curve[pc] * h)));
        }
        
        /* fold back */
 
        (*points)[pi].set_x(pwidth);
-       (*points)[pi++].set_y(2);
+       (*points)[pi++].set_y(_y_position + 2);
 
        (*points)[pi].set_x(1);
-       (*points)[pi++].set_y(2);
+       (*points)[pi++].set_y(_y_position + 2);
 
        /* connect the dots ... */
 
@@ -555,11 +575,11 @@ AudioRegionView::reset_fade_in_shape_width (jack_nframes_t width)
 void
 AudioRegionView::reset_fade_out_shape ()
 {
-       reset_fade_out_shape_width ((jack_nframes_t) audio_region()->fade_out().back()->when);
+       reset_fade_out_shape_width ((nframes_t) audio_region()->fade_out()->back()->when);
 }
 
 void
-AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
+AudioRegionView::reset_fade_out_shape_width (nframes_t width)
 {      
        if (fade_out_handle == 0) {
                return;
@@ -567,7 +587,7 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
 
        /* smallest size for a fade is 64 frames */
 
-       width = std::max ((jack_nframes_t) 64, width);
+       width = std::max ((nframes_t) 64, width);
 
        Points* points;
        double pwidth = width / samples_per_unit;
@@ -602,7 +622,7 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
        fade_out_shape->show();
 
        float curve[npoints];
-       audio_region()->fade_out().get_vector (0, audio_region()->fade_out().back()->when, curve, npoints);
+       audio_region()->fade_out()->curve().get_vector (0, audio_region()->fade_out()->back()->when, curve, npoints);
 
        if (_height > NAME_HIGHLIGHT_THRESH) {
                h = _height - NAME_HIGHLIGHT_SIZE;
@@ -619,16 +639,16 @@ AudioRegionView::reset_fade_out_shape_width (jack_nframes_t width)
 
        for (pi = 0, pc = 0; pc < npoints; ++pc) {
                (*points)[pi].set_x(_pixel_width - 1 - pwidth + (pc*xdelta));
-               (*points)[pi++].set_y(2 + (h - (curve[pc] * h)));
+               (*points)[pi++].set_y(_y_position + 2 + (h - (curve[pc] * h)));
        }
        
        /* fold back */
 
        (*points)[pi].set_x(_pixel_width);
-       (*points)[pi++].set_y(h);
+       (*points)[pi++].set_y(_y_position + h);
 
        (*points)[pi].set_x(_pixel_width);
-       (*points)[pi++].set_y(2);
+       (*points)[pi++].set_y(_y_position + 2);
 
        /* connect the dots ... */
 
@@ -643,13 +663,16 @@ AudioRegionView::set_samples_per_unit (gdouble spu)
 {
        RegionView::set_samples_per_unit (spu);
 
-       for (uint32_t n=0; n < waves.size(); ++n) {
-               waves[n]->property_samples_per_unit() = spu;
+       if (_flags & WaveformVisible) {
+               for (uint32_t n=0; n < waves.size(); ++n) {
+                       waves[n]->property_samples_per_unit() = spu;
+               }
        }
 
        if (gain_line) {
                gain_line->reset ();
        }
+
        reset_fade_shapes ();
 }
 
@@ -680,15 +703,18 @@ AudioRegionView::set_colors ()
        RegionView::set_colors();
        
        if (gain_line) {
-               gain_line->set_line_color (audio_region()->envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
+               gain_line->set_line_color (audio_region()->envelope_active() ? ARDOUR_UI::config()->canvasvar_GainLine.get() : ARDOUR_UI::config()->canvasvar_GainLineInactive.get());
        }
 
        for (uint32_t n=0; n < waves.size(); ++n) {
                if (_region->muted()) {
-                       waves[n]->property_wave_color() = color_map[cMutedWaveForm];
+                       waves[n]->property_wave_color() = UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA);
                } else {
-                       waves[n]->property_wave_color() = color_map[cWaveForm];
+                       waves[n]->property_wave_color() = ARDOUR_UI::config()->canvasvar_WaveForm.get();
                }
+
+               waves[n]->property_clip_color() = ARDOUR_UI::config()->canvasvar_WaveFormClip.get();
+               waves[n]->property_zero_color() = ARDOUR_UI::config()->canvasvar_ZeroLine.get();
        }
 }
 
@@ -702,8 +728,8 @@ AudioRegionView::show_region_editor ()
                // trackview.editor.ensure_float (*editor);
        } 
 
-       editor->show_all ();
-       editor->get_window()->raise();
+       editor->present ();
+       editor->show_all();
 }
 
 void
@@ -712,6 +738,10 @@ AudioRegionView::set_waveform_visible (bool yn)
        if (((_flags & WaveformVisible) != yn)) {
                if (yn) {
                        for (uint32_t n=0; n < waves.size(); ++n) {
+                               /* make sure the zoom level is correct, since we don't update
+                                  this when waveforms are hidden.
+                               */
+                               waves[n]->property_samples_per_unit() = samples_per_unit;
                                waves[n]->show();
                        }
                        _flags |= WaveformVisible;
@@ -759,21 +789,19 @@ AudioRegionView::set_envelope_visible (bool yn)
 void
 AudioRegionView::create_waves ()
 {
-       bool create_zero_line = true;
-
        RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
 
        if (!atv.get_diskstream()) {
                return;
        }
 
-       uint32_t nchans = atv.get_diskstream()->n_channels().get(DataType::AUDIO);
+       uint32_t nchans = atv.get_diskstream()->n_channels().n_audio();
        
        /* 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) {
                
                if (n >= audio_region()->n_channels()) {
@@ -786,27 +814,18 @@ AudioRegionView::create_waves ()
                        if (audio_region()->audio_source(n)->peaks_ready (bind (mem_fun(*this, &AudioRegionView::peaks_ready_handler), n), data_ready_connection)) {
                                create_one_wave (n, true);
                        } else {
-                               create_zero_line = false;
                        }
                } else {
                        create_one_wave (n, true);
                }
        }
-
-       if (create_zero_line) {
-               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 ();
-       }
 }
 
 void
 AudioRegionView::create_one_wave (uint32_t which, bool direct)
 {
        RouteTimeAxisView& atv (*(dynamic_cast<RouteTimeAxisView*>(&trackview))); // ick
-       uint32_t nchans = atv.get_diskstream()->n_channels().get(DataType::AUDIO);
+       uint32_t nchans = atv.get_diskstream()->n_channels().n_audio();
        uint32_t n;
        uint32_t nwaves = std::min (nchans, audio_region()->n_channels());
        gdouble ht;
@@ -833,8 +852,12 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
        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_wave_color() = _region->muted() ? UINT_RGBA_CHANGE_A(ARDOUR_UI::config()->canvasvar_WaveForm.get(), MUTED_ALPHA) : ARDOUR_UI::config()->canvasvar_WaveForm.get();
+       wave->property_clip_color() = ARDOUR_UI::config()->canvasvar_WaveFormClip.get();
+       wave->property_zero_color() = ARDOUR_UI::config()->canvasvar_ZeroLine.get();
        wave->property_region_start() = _region->start();
+       wave->property_rectified() = (bool) (_flags & WaveformRectified);
+       wave->property_logscaled() = (bool) (_flags & WaveformLogScaled);
 
        if (!(_flags & WaveformVisible)) {
                wave->hide();
@@ -859,7 +882,7 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
                        break;
                }
        }
-       
+
        if (n == nwaves && waves.empty()) {
                /* all waves are ready */
                tmp_waves.resize(nwaves);
@@ -867,11 +890,15 @@ AudioRegionView::create_one_wave (uint32_t which, bool direct)
                waves = tmp_waves;
                tmp_waves.clear ();
 
+               /* all waves created, don't hook into peaks ready anymore */
+               data_ready_connection.disconnect ();            
+
+               if(0)
                if (!zero_line) {
                        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];
+                       zero_line->property_color_rgba() = (guint) ARDOUR_UI::config()->canvasvar_ZeroLine.get();
                        manage_zero_line ();
                }
        }
@@ -881,11 +908,6 @@ 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 */
-               data_ready_connection.disconnect ();            
-       }
 }
 
 void
@@ -906,7 +928,7 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        item->w2i (x, y);
 
-       jack_nframes_t fx = trackview.editor.pixel_to_frame (x);
+       nframes_t fx = trackview.editor.pixel_to_frame (x);
 
        if (fx > _region->length()) {
                return;
@@ -914,35 +936,34 @@ AudioRegionView::add_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 
        /* compute vertical fractional position */
 
-       y = 1.0 - (y / (trackview.height - NAME_HIGHLIGHT_SIZE));
+       y = 1.0 - ((y - _y_position) / (_height - NAME_HIGHLIGHT_SIZE));
        
        /* map using gain line */
 
        gain_line->view_to_model_y (y);
 
        trackview.session().begin_reversible_command (_("add gain control point"));
-       XMLNode &before = audio_region()->envelope().get_state();
-
+       XMLNode &before = audio_region()->envelope()->get_state();
 
        if (!audio_region()->envelope_active()) {
-               XMLNode &before = audio_region()->get_state();
+               XMLNode &region_before = audio_region()->get_state();
                audio_region()->set_envelope_active(true);
-               XMLNode &after = audio_region()->get_state();
-               trackview.session().add_command (new MementoCommand<AudioRegion>(*(audio_region().get()), &before, &after));
+               XMLNode &region_after = audio_region()->get_state();
+               trackview.session().add_command (new MementoCommand<AudioRegion>(*(audio_region().get()), &region_before, &region_after));
        }
 
-       audio_region()->envelope().add (fx, y);
+       audio_region()->envelope()->add (fx, y);
        
-       XMLNode &after = audio_region()->envelope().get_state();
-       trackview.session().add_command (new MementoCommand<Curve>(audio_region()->envelope(), &before, &after));
+       XMLNode &after = audio_region()->envelope()->get_state();
+       trackview.session().add_command (new MementoCommand<AutomationList>(*audio_region()->envelope().get(), &before, &after));
        trackview.session().commit_reversible_command ();
 }
 
 void
 AudioRegionView::remove_gain_point_event (ArdourCanvas::Item *item, GdkEvent *ev)
 {
-        ControlPoint *cp = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"));
-       audio_region()->envelope().erase (cp->model);
+       ControlPoint *cp = reinterpret_cast<ControlPoint *> (item->get_data ("control_point"));
+       audio_region()->envelope()->erase (cp->model());
 }
 
 void
@@ -952,6 +973,8 @@ AudioRegionView::store_flags()
 
        node->add_property ("waveform-visible", (_flags & WaveformVisible) ? "yes" : "no");
        node->add_property ("envelope-visible", (_flags & EnvelopeVisible) ? "yes" : "no");
+       node->add_property ("waveform-rectified", (_flags & WaveformRectified) ? "yes" : "no");
+       node->add_property ("waveform-logscaled", (_flags & WaveformLogScaled) ? "yes" : "no");
 
        _region->add_extra_xml (*node);
 }
@@ -972,6 +995,18 @@ AudioRegionView::set_flags (XMLNode* node)
                        _flags |= EnvelopeVisible;
                }
        }
+
+       if ((prop = node->property ("waveform-rectified")) != 0) {
+               if (prop->value() == "yes") {
+                       _flags |= WaveformRectified;
+               }
+       }
+
+       if ((prop = node->property ("waveform-logscaled")) != 0) {
+               if (prop->value() == "yes") {
+                       _flags |= WaveformLogScaled;
+               }
+       }
 }
        
 void
@@ -1011,9 +1046,30 @@ AudioRegionView::set_waveform_shape (WaveformShape shape)
                } else {
                        _flags &= ~WaveformRectified;
                }
+               store_flags ();
+       }
+}
+
+void
+AudioRegionView::set_waveform_scale (WaveformScale scale)
+{
+       bool yn = (scale == LogWaveform);
+
+       if (yn != (bool) (_flags & WaveformLogScaled)) {
+               for (vector<WaveView *>::iterator wave = waves.begin(); wave != waves.end() ; ++wave) {
+                       (*wave)->property_logscaled() = yn;
+               }
+
+               if (yn) {
+                       _flags |= WaveformLogScaled;
+               } else {
+                       _flags &= ~WaveformLogScaled;
+               }
+               store_flags ();
        }
 }
 
+
 GhostRegion*
 AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
 {
@@ -1024,7 +1080,7 @@ AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
        GhostRegion* ghost = new GhostRegion (atv, unit_position);
        uint32_t nchans;
        
-       nchans = rtv->get_diskstream()->n_channels().get(DataType::AUDIO);
+       nchans = rtv->get_diskstream()->n_channels().n_audio();
 
        for (uint32_t n = 0; n < nchans; ++n) {
                
@@ -1044,7 +1100,9 @@ AudioRegionView::add_ghost (AutomationTimeAxisView& atv)
                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_wave_color() = ARDOUR_UI::config()->canvasvar_GhostTrackWave.get();
+               wave->property_clip_color() = ARDOUR_UI::config()->canvasvar_GhostTrackWaveClip.get();
+               wave->property_zero_color() = ARDOUR_UI::config()->canvasvar_GhostTrackZeroLine.get();
                wave->property_region_start() = _region->start();
 
                ghost->waves.push_back(wave);
@@ -1097,7 +1155,7 @@ void
 AudioRegionView::envelope_active_changed ()
 {
        if (gain_line) {
-               gain_line->set_line_color (audio_region()->envelope_active() ? color_map[cGainLine] : color_map[cGainLineInactive]);
+               gain_line->set_line_color (audio_region()->envelope_active() ? ARDOUR_UI::config()->canvasvar_GainLine.get() : ARDOUR_UI::config()->canvasvar_GainLineInactive.get());
        }
 }
 
@@ -1124,29 +1182,16 @@ AudioRegionView::set_waveview_data_src()
 }
 
 void
-AudioRegionView::color_handler (ColorID id, uint32_t val)
+AudioRegionView::color_handler ()
 {
-       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 cMutedWaveForm:
+       //case cWaveForm:
+       //case cWaveFormClip:
+       //case cZeroLine:
+       set_colors ();
 
-       case cGhostTrackWave:
-               break;
+       //case cGainLineInactive:
+       //case cGainLine:
+       envelope_active_changed();
 
-       default:
-               break;
-       }
 }