Fix warnings.
[ardour.git] / libs / ardour / region.cc
index 9a54139d091ddad2351c6db2dc374da45039661f..7fb94bb1778adebf20e9cbfd93bf614175ed090b 100644 (file)
@@ -79,51 +79,51 @@ void
 Region::make_property_quarks ()
 {
        Properties::muted.property_id = g_quark_from_static_string (X_("muted"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for muted = %1\n",      Properties::muted.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for muted = %1\n",       Properties::muted.property_id));
        Properties::opaque.property_id = g_quark_from_static_string (X_("opaque"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for opaque = %1\n",     Properties::opaque.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for opaque = %1\n",      Properties::opaque.property_id));
        Properties::locked.property_id = g_quark_from_static_string (X_("locked"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for locked = %1\n",     Properties::locked.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for locked = %1\n",      Properties::locked.property_id));
        Properties::automatic.property_id = g_quark_from_static_string (X_("automatic"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for automatic = %1\n",  Properties::automatic.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for automatic = %1\n",   Properties::automatic.property_id));
        Properties::whole_file.property_id = g_quark_from_static_string (X_("whole-file"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for whole-file = %1\n",         Properties::whole_file.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for whole-file = %1\n",  Properties::whole_file.property_id));
        Properties::import.property_id = g_quark_from_static_string (X_("import"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for import = %1\n",     Properties::import.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for import = %1\n",      Properties::import.property_id));
        Properties::external.property_id = g_quark_from_static_string (X_("external"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for external = %1\n",   Properties::external.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for external = %1\n",    Properties::external.property_id));
        Properties::sync_marked.property_id = g_quark_from_static_string (X_("sync-marked"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for sync-marked = %1\n",        Properties::sync_marked.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for sync-marked = %1\n",         Properties::sync_marked.property_id));
        Properties::left_of_split.property_id = g_quark_from_static_string (X_("left-of-split"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for left-of-split = %1\n",      Properties::left_of_split.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for left-of-split = %1\n",       Properties::left_of_split.property_id));
        Properties::right_of_split.property_id = g_quark_from_static_string (X_("right-of-split"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for right-of-split = %1\n",     Properties::right_of_split.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for right-of-split = %1\n",      Properties::right_of_split.property_id));
        Properties::hidden.property_id = g_quark_from_static_string (X_("hidden"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for hidden = %1\n",     Properties::hidden.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for hidden = %1\n",      Properties::hidden.property_id));
        Properties::position_locked.property_id = g_quark_from_static_string (X_("position-locked"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for position-locked = %1\n",    Properties::position_locked.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for position-locked = %1\n",     Properties::position_locked.property_id));
        Properties::valid_transients.property_id = g_quark_from_static_string (X_("valid-transients"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for valid-transients = %1\n",   Properties::valid_transients.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for valid-transients = %1\n",    Properties::valid_transients.property_id));
        Properties::start.property_id = g_quark_from_static_string (X_("start"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for start = %1\n",      Properties::start.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for start = %1\n",       Properties::start.property_id));
        Properties::length.property_id = g_quark_from_static_string (X_("length"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for length = %1\n",     Properties::length.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for length = %1\n",      Properties::length.property_id));
        Properties::position.property_id = g_quark_from_static_string (X_("position"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for position = %1\n",   Properties::position.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for position = %1\n",    Properties::position.property_id));
        Properties::sync_position.property_id = g_quark_from_static_string (X_("sync-position"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for sync-position = %1\n",      Properties::sync_position.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for sync-position = %1\n",       Properties::sync_position.property_id));
        Properties::layer.property_id = g_quark_from_static_string (X_("layer"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for layer = %1\n",      Properties::layer.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for layer = %1\n",       Properties::layer.property_id));
        Properties::ancestral_start.property_id = g_quark_from_static_string (X_("ancestral-start"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for ancestral-start = %1\n",    Properties::ancestral_start.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for ancestral-start = %1\n",     Properties::ancestral_start.property_id));
        Properties::ancestral_length.property_id = g_quark_from_static_string (X_("ancestral-length"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for ancestral-length = %1\n",   Properties::ancestral_length.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for ancestral-length = %1\n",    Properties::ancestral_length.property_id));
        Properties::stretch.property_id = g_quark_from_static_string (X_("stretch"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for stretch = %1\n",    Properties::stretch.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for stretch = %1\n",     Properties::stretch.property_id));
        Properties::shift.property_id = g_quark_from_static_string (X_("shift"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for shift = %1\n",      Properties::shift.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for shift = %1\n",       Properties::shift.property_id));
        Properties::position_lock_style.property_id = g_quark_from_static_string (X_("positional-lock-style"));
-        DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for position_lock_style = %1\n",        Properties::position_lock_style.property_id));
+       DEBUG_TRACE (DEBUG::Properties, string_compose ("quark for position_lock_style = %1\n",         Properties::position_lock_style.property_id));
 }
 
 void
