<@U1AFFRV1Q> commented on <@U0ZFBBUBU>’s file <htt...

# coroutines

[user]

@kingsley commented on @groostav’s file https://kotlinlang.slack.com/files/U0ZFBBUBU/FA3C0A8CS/File_watching_implementation.kt: Hey @groostav
This is really cool.

While I’m trying not to think so much about the file watcher API itself, I have a few comments that would be interesting to take a look at:

I was thinking it might be nice to take advantage of channel’s map/flatMap operators to transform these producers, since this is basically what is happening AFAICS. What do you think?

Also, instead of explicitly specifying worker for all those subsequent channels, I’d use coroutineContext, since they were all launched within the worker anyway

Finally, I’d replace the while(true) with while(isActive)

File_watching_implementation.kt