diff options
| author | Carl Hetherington <cth@carlh.net> | 2023-08-08 23:53:52 +0200 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2023-08-09 10:18:46 +0200 |
| commit | b4489de4679a41c96fe0dd5449a8721cdb2294b2 (patch) | |
| tree | d6f463c89da0fa335d6e74b37a99d51d20a1cc51 /src/lib/playlist.cc | |
| parent | fc329637dd1e35e8cde6d82f3131050b578b07cf (diff) | |
Cleanup: tidy Playlist::move_later().
Diffstat (limited to 'src/lib/playlist.cc')
| -rw-r--r-- | src/lib/playlist.cc | 18 |
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)); } |
