Would you use

isActive

instead of

true

as an infinite

while

loop condition in a coroutine if the loop calls suspending functions (

delay

, among others)? I don’t think it changes anything, but maybe it makes things more clear to the reader? Thoughts?

I use a function named

repeatWhileActive { }

for these purposes: https://github.com/LouisCAD/Splitties/tree/main/modules/coroutines#cancellable-infinite-loops

Isn’t cancellation checked on each suspending call already?

Each cancellable suspending call, yes, but I prefer to be safe in case after edits, they all go through a fast path that doesn't check for cancellation, or in case all exceptions are eaten. Plus I think

repeatWhileActive

is more descriptive than

while (true)

plus implicit exit/cancellation conditions.