@@ -245,14 +245,8 @@ Region::Region (const SourceList& srcs)
        assert (_type == srcs.front()->type());
 }
 
-/** Create a new Region from part of an existing one, starting at one of two places:
-
-    if \a offset_relative is true, then the start within \a other is given by \a offset
-    (i.e. relative to the start of \a other's sources, the start is \a offset + \a other.start()
-
-    if @param offset_relative is false, then the start within the source is given \a offset.
-*/
-Region::Region (boost::shared_ptr<const Region> other, frameoffset_t offset, bool offset_relative)
+/** Create a new Region from an existing one */
+Region::Region (boost::shared_ptr<const Region> other)
        : SessionObject(other->session(), other->name())
        , _type (other->data_type())
        , REGION_COPY_STATE (other)
@@ -276,70 +270,34 @@ Region::Region (boost::shared_ptr<const Region> other, frameoffset_t offset, boo
 
        use_sources (other->_sources);
 
-       if (!offset_relative) {
+       _position_lock_style = other->_position_lock_style;
+       _first_edit = other->_first_edit;
 
-               /* not sure why we do this, but its a hangover from ardour before
-                  property lists. this would be nice to remove.
-               */
+       _start = 0; // It seems strange _start is not inherited here?
 
-               _position_lock_style = other->_position_lock_style;
-               _first_edit = other->_first_edit;
-
-               if (offset == 0) {
-
-                       _start = 0;
-
-                       /* sync pos is relative to start of file. our start-in-file is now zero,
-                          so set our sync position to whatever the the difference between
-                          _start and _sync_pos was in the other region.
+       /* sync pos is relative to start of file. our start-in-file is now zero,
+          so set our sync position to whatever the the difference between
+          _start and _sync_pos was in the other region.
                           
-                          result is that our new sync pos points to the same point in our source(s)
-                          as the sync in the other region did in its source(s).
+          result is that our new sync pos points to the same point in our source(s)
+          as the sync in the other region did in its source(s).
                           
-                          since we start at zero in our source(s), it is not possible to use a sync point that
-                          is before the start. reset it to _start if that was true in the other region.
-                       */
+          since we start at zero in our source(s), it is not possible to use a sync point that
+          is before the start. reset it to _start if that was true in the other region.
+       */
                        
-                       if (other->sync_marked()) {
-                               if (other->_start < other->_sync_position) {
-                                       /* sync pos was after the start point of the other region */
-                                       _sync_position = other->_sync_position - other->_start;
-                               } else {
-                                       /* sync pos was before the start point of the other region. not possible here. */
-                                       _sync_marked = false;
-                                       _sync_position = _start;
-                               }
-                       } else {
-                               _sync_marked = false;
-                               _sync_position = _start;
-                       }
-               } else {
-                       /* XXX do something else ! */
-                       fatal << string_compose (_("programming error: %1"), X_("Region+offset constructor used with illegal combination of offset+relative"))
-                             << endmsg;
-                       /*NOTREACHED*/
-               }
-
-       } else {
-
-               _start = other->_start + offset;
-               
-               /* if the other region had a distinct sync point
-                  set, then continue to use it as best we can.
-                  otherwise, reset sync point back to start.
-               */
-               
-               if (other->sync_marked()) {
-                       if (other->_sync_position < _start) {
-                               _sync_marked = false;
-                               _sync_position = _start;
-               } else {
-                               _sync_position = other->_sync_position;
-                       }
+       if (other->sync_marked()) {
+               if (other->_start < other->_sync_position) {
+                       /* sync pos was after the start point of the other region */
+                       _sync_position = other->_sync_position - other->_start;
                } else {
+                       /* sync pos was before the start point of the other region. not possible here. */
                        _sync_marked = false;
                        _sync_position = _start;
                }
+       } else {
+               _sync_marked = false;
+               _sync_position = _start;
        }
 
        if (Profile->get_sae()) {
@@ -356,45 +314,78 @@ Region::Region (boost::shared_ptr<const Region> other, frameoffset_t offset, boo
        assert (_type == other->data_type());
 }
 
-/** Create a copy of @param other but with different sources. Used by filters */
-Region::Region (boost::shared_ptr<const Region> other, const SourceList& srcs)
-       : SessionObject (other->session(), other->name())
-       , _type (srcs.front()->type())
+/** Create a new Region from part of an existing one.
+
+    the start within \a other is given by \a offset
+    (i.e. relative to the start of \a other's sources, the start is \a offset + \a other.start()
+*/
+Region::Region (boost::shared_ptr<const Region> other, frameoffset_t offset)
+       : SessionObject(other->session(), other->name())
+       , _type (other->data_type())
        , REGION_COPY_STATE (other)
        , _last_length (other->_last_length)
-       , _last_position (other->_last_position)
-       , _first_edit (EditChangesID)
-       , _read_data_count (0)
-       , _last_layer_op (other->_last_layer_op)
+       , _last_position(other->_last_position) \
+       , _first_edit (EditChangesNothing)
+       , _read_data_count(0)
+       , _last_layer_op (0)
        , _pending_explicit_relayer (false)
+
 {
        register_properties ();
 
+       /* override state that may have been incorrectly inherited from the other region
+        */
+
+       _position = 0;
        _locked = false;
-       _position_locked = false;
+       _whole_file = false;
+       _hidden = false;
 
-       other->_first_edit = EditChangesName;
+       use_sources (other->_sources);
 
-       if (other->_extra_xml) {
-               _extra_xml = new XMLNode (*other->_extra_xml);
+       _start = other->_start + offset;
+               
+       /* if the other region had a distinct sync point
+          set, then continue to use it as best we can.
+          otherwise, reset sync point back to start.
+       */
+               
+       if (other->sync_marked()) {
+               if (other->_sync_position < _start) {
+                       _sync_marked = false;
+                       _sync_position = _start;
+               } else {
+                       _sync_position = other->_sync_position;
+               }
        } else {
-               _extra_xml = 0;
+               _sync_marked = false;
+               _sync_position = _start;
        }
 
-       use_sources (srcs);
-       assert(_sources.size() > 0);
+       if (Profile->get_sae()) {
+               /* reset sync point to start if its ended up
+                  outside region bounds.
+               */
+
+               if (_sync_position < _start || _sync_position >= _start + _length) {
+                       _sync_marked = false;
+                       _sync_position = _start;
+               }
+       }
+
+       assert (_type == other->data_type());
 }
 
-/** Simple "copy" constructor */
-Region::Region (boost::shared_ptr<const Region> other)
-       : SessionObject(other->session(), other->name())
-       , _type(other->data_type())
+/** Create a copy of @param other but with different sources. Used by filters */
+Region::Region (boost::shared_ptr<const Region> other, const SourceList& srcs)
+       : SessionObject (other->session(), other->name())
+       , _type (srcs.front()->type())
        , REGION_COPY_STATE (other)
        , _last_length (other->_last_length)
        , _last_position (other->_last_position)
        , _first_edit (EditChangesID)
-       , _read_data_count(0)
-       , _last_layer_op(other->_last_layer_op)
+       , _read_data_count (0)
+       , _last_layer_op (other->_last_layer_op)
        , _pending_explicit_relayer (false)
 {
        register_properties ();
@@ -410,7 +401,7 @@ Region::Region (boost::shared_ptr<const Region> other)
                _extra_xml = 0;
        }
 
-       use_sources (other->_sources);
+       use_sources (srcs);
        assert(_sources.size() > 0);
 }
 
@@ -432,6 +423,7 @@ Region::set_name (const std::string& str)
        if (_name != str) {
                SessionObject::set_name(str); // EMIT SIGNAL NameChanged()
                assert(_name == str);
+
                send_change (Properties::name);
        }
 
@@ -452,7 +444,7 @@ Region::set_length (framecnt_t len, void */*src*/)
                   length impossible.
                */
 
-               if (max_frames - len < _position) {
+               if (max_framepos - len < _position) {
                        return;
                }
 
@@ -462,7 +454,7 @@ Region::set_length (framecnt_t len, void */*src*/)
 
 
                _last_length = _length;
-               _length = len;
+               set_length_internal (len);
                _whole_file = false;
                first_edit ();
                maybe_uncopy ();
@@ -476,6 +468,12 @@ Region::set_length (framecnt_t len, void */*src*/)
        }
 }
 
+void
+Region::set_length_internal (framecnt_t len)
+{
+       _length = len;
+}
+
 void
 Region::maybe_uncopy ()
 {
@@ -493,6 +491,7 @@ Region::first_edit ()
                _first_edit = EditChangesNothing;
 
                send_change (Properties::name);
+
                RegionFactory::CheckNewRegion (shared_from_this());
        }
 }
@@ -562,6 +561,7 @@ Region::set_position_lock_style (PositionLockStyle ps)
                }
 
                send_change (Properties::position_lock_style);
+               
        }
 }
 
