D20029: Capture counting corrected

[Wolfgang Reissenberger]

wreissenberger added a comment.


  > That's a good idea, but weeell I have two disagreements : first this is integer calculation and you probably need to reorder your operators, and second, if I understand correctly, you are considering the amount of captures to get equiprobable scattering over all sequence jobs.
  >  I'll nonetheless test this asap.
  
  Could you please be more specific? To be honest, I do not understand what you mean.

INLINE COMMENTS

> TallFurryMan wrote in scheduler.cpp:4601
> I believe that if you arrive here with currentJob equal to nullptr, there are precautions to take earlier than line 4676?

`currentJob == nullptr` might be result of `evaluateJobs()` - so I do not see how to handle this before line 4676.

REPOSITORY
  R321 KStars

REVISION DETAIL
  https://phabricator.kde.org/D20029

To: wreissenberger, mutlaqja, TallFurryMan
Cc: kde-edu, narvaez, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20190407/94f74faf/attachment.html>