fixes from chris cannam for rb_effect bugs
[ardour.git] / libs / ardour / audioregion.cc
index bbcbcae91ff4c989fcd1c0261d1c5c02ea8b10ab..b6c1a131de124474121bbf0ecce64129b27174cd 100644 (file)
     along with this program; if not, write to the Free Software
     Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 
-    $Id$
 */
 
 #include <cmath>
 #include <climits>
 #include <cfloat>
+#include <algorithm>
 
 #include <set>
 
@@ -32,6 +32,8 @@
 #include <pbd/basename.h>
 #include <pbd/xml++.h>
 #include <pbd/stacktrace.h>
+#include <pbd/enumwriter.h>
+#include <pbd/convert.h>
 
 #include <ardour/audioregion.h>
 #include <ardour/session.h>
 #include <ardour/playlist.h>
 #include <ardour/audiofilter.h>
 #include <ardour/audiofilesource.h>
-#include <ardour/destructive_filesource.h>
+#include <ardour/region_factory.h>
+#include <ardour/transient_detector.h>
 
 #include "i18n.h"
 #include <locale.h>
 
 using namespace std;
 using namespace ARDOUR;
+using namespace PBD;
 
 /* a Session will reset these to its chosen defaults by calling AudioRegion::set_default_fade() */
 
@@ -80,7 +84,8 @@ AudioRegion::AudioRegion (boost::shared_ptr<AudioSource> src, nframes_t start, n
        set_default_fades ();
        set_default_envelope ();
 
-       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       listen_to_my_curves ();
+       listen_to_my_sources ();
 }
 
 AudioRegion::AudioRegion (boost::shared_ptr<AudioSource> src, nframes_t start, nframes_t length, const string& name, layer_t layer, Flag flags)
@@ -92,7 +97,7 @@ AudioRegion::AudioRegion (boost::shared_ptr<AudioSource> src, nframes_t start, n
        /* basic AudioRegion constructor */
 
        sources.push_back (src);
-       master_sources.push_back (src);
+       master_sources.push_back (src); 
        src->GoingAway.connect (mem_fun (*this, &AudioRegion::source_deleted));
 
        boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource> (src);
@@ -105,18 +110,19 @@ AudioRegion::AudioRegion (boost::shared_ptr<AudioSource> src, nframes_t start, n
        set_default_fades ();
        set_default_envelope ();
 
-       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       listen_to_my_curves ();
+       listen_to_my_sources ();
 }
 
-AudioRegion::AudioRegion (SourceList& srcs, nframes_t start, nframes_t length, const string& name, layer_t layer, Flag flags)
+AudioRegion::AudioRegion (const SourceList& srcs, nframes_t start, nframes_t length, const string& name, layer_t layer, Flag flags)
        : Region (start, length, name, layer, flags),
          _fade_in (0.0, 2.0, 1.0, false),
          _fade_out (0.0, 2.0, 1.0, false),
          _envelope (0.0, 2.0, 1.0, false)
 {
        /* basic AudioRegion constructor */
-
-       for (SourceList::iterator i=srcs.begin(); i != srcs.end(); ++i) {
+       
+       for (SourceList::const_iterator i=srcs.begin(); i != srcs.end(); ++i) {
                sources.push_back (*i);
                master_sources.push_back (*i);
                (*i)->GoingAway.connect (mem_fun (*this, &AudioRegion::source_deleted));
@@ -132,7 +138,8 @@ AudioRegion::AudioRegion (SourceList& srcs, nframes_t start, nframes_t length, c
        set_default_fades ();
        set_default_envelope ();
 
-       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       listen_to_my_curves ();
+       listen_to_my_sources ();
 }
 
 
@@ -167,6 +174,7 @@ AudioRegion::AudioRegion (boost::shared_ptr<const AudioRegion> other, nframes_t
                        (*i)->GoingAway.connect (mem_fun (*this, &AudioRegion::source_deleted));
                }
                master_sources.push_back (*i);
+
        }
 
        /* return to default fades if the existing ones are too long */
@@ -196,7 +204,8 @@ AudioRegion::AudioRegion (boost::shared_ptr<const AudioRegion> other, nframes_t
 
        _scale_amplitude = other->_scale_amplitude;
 
-       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       listen_to_my_curves ();
+       listen_to_my_sources ();
 }
 
 AudioRegion::AudioRegion (boost::shared_ptr<const AudioRegion> other)
@@ -237,7 +246,8 @@ AudioRegion::AudioRegion (boost::shared_ptr<const AudioRegion> other)
        _fade_in_disabled = 0;
        _fade_out_disabled = 0;
        
-       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       listen_to_my_curves ();
+       listen_to_my_sources ();
 }
 
 AudioRegion::AudioRegion (boost::shared_ptr<AudioSource> src, const XMLNode& node)
@@ -261,7 +271,8 @@ AudioRegion::AudioRegion (boost::shared_ptr<AudioSource> src, const XMLNode& nod
                throw failed_constructor();
        }
 
-       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       listen_to_my_curves ();
+       listen_to_my_sources ();
 }
 
 AudioRegion::AudioRegion (SourceList& srcs, const XMLNode& node)
@@ -301,48 +312,85 @@ AudioRegion::AudioRegion (SourceList& srcs, const XMLNode& node)
                throw failed_constructor();
        }
 
