Sequence content after trim changes (#1327). This gives me a slight
authorCarl Hetherington <cth@carlh.net>
Mon, 9 Jul 2018 14:49:32 +0000 (15:49 +0100)
committerCarl Hetherington <cth@carlh.net>
Mon, 9 Jul 2018 14:49:32 +0000 (15:49 +0100)
sense of unease but let's see how it goes.

ChangeLog
src/lib/playlist.cc

index e511685..8804ff1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,7 @@
 2018-07-09  Carl Hetherington  <cth@carlh.net>
 
+       * Maintain content sequence after trim (#1327).
+
        * Keep video content position and trim on frame boundaries
        even if the content's frame rate is forced to a new value (#1335).
 
index d2ac95c..3376a55 100644 (file)
@@ -67,7 +67,12 @@ Playlist::~Playlist ()
 void
 Playlist::content_changed (weak_ptr<Content> content, int property, bool frequent)
 {
-       if (property == ContentProperty::LENGTH || property == VideoContentProperty::FRAME_TYPE) {
+       if (
+               property == ContentProperty::TRIM_START ||
+               property == ContentProperty::TRIM_END ||
+               property == ContentProperty::LENGTH ||
+               property == VideoContentProperty::FRAME_TYPE
+               ) {
                /* Don't respond to position changes here, as:
                   - sequencing after earlier/later changes is handled by move_earlier/move_later
                   - any other position changes will be timeline drags which should not result in content