<table><tr><td style="">wreissenberger requested changes to this revision.<br />wreissenberger added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14942">View Revision</a></tr></table><br /><div><div><p>I did some tests, looks good. There is one test that already failed in <a href="https://phabricator.kde.org/D14684" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: line-through;">D14684</a> and is still present. It might be exotic, but it leads to continuous looping.</p>

<p>The setup is the following: create a schedule with two jobs for the same target with the same sequence job (say 1xLum for example). The first job starts and captures the first frame.</p>

<p>And here is where the problem occurs: now calling <em>Scheduler::updateCompletedJobsCount()</em> detects the one frame, but when evaluating the second job (which did not run yet), the value in the captured frames map is overwritten. This leads to an endless loop of the first job.</p>

<p>Here is a test case that demonstrates the problem: <a href="https://phabricator.kde.org/F6222844" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">F6222844: testcases.tar.gz</a></p>

<p>Copy the files into /tmp. The frames are stored in /var/tmp/kstars_tests</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R321 KStars</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14942">https://phabricator.kde.org/D14942</a></div></div><br /><div><strong>To: </strong>TallFurryMan, mutlaqja, wreissenberger<br /><strong>Cc: </strong>kde-edu, narvaez, apol<br /></div>