Locking tweak.
authorCarl Hetherington <cth@carlh.net>
Tue, 6 Nov 2018 22:11:25 +0000 (22:11 +0000)
committerCarl Hetherington <cth@carlh.net>
Tue, 6 Nov 2018 22:11:25 +0000 (22:11 +0000)
src/lib/j2k_image_proxy.cc

index 1422b69..6989439 100644 (file)
@@ -171,7 +171,11 @@ J2KImageProxy::prepare (optional<dcp::Size> target_size) const
 pair<shared_ptr<Image>, int>
 J2KImageProxy::image (optional<dcp::NoteHandler>, optional<dcp::Size> target_size) const
 {
-       return make_pair (_image, prepare(target_size));
+       int const r = prepare (target_size);
+       /* I think this is safe without a lock on mutex.  _image is guaranteed to be
+          set up when prepare() has happened.
+       */
+       return make_pair (_image, r);
 }
 
 void