@@ -577,6 +577,11 @@ Region::update_position_after_tempo_map_change ()
        TempoMap& map (_session.tempo_map());
        framepos_t pos = map.frame_time (_bbt_time);
        set_position_internal (pos, false);
+
+       /* do this even if the position is the same. this helps out
+          a GUI that has moved its representation already.
+       */
+       send_change (Properties::position);
 }
 
 void
@@ -587,6 +592,12 @@ Region::set_position (framepos_t pos, void* /*src*/)
        }
 
        set_position_internal (pos, true);
+
+       /* do this even if the position is the same. this helps out
+          a GUI that has moved its representation already.
+       */
+       send_change (Properties::position);
+       
 }
 
 void
@@ -602,9 +613,9 @@ Region::set_position_internal (framepos_t pos, bool allow_bbt_recompute)
                   XXX is this the right thing to do?
                */
 
-               if (max_frames - _length < _position) {
+               if (max_framepos - _length < _position) {
                        _last_length = _length;
-                       _length = max_frames - _position;
+                       _length = max_framepos - _position;
                }
 
                if (allow_bbt_recompute) {
@@ -613,11 +624,6 @@ Region::set_position_internal (framepos_t pos, bool allow_bbt_recompute)
 
                //invalidate_transients ();
        }
-
-       /* do this even if the position is the same. this helps out
-          a GUI that has moved its representation already.
-       */
-       send_change (Properties::position);
 }
 
 void
