D22446: Adding error handling strategy control to Scheduler

[Wolfgang Reissenberger]

wreissenberger marked 9 inline comments as done.
wreissenberger added a comment.


  Serialization changed as suggested, log messages corrected. Now we need to agree how to proceed with the problem, that restarting aborted jobs immediately conflicts with the idea of having multi-day schedules.
  
  @mutlaqja: what do you think?

INLINE COMMENTS

> TallFurryMan wrote in scheduler.cpp:3912
> Enum-to-int conversions are dangerous, we should review this method at some point...

Agreed.

> TallFurryMan wrote in scheduler.cpp:3922
> The default value "true" is a bit lost in the code here, can we do better?
> Line 3942 for instance, startup procedure block, has the boolean flag present or not in the "StartupProcedure" element.
> The default is clear at lines 3929. Wouldn't that be better for maintenance?

Good point, changed.

> TallFurryMan wrote in scheduler.cpp:4491
> Comment on the reason of the setCurrentJob move?

There's no reason for - reverting it.

REPOSITORY
  R321 KStars

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

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/20190714/680cd4ab/attachment-0001.html>