diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-12-20 21:49:23 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-12-20 21:49:23 +0100 |
| commit | 3394e86ea65bd9210e148443ac46e95903ad979e (patch) | |
| tree | 2ce373f064061f24263307944bdac7ffa39370f5 | |
| parent | afa2982b4b62dfb7fb5612b99e1c33f453866255 (diff) | |
Remove "computing digest" sub-job.
It doesn't seem very helpful, and having a sub-job messes up progress
reporting when examining lots of files (each of which doesn't take very
long) as in the UI we just end up seeing the 0% of each sub-job one
after the other.
When we have a lot of sub jobs we really want to just report overall
progress, I think, especially if the sub jobs will last roughly the same
time.
| -rw-r--r-- | src/lib/content.cc | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/src/lib/content.cc b/src/lib/content.cc index 76b8eb666..8f486380a 100644 --- a/src/lib/content.cc +++ b/src/lib/content.cc @@ -167,12 +167,8 @@ Content::calculate_digest() const void -Content::examine(shared_ptr<const Film>, shared_ptr<Job> job, bool) +Content::examine(shared_ptr<const Film>, shared_ptr<Job>, bool) { - if (job) { - job->sub(_("Computing digest")); - } - auto const d = calculate_digest(); boost::mutex::scoped_lock lm(_mutex); |