D16449: [scripting] Introduce redirect function

davidedmundson added inline comments.

INLINE COMMENTS

> zzag wrote in anidata_p.h:101
> So, instead of having two booleans, we would have a single enum, right?
> Yeah, it makes sense, but this would be unrelated to this change.

Instead of

bool keepAtTarget;
AnimationEffect::TerminationPolicy  terminationPolicy;

We have either:

AnimationEffect::TerminationPolicy targetTerminationPolicy;
AnimationEffect::TerminationPolicy sourceTerminationPolicy;

OR

AnimationEffect::TerminationPolicy terminationPolicy;

which has flags for {DontTerminate, TerminateAtSource, TerminateAtTarget}

That then becomes related to this patch as we'd have to name things appropriately

REPOSITORY
  R108 KWin

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

To: zzag, #kwin
Cc: davidedmundson, abetts, kwin, mkulinski, ragreen, jackyalcine, Pitel, iodelay, bwowk, ZrenBot, ngraham, lesliezhai, ali-mohamed, hardening, jensreuterberg, sebas, apol, mart