-       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       listen_to_my_curves ();
+       listen_to_my_sources ();
 }
 
 AudioRegion::~AudioRegion ()
 {
+       boost::shared_ptr<Playlist> pl (playlist());
+
+       if (pl) {
+               for (SourceList::const_iterator i = sources.begin(); i != sources.end(); ++i) {
+                       (*i)->remove_playlist (pl);
+               }
+       }
+
        notify_callbacks ();
        GoingAway (); /* EMIT SIGNAL */
 }
 
+void
+AudioRegion::listen_to_my_sources ()
+{
+       for (SourceList::const_iterator i = sources.begin(); i != sources.end(); ++i) {
+               (*i)->AnalysisChanged.connect (mem_fun (*this, &AudioRegion::invalidate_transients));
+       }
+}
+
+void
+AudioRegion::listen_to_my_curves ()
+{
+       _envelope.StateChanged.connect (mem_fun (*this, &AudioRegion::envelope_changed));
+       _fade_in.StateChanged.connect (mem_fun (*this, &AudioRegion::fade_in_changed));
+       _fade_out.StateChanged.connect (mem_fun (*this, &AudioRegion::fade_out_changed));
+}
+
 bool
-AudioRegion::verify_length (nframes_t len)
+AudioRegion::verify_length (nframes_t& len)
 {
-       if (boost::dynamic_pointer_cast<DestructiveFileSource>(source())) {
+       boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(source());
+
+       if (afs && afs->destructive()) {
                return true;
        }
 
+       nframes_t maxlen = 0;
+
        for (uint32_t n=0; n < sources.size(); ++n) {
-               if (_start > sources[n]->length() - len) {
-                       return false;
-               }
+               maxlen = max (maxlen, sources[n]->length() - _start);
        }
+       
+       len = min (len, maxlen);
+       
        return true;
 }
 
 bool
-AudioRegion::verify_start_and_length (nframes_t new_start, nframes_t new_length)
+AudioRegion::verify_start_and_length (nframes_t new_start, nframes_t& new_length)
 {
-       if (boost::dynamic_pointer_cast<DestructiveFileSource>(source())) {
+       boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(source());
+
+       if (afs && afs->destructive()) {
                return true;
        }
 
+       nframes_t maxlen = 0;
+
        for (uint32_t n=0; n < sources.size(); ++n) {
-               if (new_length > sources[n]->length() - new_start) {
-                       return false;
-               }
+               maxlen = max (maxlen, sources[n]->length() - new_start);
        }
+
+       new_length = min (new_length, maxlen);
+
        return true;
 }
 bool
 AudioRegion::verify_start (nframes_t pos)
 {
-       if (boost::dynamic_pointer_cast<DestructiveFileSource>(source())) {
+       boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(source());
+
+       if (afs && afs->destructive()) {
                return true;
        }
 
@@ -357,7 +405,9 @@ AudioRegion::verify_start (nframes_t pos)
 bool
 AudioRegion::verify_start_mutable (nframes_t& new_start)
 {
-       if (boost::dynamic_pointer_cast<DestructiveFileSource>(source())) {
+       boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(source());
+
+       if (afs && afs->destructive()) {
                return true;
        }
 
@@ -404,36 +454,48 @@ AudioRegion::read_peaks (PeakData *buf, nframes_t npeaks, nframes_t offset, nfra
        }
 }
 
+nframes64_t
+AudioRegion::read (Sample* buf, nframes64_t position, nframes64_t cnt, int channel) const
+{
+       /* raw read, no fades, no gain, nada */
+       return _read_at (sources, _length, buf, 0, 0, _position + position, cnt, channel, 0, 0, true);
+}
+
 nframes_t
 AudioRegion::read_at (Sample *buf, Sample *mixdown_buffer, float *gain_buffer, nframes_t position, 
                      nframes_t cnt, 
                      uint32_t chan_n, nframes_t read_frames, nframes_t skip_frames) const
 {
-       return _read_at (sources, buf, mixdown_buffer, gain_buffer, position, cnt, chan_n, read_frames, skip_frames);
+       /* regular diskstream/butler read complete with fades etc */
+       return _read_at (sources, _length, buf, mixdown_buffer, gain_buffer, position, cnt, chan_n, read_frames, skip_frames, false);
 }
 
 nframes_t
 AudioRegion::master_read_at (Sample *buf, Sample *mixdown_buffer, float *gain_buffer, nframes_t position, 
                             nframes_t cnt, uint32_t chan_n) const
 {
-       return _read_at (master_sources, buf, mixdown_buffer, gain_buffer, position, cnt, chan_n, 0, 0);
+       return _read_at (master_sources, master_sources.front()->length(), buf, mixdown_buffer, gain_buffer, position, cnt, chan_n, 0, 0);
 }
 
 nframes_t
-AudioRegion::_read_at (const SourceList& srcs, Sample *buf, Sample *mixdown_buffer, float *gain_buffer,
+AudioRegion::_read_at (const SourceList& srcs, nframes_t limit,
+                      Sample *buf, Sample *mixdown_buffer, float *gain_buffer,
                       nframes_t position, nframes_t cnt, 
-                      uint32_t chan_n, nframes_t read_frames, nframes_t skip_frames) const
+                      uint32_t chan_n, 
+                      nframes_t read_frames, 
+                      nframes_t skip_frames,
+                      bool raw) const
 {
        nframes_t internal_offset;
        nframes_t buf_offset;
        nframes_t to_read;
 
-       /* precondition: caller has verified that we cover the desired section */
-
-       if (chan_n >= sources.size()) {
+       if (muted() && !raw) {
                return 0; /* read nothing */
        }
-       
+
+       /* precondition: caller has verified that we cover the desired section */
+
        if (position < _position) {
                internal_offset = 0;
                buf_offset = _position - position;
@@ -443,128 +505,151 @@ AudioRegion::_read_at (const SourceList& srcs, Sample *buf, Sample *mixdown_buff
                buf_offset = 0;
        }
 
-       if (internal_offset >= _length) {
+       if (internal_offset >= limit) {
                return 0; /* read nothing */
        }
 
-       if ((to_read = min (cnt, _length - internal_offset)) == 0) {
+       if ((to_read = min (cnt, limit - internal_offset)) == 0) {
                return 0; /* read nothing */
        }
 
-       if (opaque()) {
+       if (opaque() || raw) {
                /* overwrite whatever is there */
                mixdown_buffer = buf + buf_offset;
        } else {
                mixdown_buffer += buf_offset;
        }
 
-       if (muted()) {
-               return 0; /* read nothing */
-       }
-
-       _read_data_count = 0;
-
-       if (srcs[chan_n]->read (mixdown_buffer, _start + internal_offset, to_read) != to_read) {
-               return 0; /* "read nothing" */
+       if (!raw) {
+               _read_data_count = 0;
        }
 
-       _read_data_count += srcs[chan_n]->read_data_count();
+       if (chan_n < n_channels()) {
 
-       /* fade in */
-
-       if (_flags & FadeIn) {
-
-               nframes_t fade_in_length = (nframes_t) _fade_in.back()->when;
+               if (srcs[chan_n]->read (mixdown_buffer, _start + internal_offset, to_read) != to_read) {
+                       
+                       return 0; /* "read nothing" */
+               }
                
-               /* see if this read is within the fade in */
+               if (!raw) {
+                       _read_data_count += srcs[chan_n]->read_data_count();
+               }
 
-               if (internal_offset < fade_in_length) {
-                       
-                       nframes_t limit;
+       } else {
+               
+               /* track is N-channel, this region has less channels; silence the ones
+                  we don't have.
+               */
 
-                       limit = min (to_read, fade_in_length - internal_offset);
+               memset (mixdown_buffer, 0, sizeof (Sample) * cnt);
 
-                       _fade_in.get_vector (internal_offset, internal_offset+limit, gain_buffer, limit);
+               /* no fades required */
 
-                       for (nframes_t n = 0; n < limit; ++n) {
-                               mixdown_buffer[n] *= gain_buffer[n];
-                       }
+               if (!raw) {
+                       goto merge;
                }
        }
-       
-       /* fade out */
 
-       if (_flags & FadeOut) {
-       
-               /* see if some part of this read is within the fade out */
+       /* fade in */
 
+       if (!raw) {
+       
+               if (_flags & FadeIn) {
+                       
+                       nframes_t fade_in_length = (nframes_t) _fade_in.back()->when;
+                       
+                       /* see if this read is within the fade in */
+                       
+                       if (internal_offset < fade_in_length) {
+                               
+                               nframes_t limit;
+                               
+                               limit = min (to_read, fade_in_length - internal_offset);
+                               
+                               _fade_in.get_vector (internal_offset, internal_offset+limit, gain_buffer, limit);
+                               
+                               for (nframes_t n = 0; n < limit; ++n) {
+                                       mixdown_buffer[n] *= gain_buffer[n];
+                               }
+                       }
+               }
+               
+               /* fade out */
+               
+               if (_flags & FadeOut) {
+                       
+                       /* see if some part of this read is within the fade out */
+                       
                /* .................        >|            REGION
-                                           _length
+                                           limit
                                            
                                  {           }            FADE
                                             fade_out_length
                                  ^                                          
-                               _length - fade_out_length
+                                limit - fade_out_length
                         |--------------|
                         ^internal_offset
                                        ^internal_offset + to_read
-
-                  we need the intersection of [internal_offset,internal_offset+to_read] with
-                  [_length - fade_out_length, _length]
-
+                                      
+                                      we need the intersection of [internal_offset,internal_offset+to_read] with
+                                      [limit - fade_out_length, limit]
+                                      
                */
 
        
-               nframes_t fade_out_length = (nframes_t) _fade_out.back()->when;
-               nframes_t fade_interval_start = max(internal_offset, _length-fade_out_length);
-               nframes_t fade_interval_end   = min(internal_offset + to_read, _length);
-
-               if (fade_interval_end > fade_interval_start) {
-                       /* (part of the) the fade out is  in this buffer */
+                       nframes_t fade_out_length = (nframes_t) _fade_out.back()->when;
+                       nframes_t fade_interval_start = max(internal_offset, limit-fade_out_length);
+                       nframes_t fade_interval_end   = min(internal_offset + to_read, limit);
                        
-                       nframes_t limit = fade_interval_end - fade_interval_start;
-                       nframes_t curve_offset = fade_interval_start - (_length-fade_out_length);
-                       nframes_t fade_offset = fade_interval_start - internal_offset;
-                                                                      
-                       _fade_out.get_vector (curve_offset,curve_offset+limit, gain_buffer, limit);
-
-                       for (nframes_t n = 0, m = fade_offset; n < limit; ++n, ++m) {
-                               mixdown_buffer[m] *= gain_buffer[n];
-                       }
-               } 
-
-       }
-
-       /* Regular gain curves */
-
-       if (envelope_active())  {
-               _envelope.get_vector (internal_offset, internal_offset + to_read, gain_buffer, to_read);
+                       if (fade_interval_end > fade_interval_start) {
+                               /* (part of the) the fade out is  in this buffer */
+                               
+                               nframes_t limit = fade_interval_end - fade_interval_start;
+                               nframes_t curve_offset = fade_interval_start - (limit-fade_out_length);
+                               nframes_t fade_offset = fade_interval_start - internal_offset;
+                               
+                               _fade_out.get_vector (curve_offset,curve_offset+limit, gain_buffer, limit);
+                               
+                               for (nframes_t n = 0, m = fade_offset; n < limit; ++n, ++m) {
+                                       mixdown_buffer[m] *= gain_buffer[n];
+                               }
+                       } 
+                       
+               }
                
-               if (_scale_amplitude != 1.0f) {
-                       for (nframes_t n = 0; n < to_read; ++n) {
-                               mixdown_buffer[n] *= gain_buffer[n] * _scale_amplitude;
+               /* Regular gain curves */
+               
+               if (envelope_active())  {
+                       _envelope.get_vector (internal_offset, internal_offset + to_read, gain_buffer, to_read);
+                       
+                       if (_scale_amplitude != 1.0f) {
+                               for (nframes_t n = 0; n < to_read; ++n) {
+                                       mixdown_buffer[n] *= gain_buffer[n] * _scale_amplitude;
+                               }
+                       } else {
+                               for (nframes_t n = 0; n < to_read; ++n) {
+                                       mixdown_buffer[n] *= gain_buffer[n];
+                               }
                        }
-               } else {
+               } else if (_scale_amplitude != 1.0f) {
+                       Session::apply_gain_to_buffer (mixdown_buffer, to_read, _scale_amplitude);
+               }
+       
+         merge:
+               
+               if (!opaque()) {
+                       
+                       /* gack. the things we do for users.
+                        */
+                       
+                       buf += buf_offset;
+                       
                        for (nframes_t n = 0; n < to_read; ++n) {
-                               mixdown_buffer[n] *= gain_buffer[n];
+                               buf[n] += mixdown_buffer[n];
                        }
-               }
-       } else if (_scale_amplitude != 1.0f) {
-               Session::apply_gain_to_buffer (mixdown_buffer, to_read, _scale_amplitude);
+               } 
        }
 
-       if (!opaque()) {
-
-               /* gack. the things we do for users.
-                */
-
-               buf += buf_offset;
-
-               for (nframes_t n = 0; n < to_read; ++n) {
-                       buf[n] += mixdown_buffer[n];
-               }
-       } 
-       
        return to_read;
 }
        
@@ -577,8 +662,8 @@ AudioRegion::state (bool full)
        char buf2[64];
        LocaleGuard lg (X_("POSIX"));
        
-       snprintf (buf, sizeof (buf), "0x%x", (int) _flags);
-       node.add_property ("flags", buf);
+       node.add_property ("flags", enum_2_string (_flags));
+
        snprintf (buf, sizeof(buf), "%.12g", _scale_amplitude);
        node.add_property ("scale-gain", buf);
 
@@ -640,6 +725,12 @@ AudioRegion::state (bool full)
                child->add_property ("default", "yes");
        }
 
+       for (uint32_t n=0; n < master_sources.size(); ++n) {
+               snprintf (buf2, sizeof(buf2), "master-source-%d", n);
+               master_sources[n]->id().print (buf, sizeof (buf));
+               node.add_property (buf2, buf);
+       }
+
        if (full && _extra_xml) {
                node.add_child_copy (*_extra_xml);
        }
@@ -657,9 +748,11 @@ AudioRegion::set_live_state (const XMLNode& node, Change& what_changed, bool sen
        Region::set_live_state (node, what_changed, false);
 
        uint32_t old_flags = _flags;
-               
+
        if ((prop = node.property ("flags")) != 0) {
-               _flags = Flag (strtol (prop->value().c_str(), (char **) 0, 16));
+               _flags = Flag (string_2_enum (prop->value(), _flags));
+
+               //_flags = Flag (strtol (prop->value().c_str(), (char **) 0, 16));
 
                _flags = Flag (_flags & ~Region::LeftOfSplit);
                _flags = Flag (_flags & ~Region::RightOfSplit);
@@ -677,6 +770,7 @@ AudioRegion::set_live_state (const XMLNode& node, Change& what_changed, bool sen
 
        if ((prop = node.property ("scale-gain")) != 0) {
                _scale_amplitude = atof (prop->value().c_str());
+               what_changed = Change (what_changed|ScaleAmplitudeChanged);
        } else {
                _scale_amplitude = 1.0;
        }
@@ -705,17 +799,44 @@ AudioRegion::set_live_state (const XMLNode& node, Change& what_changed, bool sen
                        
                        _fade_in.clear ();
                        
-                       if ((prop = child->property ("default")) != 0 || (prop = child->property ("steepness")) != 0 || _fade_in.set_state (*child)) {
+                       if ((prop = child->property ("default")) != 0 || (prop = child->property ("steepness")) != 0) {
                                set_default_fade_in ();
-                       } 
+                       } else {
+                               XMLNode* grandchild = child->child ("AutomationList");
+                               if (grandchild) {
+                                       _fade_in.set_state (*grandchild);
+                               }
+                       }
+
+                       if ((prop = child->property ("active")) != 0) {
+                               if (prop->value() == "yes") {
+                                       set_fade_in_active (true);
+                               } else {
+                                       set_fade_in_active (true);
+                               }
+                       }
 
                } else if (child->name() == "FadeOut") {
                        
                        _fade_out.clear ();
 
-                       if ((prop = child->property ("default")) != 0 || (prop = child->property ("steepness")) != 0 || _fade_out.set_state (*child)) {
+                       if ((prop = child->property ("default")) != 0 || (prop = child->property ("steepness")) != 0) {
                                set_default_fade_out ();
-                       } 
+                       } else {
+                               XMLNode* grandchild = child->child ("AutomationList");
+                               if (grandchild) {
+                                       _fade_out.set_state (*grandchild);
+                               } 
+                       }
+
+                       if ((prop = child->property ("active")) != 0) {
+                               if (prop->value() == "yes") {
+                                       set_fade_out_active (true);
+                               } else {
+                                       set_fade_out_active (false);
+                               }
+                       }
+
                } 
        }
 
@@ -757,49 +878,49 @@ AudioRegion::set_fade_in (FadeShape shape, nframes_t len)
 
        switch (shape) {
        case Linear:
-               _fade_in.add (0.0, 0.0);
-               _fade_in.add (len, 1.0);
+               _fade_in.fast_simple_add (0.0, 0.0);
+               _fade_in.fast_simple_add (len, 1.0);
                break;
 
        case Fast:
-               _fade_in.add (0, 0);
-               _fade_in.add (len * 0.389401, 0.0333333);
-               _fade_in.add (len * 0.629032, 0.0861111);
-               _fade_in.add (len * 0.829493, 0.233333);
-               _fade_in.add (len * 0.9447, 0.483333);
-               _fade_in.add (len * 0.976959, 0.697222);
-               _fade_in.add (len, 1);
+               _fade_in.fast_simple_add (0, 0);
+               _fade_in.fast_simple_add (len * 0.389401, 0.0333333);
+               _fade_in.fast_simple_add (len * 0.629032, 0.0861111);
+               _fade_in.fast_simple_add (len * 0.829493, 0.233333);
+               _fade_in.fast_simple_add (len * 0.9447, 0.483333);
+               _fade_in.fast_simple_add (len * 0.976959, 0.697222);
+               _fade_in.fast_simple_add (len, 1);
                break;
 
        case Slow:
-               _fade_in.add (0, 0);
-               _fade_in.add (len * 0.0207373, 0.197222);
-               _fade_in.add (len * 0.0645161, 0.525);
-               _fade_in.add (len * 0.152074, 0.802778);
-               _fade_in.add (len * 0.276498, 0.919444);
-               _fade_in.add (len * 0.481567, 0.980556);
-               _fade_in.add (len * 0.767281, 1);
-               _fade_in.add (len, 1);
+               _fade_in.fast_simple_add (0, 0);
+               _fade_in.fast_simple_add (len * 0.0207373, 0.197222);
+               _fade_in.fast_simple_add (len * 0.0645161, 0.525);
+               _fade_in.fast_simple_add (len * 0.152074, 0.802778);
+               _fade_in.fast_simple_add (len * 0.276498, 0.919444);
+               _fade_in.fast_simple_add (len * 0.481567, 0.980556);
+               _fade_in.fast_simple_add (len * 0.767281, 1);
+               _fade_in.fast_simple_add (len, 1);
                break;
 
        case LogA:
-               _fade_in.add (0, 0);
-               _fade_in.add (len * 0.0737327, 0.308333);
-               _fade_in.add (len * 0.246544, 0.658333);
-               _fade_in.add (len * 0.470046, 0.886111);
-               _fade_in.add (len * 0.652074, 0.972222);
-               _fade_in.add (len * 0.771889, 0.988889);
-               _fade_in.add (len, 1);
+               _fade_in.fast_simple_add (0, 0);
+               _fade_in.fast_simple_add (len * 0.0737327, 0.308333);
+               _fade_in.fast_simple_add (len * 0.246544, 0.658333);
+               _fade_in.fast_simple_add (len * 0.470046, 0.886111);
+               _fade_in.fast_simple_add (len * 0.652074, 0.972222);
+               _fade_in.fast_simple_add (len * 0.771889, 0.988889);
+               _fade_in.fast_simple_add (len, 1);
                break;
 
        case LogB:
-               _fade_in.add (0, 0);
-               _fade_in.add (len * 0.304147, 0.0694444);
-               _fade_in.add (len * 0.529954, 0.152778);
-               _fade_in.add (len * 0.725806, 0.333333);
-               _fade_in.add (len * 0.847926, 0.558333);
-               _fade_in.add (len * 0.919355, 0.730556);
-               _fade_in.add (len, 1);
+               _fade_in.fast_simple_add (0, 0);
+               _fade_in.fast_simple_add (len * 0.304147, 0.0694444);
+               _fade_in.fast_simple_add (len * 0.529954, 0.152778);
+               _fade_in.fast_simple_add (len * 0.725806, 0.333333);
+               _fade_in.fast_simple_add (len * 0.847926, 0.558333);
+               _fade_in.fast_simple_add (len * 0.919355, 0.730556);
+               _fade_in.fast_simple_add (len, 1);
                break;
        }
 
@@ -817,47 +938,47 @@ AudioRegion::set_fade_out (FadeShape shape, nframes_t len)
 
        switch (shape) {
        case Fast:
-               _fade_out.add (len * 0, 1);
-               _fade_out.add (len * 0.023041, 0.697222);
-               _fade_out.add (len * 0.0553,   0.483333);
-               _fade_out.add (len * 0.170507, 0.233333);
-               _fade_out.add (len * 0.370968, 0.0861111);
-               _fade_out.add (len * 0.610599, 0.0333333);
-               _fade_out.add (len * 1, 0);
+               _fade_out.fast_simple_add (len * 0, 1);
+               _fade_out.fast_simple_add (len * 0.023041, 0.697222);
+               _fade_out.fast_simple_add (len * 0.0553,   0.483333);
+               _fade_out.fast_simple_add (len * 0.170507, 0.233333);
+               _fade_out.fast_simple_add (len * 0.370968, 0.0861111);
+               _fade_out.fast_simple_add (len * 0.610599, 0.0333333);
+               _fade_out.fast_simple_add (len * 1, 0);
                break;
 
        case LogA:
-               _fade_out.add (len * 0, 1);
-               _fade_out.add (len * 0.228111, 0.988889);
-               _fade_out.add (len * 0.347926, 0.972222);
-               _fade_out.add (len * 0.529954, 0.886111);
-               _fade_out.add (len * 0.753456, 0.658333);
-               _fade_out.add (len * 0.9262673, 0.308333);
-               _fade_out.add (len * 1, 0);
+               _fade_out.fast_simple_add (len * 0, 1);
+               _fade_out.fast_simple_add (len * 0.228111, 0.988889);
+               _fade_out.fast_simple_add (len * 0.347926, 0.972222);
+               _fade_out.fast_simple_add (len * 0.529954, 0.886111);
+               _fade_out.fast_simple_add (len * 0.753456, 0.658333);
+               _fade_out.fast_simple_add (len * 0.9262673, 0.308333);
+               _fade_out.fast_simple_add (len * 1, 0);
                break;
 
        case Slow:
-               _fade_out.add (len * 0, 1);
-               _fade_out.add (len * 0.305556, 1);
-               _fade_out.add (len * 0.548611, 0.991736);
-               _fade_out.add (len * 0.759259, 0.931129);
-               _fade_out.add (len * 0.918981, 0.68595);
-               _fade_out.add (len * 0.976852, 0.22865);
-               _fade_out.add (len * 1, 0);
+               _fade_out.fast_simple_add (len * 0, 1);
+               _fade_out.fast_simple_add (len * 0.305556, 1);
+               _fade_out.fast_simple_add (len * 0.548611, 0.991736);
+               _fade_out.fast_simple_add (len * 0.759259, 0.931129);
+               _fade_out.fast_simple_add (len * 0.918981, 0.68595);
+               _fade_out.fast_simple_add (len * 0.976852, 0.22865);
+               _fade_out.fast_simple_add (len * 1, 0);
                break;
 
        case LogB:
-               _fade_out.add (len * 0, 1);
-               _fade_out.add (len * 0.080645, 0.730556);
-               _fade_out.add (len * 0.277778, 0.289256);
-               _fade_out.add (len * 0.470046, 0.152778);
-               _fade_out.add (len * 0.695853, 0.0694444);
-               _fade_out.add (len * 1, 0);
+               _fade_out.fast_simple_add (len * 0, 1);
+               _fade_out.fast_simple_add (len * 0.080645, 0.730556);
+               _fade_out.fast_simple_add (len * 0.277778, 0.289256);
+               _fade_out.fast_simple_add (len * 0.470046, 0.152778);
+               _fade_out.fast_simple_add (len * 0.695853, 0.0694444);
+               _fade_out.fast_simple_add (len * 1, 0);
                break;
 
        case Linear:
-               _fade_out.add (len * 0, 1);
-               _fade_out.add (len * 1, 0);
+               _fade_out.fast_simple_add (len * 0, 1);
+               _fade_out.fast_simple_add (len * 1, 0);
                break;
        }
 
@@ -870,6 +991,10 @@ AudioRegion::set_fade_out (FadeShape shape, nframes_t len)
 void
 AudioRegion::set_fade_in_length (nframes_t len)
 {
+       if (len > _length) {
+               len = _length - 1;
+       }
+
        bool changed = _fade_in.extend_to (len);
 
        if (changed) {
@@ -881,13 +1006,16 @@ AudioRegion::set_fade_in_length (nframes_t len)
 void
 AudioRegion::set_fade_out_length (nframes_t len)
 {
+       if (len > _length) {
+               len = _length - 1;
+       }
+
        bool changed =  _fade_out.extend_to (len);
 
        if (changed) {
                _flags = Flag (_flags & ~DefaultFadeOut);
+               send_change (FadeOutChanged);
        }
-
-       send_change (FadeOutChanged);
 }
 
 void
@@ -920,6 +1048,18 @@ AudioRegion::set_fade_out_active (bool yn)
        send_change (FadeOutActiveChanged);
 }
 
+bool
+AudioRegion::fade_in_is_default () const
+{
+       return _fade_in_shape == Linear && _fade_in.back()->when == 64;
+}
+
+bool
+AudioRegion::fade_out_is_default () const
+{
+       return _fade_out_shape == Linear && _fade_out.back()->when == 64;
+}
+
 void
 AudioRegion::set_default_fade_in ()
 {
@@ -946,8 +1086,8 @@ AudioRegion::set_default_envelope ()
 {
        _envelope.freeze ();
        _envelope.clear ();
-       _envelope.add (0, 1.0f);
-       _envelope.add (_length, 1.0f);
+       _envelope.fast_simple_add (0, 1.0f);
+       _envelope.fast_simple_add (_length, 1.0f);
        _envelope.thaw ();
 }
 
@@ -993,25 +1133,48 @@ AudioRegion::recompute_at_start ()
 }
 
 int
-AudioRegion::separate_by_channel (Session& session, vector<AudioRegion*>& v) const
+AudioRegion::separate_by_channel (Session& session, vector<boost::shared_ptr<AudioRegion> >& v) const
 {
        SourceList srcs;
        string new_name;
+       int n;
+
+       if (sources.size() < 2) {
+               return 0;
+       }
 
-       for (SourceList::const_iterator i = master_sources.begin(); i != master_sources.end(); ++i) {
+       n = 0;
+
+       for (SourceList::const_iterator i = sources.begin(); i != sources.end(); ++i) {
 
                srcs.clear ();
                srcs.push_back (*i);
 
-               /* generate a new name */
-               
-               if (session.region_name (new_name, _name)) {
-                       return -1;
+               new_name = _name;
+
+               if (sources.size() == 2) {
+                       if (n == 0) {
+                               new_name += "-L";
+                       } else {
+                               new_name += "-R";
+                       }
+               } else {
+                       new_name += '-';
+                       new_name += ('0' + n + 1);
                }
 
-               /* create a copy with just one source */
+               /* create a copy with just one source. prevent if from being thought of as "whole file" even if 
+                  it covers the entire source file(s).
+                */
+
+               Flag f = Flag (_flags & ~WholeFile);
 
-               v.push_back (new AudioRegion (srcs, _start, _length, new_name, _layer, _flags));
+               boost::shared_ptr<Region> r = RegionFactory::create (srcs, _start, _length, new_name, _layer, f);
+               boost::shared_ptr<AudioRegion> ar = boost::dynamic_pointer_cast<AudioRegion> (r);
+
+               v.push_back (ar);
+               
+               ++n;
        }
 
        return 0;
@@ -1037,6 +1200,12 @@ AudioRegion::master_source_names ()
        return names;
 }
 
+void
+AudioRegion::set_master_sources (const SourceList& srcs)
+{
+       master_sources = srcs;
+}
+
 bool
 AudioRegion::source_equivalent (boost::shared_ptr<const Region> o) const
 {
@@ -1135,31 +1304,33 @@ AudioRegion::exportme (Session& session, AudioExportSpecification& spec)
 }
 
 boost::shared_ptr<Region>
-AudioRegion::get_parent()
+AudioRegion::get_parent() const
 {
-       boost::shared_ptr<Region> r;
+       boost::shared_ptr<Playlist> pl (playlist());
 
-       if (_playlist) {
+       if (pl) {
                boost::shared_ptr<AudioRegion> ar;
-               boost::shared_ptr<AudioRegion> grrr2 = boost::dynamic_pointer_cast<AudioRegion> (shared_from_this());
+               boost::shared_ptr<AudioRegion const> grrr2 = boost::dynamic_pointer_cast<AudioRegion const> (shared_from_this());
                
-               if (grrr2 && (ar = _playlist->session().find_whole_file_parent (grrr2))) {
+               if (grrr2 && (ar = pl->session().find_whole_file_parent (grrr2))) {
                        return boost::static_pointer_cast<Region> (ar);
                }
        }
        
-       return r;
+       return boost::shared_ptr<Region>();
 }
 
 void
 AudioRegion::set_scale_amplitude (gain_t g)
 {
+       boost::shared_ptr<Playlist> pl (playlist());
+
        _scale_amplitude = g;
 
        /* tell the diskstream we're in */
-
-       if (_playlist) {
-               _playlist->Modified();
+       
+       if (pl) {
+               pl->Modified();
        }
 
        /* tell everybody else */
@@ -1226,8 +1397,10 @@ AudioRegion::normalize_to (float target_dB)
 
        /* tell the diskstream we're in */
 
-       if (_playlist) {
-               _playlist->Modified();
+       boost::shared_ptr<Playlist> pl (playlist());
+
+       if (pl) {
+               pl->Modified();
        }
 
        /* tell everybody else */
@@ -1236,7 +1409,19 @@ AudioRegion::normalize_to (float target_dB)
 }
 
 void
-AudioRegion::envelope_changed (Change ignored)
+AudioRegion::fade_in_changed ()
+{
+       send_change (FadeInChanged);
+}
+
+void
+AudioRegion::fade_out_changed ()
+{
+       send_change (FadeOutChanged);
+}
+
+void
+AudioRegion::envelope_changed ()
 {
        send_change (EnvelopeChanged);
 }
@@ -1245,14 +1430,16 @@ void
 AudioRegion::suspend_fade_in ()
 {
        if (++_fade_in_disabled == 1) {
-               set_fade_in_active (false);
+               if (fade_in_is_default()) {
+                       set_fade_in_active (false);
+               }
        }
 }
 
 void
 AudioRegion::resume_fade_in ()
 {
-       if (_fade_in_disabled && --_fade_in_disabled == 0) {
+       if (--_fade_in_disabled == 0 && _fade_in_disabled) {
                set_fade_in_active (true);
        }
 }
@@ -1261,14 +1448,16 @@ void
 AudioRegion::suspend_fade_out ()
 {
        if (++_fade_out_disabled == 1) {
-               set_fade_out_active (false);
+               if (fade_out_is_default()) {
+                       set_fade_out_active (false);
+               }
        }
 }
 
 void
 AudioRegion::resume_fade_out ()
 {
-       if (_fade_out_disabled && --_fade_out_disabled == 0) {
+       if (--_fade_out_disabled == 0 &&_fade_out_disabled) {
                set_fade_out_active (true);
        }
 }
@@ -1289,12 +1478,164 @@ AudioRegion::speed_mismatch (float sr) const
 void
 AudioRegion::source_offset_changed ()
 {
-       if (boost::dynamic_pointer_cast<DestructiveFileSource>(sources.front())) {
+       boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(sources.front());
+
+       if (afs && afs->destructive()) {
                // set_start (source()->natural_position(), this);
                set_position (source()->natural_position(), this);
        } 
 }
 
+void
+AudioRegion::set_playlist (boost::weak_ptr<Playlist> wpl)
+{
+       boost::shared_ptr<Playlist> old_playlist = (_playlist.lock());
+
+       boost::shared_ptr<Playlist> pl (wpl.lock());
+
+       if (old_playlist == pl) {
+               return;
+       }
+
+       Region::set_playlist (wpl);
+
+       if (pl) {
+               if (old_playlist) {
+                       for (SourceList::const_iterator i = sources.begin(); i != sources.end(); ++i) {
+                               (*i)->remove_playlist (_playlist);      
+                               (*i)->add_playlist (pl);
+                       }
+               } else {
+                       for (SourceList::const_iterator i = sources.begin(); i != sources.end(); ++i) {
+                               (*i)->add_playlist (pl);
+                       }
+               }
+       } else {
+               if (old_playlist) {
+                       for (SourceList::const_iterator i = sources.begin(); i != sources.end(); ++i) {
+                               (*i)->remove_playlist (old_playlist);
+                       }
+               }
+       }
+}
+
+int
+AudioRegion::get_transients (AnalysisFeatureList& results, bool force_new)
+{
+       boost::shared_ptr<Playlist> pl = playlist();
+
+       if (!pl) {
+               return -1;
+       }
+
+       if (valid_transients && !force_new) {
+               results = _transients;
+               return 0;
+       }
+
+       SourceList::iterator s;
+       
+       for (s = sources.begin() ; s != sources.end(); ++s) {
+               if (!(*s)->has_been_analysed()) {
+                       cerr << "For " << name() << " source " << (*s)->name() << " has not been analyzed\n";
+                       break;
+               }
+       }
+       
+       if (s == sources.end()) {
+               /* all sources are analyzed, merge data from each one */
+
+               for (s = sources.begin() ; s != sources.end(); ++s) {
+
+                       /* find the set of transients within the bounds of this region */
+
+                       AnalysisFeatureList::iterator low = lower_bound ((*s)->transients.begin(),
+                                                                        (*s)->transients.end(),
+                                                                        _start);
+
+                       AnalysisFeatureList::iterator high = upper_bound ((*s)->transients.begin(),
+                                                                         (*s)->transients.end(),
+                                                                         _start + _length);
+                                                                        
+                       /* and add them */
+
+                       results.insert (results.end(), low, high);
+               }
+
+               TransientDetector::cleanup_transients (results, pl->session().frame_rate(), 3.0);
+               
+               /* translate all transients to current position */
+
+               for (AnalysisFeatureList::iterator x = results.begin(); x != results.end(); ++x) {
+                       (*x) -= _start;
+                       (*x) += _position;
+               }
+
+               _transients = results;
+               valid_transients = true;
+
+               return 0;
+       }
+
+       cerr << "startup analysis of " << _name << endl;
+
+       TransientDetector t (pl->session().frame_rate());
+       bool existing_results = !results.empty();
+
+       _transients.clear ();
+       valid_transients = false;
+
+       for (uint32_t i = 0; i < n_channels(); ++i) {
+
+               AnalysisFeatureList these_results;
+
+               t.reset ();
+
+               cerr << "working on channel " << i << endl;
+
+               if (t.run ("", this, i, these_results)) {
+                       return -1;
+               }
+
+               cerr << "done\n";
+
+               /* translate all transients to give absolute position */
+               
+               for (AnalysisFeatureList::iterator i = these_results.begin(); i != these_results.end(); ++i) {
+                       (*i) += _position;
+               }
+
+               /* merge */
+               
+               _transients.insert (_transients.end(), these_results.begin(), these_results.end());
+       }
+       
+       if (!results.empty()) {
+               if (existing_results) {
+                       
+                       /* merge our transients into the existing ones, then clean up
+                          those.
+                       */
+
+                       results.insert (results.end(), _transients.begin(), _transients.end());
+                       TransientDetector::cleanup_transients (results, pl->session().frame_rate(), 3.0);
+               }
+
+               /* make sure ours are clean too */
+
+               TransientDetector::cleanup_transients (_transients, pl->session().frame_rate(), 3.0);
+
+       } else {
+
+               TransientDetector::cleanup_transients (_transients, pl->session().frame_rate(), 3.0);
+               results = _transients;
+       }
+
+       valid_transients = true;
+
+       return 0;
+}
+
 extern "C" {
 
        int region_read_peaks_from_c (void *arg, uint32_t npeaks, uint32_t start, uint32_t cnt, intptr_t data, uint32_t n_chan, double samples_per_unit)