@@ -628,8 +634,7 @@ Region::set_position_on_top (framepos_t pos, void* /*src*/)
        }
 
        if (_position != pos) {
-               _last_position = _position;
-               _position = pos;
+               set_position_internal (pos, true);
        }
 
        boost::shared_ptr<Playlist> pl (playlist());
@@ -641,7 +646,6 @@ Region::set_position_on_top (framepos_t pos, void* /*src*/)
        /* do this even if the position is the same. this helps out
           a GUI that has moved its representation already.
        */
-
        send_change (Properties::position);
 }
 
@@ -664,22 +668,24 @@ Region::nudge_position (frameoffset_t n, void* /*src*/)
                return;
        }
 
-       _last_position = _position;
+       framepos_t new_position = _position;
 
        if (n > 0) {
-               if (_position > max_frames - n) {
-                       _position = max_frames;
+               if (_position > max_framepos - n) {
+                       new_position = max_framepos;
                } else {
-                       _position += n;
+                       new_position += n;
                }
        } else {
                if (_position < -n) {
-                       _position = 0;
+                       new_position = 0;
                } else {
-                       _position += n;
+                       new_position += n;
                }
        }
 
+       set_position_internal (new_position, true);
+
        send_change (Properties::position);
 }
 
@@ -725,18 +731,12 @@ Region::trim_start (framepos_t new_position, void */*src*/)
                return;
        }
        framepos_t new_start;
