search

Catfriend1 / syncthing-android

Syncthing-Fork - A Syncthing Wrapper for Android.
Mozilla Public License 2.0
1.14k stars 40 forks source link

Incremental Kotlin migration #1112

Open ds84182 opened 2 weeks ago

ds84182 commented 2 weeks ago

androidx APIs are simpler to use with Kotlin, and its built-in async support would reduce what I call "listener & callback soup". For example, here, could be 3 calls to launch then a call to joinAll.

I think it would reduce code complexity & make maintenance and new features easier to implement. Thoughts, @Catfriend1?

Catfriend1 commented 2 weeks ago

@ds84182 thanks for your feedback und suggestion. I like it technically, but won't do it myself. at the moment I'm just happy if the whole thing works on recent android's. :-)