summaryrefslogtreecommitdiff
path: root/src/lib/playlist.cc
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2023-08-08 23:53:52 +0200
committerCarl Hetherington <cth@carlh.net>2023-08-09 10:18:46 +0200
commitb4489de4679a41c96fe0dd5449a8721cdb2294b2 (patch)
treed6f463c89da0fa335d6e74b37a99d51d20a1cc51 /src/lib/playlist.cc
parentfc329637dd1e35e8cde6d82f3131050b578b07cf (diff)
Cleanup: tidy Playlist::move_later().
Diffstat (limited to 'src/lib/playlist.cc')
-rw-r--r--src/lib/playlist.cc18
1 files changed, 6 insertions, 12 deletions
diff --git a/src/lib/playlist.cc b/src/lib/playlist.cc
index 80037b575..2f261d8a0 100644
--- a/src/lib/playlist.cc
+++ b/src/lib/playlist.cc
@@ -629,24 +629,18 @@ void
Playlist::move_later (shared_ptr<const Film> film, shared_ptr<Content> c)
{
auto cont = content ();
- auto i = cont.begin();
- while (i != cont.end() && *i != c) {
- ++i;
- }
- DCPOMATIC_ASSERT (i != cont.end());
-
- ContentList::iterator next = i;
- ++next;
+ auto iter = std::find(cont.begin(), cont.end(), c);
+ DCPOMATIC_ASSERT(iter != cont.end());
+ ContentList::iterator next = std::next(iter);
if (next == cont.end()) {
+ /* This content is already at the end */
return;
}
- auto next_c = *next;
-
- next_c->set_position (film, c->position());
- c->set_position (film, c->position() + next_c->length_after_trim(film));
+ (*next)->set_position(film, c->position());
+ c->set_position(film, c->position() + (*next)->length_after_trim(film));
}