-       frameoffset_t start_shift;
-
-       if (new_position > _position) {
-               start_shift = new_position - _position;
-       } else {
-               start_shift = -(_position - new_position);
-       }
+       frameoffset_t const start_shift = new_position - _position;
 
        if (start_shift > 0) {
 
-               if (_start > max_frames - start_shift) {
-                       new_start = max_frames;
+               if (_start > max_framepos - start_shift) {
+                       new_start = max_framepos;
                } else {
                        new_start = _start + start_shift;
                }
@@ -752,6 +752,7 @@ Region::trim_start (framepos_t new_position, void */*src*/)
                } else {
                        new_start = _start + start_shift;
                }
+               
        } else {
                return;
        }
@@ -774,26 +775,26 @@ Region::trim_front (framepos_t new_position, void *src)
 }
 
 void
-Region::cut_front (nframes_t new_position, void *src)
+Region::cut_front (framepos_t new_position, void *src)
 {
        modify_front (new_position, true, src);
 }
 
 void
-Region::cut_end (nframes_t new_endpoint, void *src)
+Region::cut_end (framepos_t new_endpoint, void *src)
 {
        modify_end (new_endpoint, true, src);
 }
 
 void
-Region::modify_front (nframes_t new_position, bool reset_fade, void *src)
+Region::modify_front (framepos_t new_position, bool reset_fade, void *src)
 {
        if (locked()) {
                return;
        }
 
-       nframes_t end = last_frame();
-       nframes_t source_zero;
+       framepos_t end = last_frame();
+       framepos_t source_zero;
 
        if (_position > _start) {
                source_zero = _position - _start;
@@ -803,12 +804,13 @@ Region::modify_front (nframes_t new_position, bool reset_fade, void *src)
 
        if (new_position < end) { /* can't trim it zero or negative length */
                
-               nframes_t newlen = 0;
-               nframes64_t delta = 0;
+               framecnt_t newlen = 0;
+               framepos_t delta = 0;
 
-               /* can't trim it back passed where source position zero is located */
-               
-               new_position = max (new_position, source_zero);
+               if (!can_trim_start_before_source_start ()) {
+                       /* can't trim it back past where source position zero is located */
+                       new_position = max (new_position, source_zero);
+               }
                
                if (new_position > _position) {
                        newlen = _length - (new_position - _position);
@@ -821,10 +823,10 @@ Region::modify_front (nframes_t new_position, bool reset_fade, void *src)
                trim_to_internal (new_position, newlen, src);
                
                if (reset_fade) {
-                        _right_of_split = true;
+                       _right_of_split = true;
                }
        
-                if (!property_changes_suspended()) {
+               if (!property_changes_suspended()) {
                        recompute_at_start ();
                }
                
@@ -835,7 +837,7 @@ Region::modify_front (nframes_t new_position, bool reset_fade, void *src)
 }
 
 void
-Region::modify_end (nframes_t new_endpoint, bool reset_fade, void* /*src*/)
+Region::modify_end (framepos_t new_endpoint, bool reset_fade, void* /*src*/)
 {
        if (locked()) {
                return;
@@ -844,7 +846,7 @@ Region::modify_end (nframes_t new_endpoint, bool reset_fade, void* /*src*/)
        if (new_endpoint > _position) {
                trim_to_internal (_position, new_endpoint - _position +1, this);
                if (reset_fade) {
-                        _left_of_split = true;
+                       _left_of_split = true;
                }
                if (!property_changes_suspended()) {
                        recompute_at_end ();
@@ -880,30 +882,25 @@ Region::trim_to (framepos_t position, framecnt_t length, void *src)
 void
 Region::trim_to_internal (framepos_t position, framecnt_t length, void */*src*/)
 {
-       frameoffset_t start_shift;
        framepos_t new_start;
 
        if (locked()) {
                return;
        }
 
-       if (position > _position) {
-               start_shift = position - _position;
-       } else {
-               start_shift = -(_position - position);
-       }
+       frameoffset_t const start_shift = position - _position;
 
        if (start_shift > 0) {
 
-               if (_start > max_frames - start_shift) {
-                       new_start = max_frames;
+               if (_start > max_framepos - start_shift) {
+                       new_start = max_framepos;
                } else {
                        new_start = _start + start_shift;
                }
 
        } else if (start_shift < 0) {
 
-               if (_start < -start_shift) {
+               if (_start < -start_shift && !can_trim_start_before_source_start ()) {
                        new_start = 0;
                } else {
                        new_start = _start + start_shift;
@@ -927,14 +924,14 @@ Region::trim_to_internal (framepos_t position, framecnt_t length, void */*src*/)
                if (!property_changes_suspended()) {
                        _last_length = _length;
                }
-               _length = length;
+               set_length_internal (length);
                what_changed.add (Properties::length);
        }
        if (_position != position) {
                if (!property_changes_suspended()) {
                        _last_position = _position;
                }
-               _position = position;
+               set_position_internal (position, true);
                what_changed.add (Properties::position);
        }
 
@@ -1028,6 +1025,7 @@ Region::set_sync_position (framepos_t absolute_pos)
                if (!property_changes_suspended()) {
                        maybe_uncopy ();
                }
+
                send_change (Properties::sync_position);
        }
 }
@@ -1040,12 +1038,13 @@ Region::clear_sync_position ()
                if (!property_changes_suspended()) {
                        maybe_uncopy ();
                }
+
                send_change (Properties::sync_position);
        }
 }
 
 /* @return the sync point relative the first frame of the region */
-framepos_t
+frameoffset_t
 Region::sync_offset (int& dir) const
 {
        if (sync_marked()) {
@@ -1077,7 +1076,7 @@ Region::adjust_to_sync (framepos_t pos) const
                        pos = 0;
                }
        } else {
-               if (max_frames - pos > offset) {
+               if (max_framepos - pos > offset) {
                        pos += offset;
                }
        }
@@ -1285,7 +1284,7 @@ Region::_set_state (const XMLNode& node, int /*version*/, PropertyChange& what_c
 void
 Region::suspend_property_changes ()
 {
-        Stateful::suspend_property_changes ();
+       Stateful::suspend_property_changes ();
        _last_length = _length;
        _last_position = _position;
 }
@@ -1308,9 +1307,9 @@ Region::send_change (const PropertyChange& what_changed)
                return;
        }
 
-        Stateful::send_change (what_changed);
+       Stateful::send_change (what_changed);
 
-       if (!_no_property_changes) {
+       if (!Stateful::frozen()) {
                
                /* Try and send a shared_pointer unless this is part of the constructor.
                   If so, do nothing.
@@ -1361,7 +1360,7 @@ Region::region_list_equivalent (boost::shared_ptr<const Region> other) const
 void
 Region::source_deleted (boost::weak_ptr<Source>)
 {
-        drop_sources ();
+       drop_sources ();
 
        if (!_session.deletion_in_progress()) {
                /* this is a very special case: at least one of the region's
@@ -1392,16 +1391,16 @@ Region::master_source_names ()
 void
 Region::set_master_sources (const SourceList& srcs)
 {
-        for (SourceList::const_iterator i = _master_sources.begin (); i != _master_sources.end(); ++i) {
-                (*i)->dec_use_count ();
-        }
+       for (SourceList::const_iterator i = _master_sources.begin (); i != _master_sources.end(); ++i) {
+               (*i)->dec_use_count ();
+       }
 
        _master_sources = srcs;
        assert (_sources.size() == _master_sources.size());
 
-        for (SourceList::const_iterator i = _master_sources.begin (); i != _master_sources.end(); ++i) {
-                (*i)->inc_use_count ();
-        }
+       for (SourceList::const_iterator i = _master_sources.begin (); i != _master_sources.end(); ++i) {
+               (*i)->inc_use_count ();
+       }
 }
 
 bool
@@ -1409,11 +1408,11 @@ Region::source_equivalent (boost::shared_ptr<const Region> other) const
 {
        if (!other)
                return false;
-        
-        if ((_sources.size() != other->_sources.size()) ||
-            (_master_sources.size() != other->_master_sources.size())) {
-                return false;
-        }
+       
+       if ((_sources.size() != other->_sources.size()) ||
+           (_master_sources.size() != other->_master_sources.size())) {
+               return false;
+       }
 
        SourceList::const_iterator i;
        SourceList::const_iterator io;
@@ -1433,6 +1432,27 @@ Region::source_equivalent (boost::shared_ptr<const Region> other) const
        return true;
 }
 
+std::string
+Region::source_string () const
+{
+       //string res = itos(_sources.size());
+
+       stringstream res;
+       res << _sources.size() << ":";
+       
+       SourceList::const_iterator i;
+
+       for (i = _sources.begin(); i != _sources.end(); ++i) {
+               res << (*i)->id() << ":";
+       }
+
+       for (i = _master_sources.begin(); i != _master_sources.end(); ++i) {
+               res << (*i)->id() << ":";
+       }
+
+       return res.str();
+}
+
 bool
 Region::uses_source (boost::shared_ptr<const Source> source) const
 {
@@ -1444,11 +1464,11 @@ Region::uses_source (boost::shared_ptr<const Source> source) const
        return false;
 }
 
-sframes_t
+framecnt_t
 Region::source_length(uint32_t n) const
 {
-        assert (n < _sources.size());
-       return _sources[n]->length(_position - _start);
+       assert (n < _sources.size());
+       return _sources[n]->length (_position - _start);
 }
 
 bool
@@ -1535,9 +1555,9 @@ Region::get_parent() const
 }
 
 int
-Region::apply (Filter& filter)
+Region::apply (Filter& filter, Progress* progress)
 {
-       return filter.run (shared_from_this());
+       return filter.run (shared_from_this(), progress);
 }
 
 
@@ -1553,17 +1573,17 @@ Region::invalidate_transients ()
 void
 Region::drop_sources ()
 {
-        for (SourceList::const_iterator i = _sources.begin (); i != _sources.end(); ++i) {
-                (*i)->dec_use_count ();
-        }
+       for (SourceList::const_iterator i = _sources.begin (); i != _sources.end(); ++i) {
+               (*i)->dec_use_count ();
+       }
 
        _sources.clear ();
 
-        for (SourceList::const_iterator i = _master_sources.begin (); i != _master_sources.end(); ++i) {
-                (*i)->dec_use_count ();
-        }
+       for (SourceList::const_iterator i = _master_sources.begin (); i != _master_sources.end(); ++i) {
+               (*i)->dec_use_count ();
+       }
 
-        _master_sources.clear ();
+       _master_sources.clear ();
 }
 
 void
@@ -1574,16 +1594,44 @@ Region::use_sources (SourceList const & s)
        for (SourceList::const_iterator i = s.begin (); i != s.end(); ++i) {
 
                _sources.push_back (*i);
-                (*i)->inc_use_count ();
+               (*i)->inc_use_count ();
                _master_sources.push_back (*i);
-                (*i)->inc_use_count ();
+               (*i)->inc_use_count ();
 
-                /* connect only once to DropReferences, even if sources are replicated
-                 */
+               /* connect only once to DropReferences, even if sources are replicated
+                */
 
                if (unique_srcs.find (*i) == unique_srcs.end ()) {
                        unique_srcs.insert (*i);
-                        (*i)->DropReferences.connect_same_thread (*this, boost::bind (&Region::source_deleted, this, boost::weak_ptr<Source>(*i)));
-                }
+                       (*i)->DropReferences.connect_same_thread (*this, boost::bind (&Region::source_deleted, this, boost::weak_ptr<Source>(*i)));
+               }
+       }
+}
+
+Trimmable::CanTrim
+Region::can_trim () const
+{
+       CanTrim ct = CanTrim (0);
+
+       if (locked()) {
+               return ct;
+       }
+
+       /* if not locked, we can always move the front later, and the end earlier 
+        */
+
+       ct = CanTrim (ct | FrontTrimLater | EndTrimEarlier);
+
+       if (start() != 0 || can_trim_start_before_source_start ()) {
+               ct = CanTrim (ct | FrontTrimEarlier);
        }
+
+       if (!_sources.empty()) {
+               if ((start() + length()) < _sources.front()->length (0)) {
+                       ct = CanTrim (ct | EndTrimLater);
+               }
+       }
+
+       return ct;
 }
+