It's now safe to get _video_container_size without the lock.

diff --git a/src/lib/player.cc b/src/lib/player.cc
index 13cd8f11badcb35102f33af52249d3abdbcdc79c..b3f11c8000a62c086fa686d84868f5822b0ef7d1 100644 (file)
--- a/src/lib/player.cc
+++ b/src/lib/player.cc
@@ -307,10 +307,9 @@ Player::playlist_content_change (ChangeType type, int property, bool frequent)
 {
        if (property == VideoContentProperty::CROP) {
                if (type == ChangeType::DONE) {
-                       auto const vcs = video_container_size();
                        boost::mutex::scoped_lock lm (_mutex);
                        for (auto const& i: _delay) {
-                               i.first->reset_metadata (_film, vcs);
+                               i.first->reset_metadata(_film, _video_container_size);
                        }
                }
        } else {