diff options
| author | Carl Hetherington <cth@carlh.net> | 2020-05-20 01:51:25 +0200 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2020-05-20 21:56:28 +0200 |
| commit | 93f6b7f36dba0eed49936dcd014a9cf5941a66ac (patch) | |
| tree | 33f1b92003305fb3c9fdc7cd4fc4a05ff8ce6b03 | |
| parent | 20c70c93a38551842721ddee2e08c7b33a7bb924 (diff) | |
Difficult to see how any unprotected write with multiple readers
could be safe.
| -rw-r--r-- | src/lib/writer.cc | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/lib/writer.cc b/src/lib/writer.cc index d346c4a4f..699f220c4 100644 --- a/src/lib/writer.cc +++ b/src/lib/writer.cc @@ -777,10 +777,9 @@ Writer::video_reel (int frame) const void Writer::set_digest_progress (Job* job, float progress) { - /* I believe this is thread-safe */ - _digest_progresses[boost::this_thread::get_id()] = progress; - boost::mutex::scoped_lock lm (_digest_progresses_mutex); + + _digest_progresses[boost::this_thread::get_id()] = progress; float min_progress = FLT_MAX; for (map<boost::thread::id, float>::const_iterator i = _digest_progresses.begin(); i != _digest_progresses.end(); ++i) { min_progress = min (min_progress, i->second); |