Add a comment. v2.15.23
authorCarl Hetherington <cth@carlh.net>
Mon, 14 Oct 2019 13:59:26 +0000 (15:59 +0200)
committerCarl Hetherington <cth@carlh.net>
Mon, 14 Oct 2019 15:34:40 +0000 (17:34 +0200)
src/lib/check_content_change_job.cc

index 2bc562f57730339bce035be957368f7a29e8d8b2..e1226e79c6af3ae7c21cc74115875f9711a1111d 100644 (file)
@@ -103,6 +103,11 @@ CheckContentChangeJob::run ()
                JobManager::instance()->add (_following);
        }
 
+       /* Only set this job as finished once we have added the following job, otherwise I think
+          it's possible that the tests will sporadically fail if they check for all jobs being
+          complete in the gap between this one finishing and _following being added.
+       */
+
        set_progress (1);
        set_state (FINISHED_OK);
 }