D14928: Restarting looping schedule continues capturing

[Eric Dejouhanet]

TallFurryMan requested changes to this revision.
TallFurryMan added inline comments.
This revision now requires changes to proceed.

INLINE COMMENTS

> scheduler.cpp:4340
>  
> -    SchedulerJob::CapturedFramesMap fMap = currentJob->getCapturedFramesMap();
> +    if (currentJob->getCompletionCondition() != SchedulerJob::FINISH_LOOP) {
> +        // In case of repeating until terminated, we leave the captured frames map empty

I propose you use a switch case so that we don't forget the special situation of FINISH_LOOP. I think we should also do the same for FINISH_AT, as there are no requirements on captures there neither.

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

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/20180819/532c3eca/attachment.html>