Hey Cedric have you heard of Moshi and Jackson considering migrating from `kotlin reflect` to `kotlinx metadata` It s supposed to be <https github com ZacSweers MoshiX blob main moshi metadata reflect README md plain=1 L36|faster> and provides a nice reduction in binary size 3 MB to 1 MB Have you considered performing a similar change <https github com square moshi pull 1183> <https github com FasterXML jackson module kotlin issues 450>

@cedric - would you consider adding a few more devs to the collaborators on this project to help out? if you're backing away from maintaining it, that might be a way to keep it moving forward without sucking up your time ¯\_(ツ)_/¯

(...and apologies for the sunday afternoon mention)

cedric

08/07/2022, 8:06 PM

@Larry Meadors Absolutely! I feel bad for not having enough time to keep working on this project, happy to enlist some help. Why don't you send me a PR, just for sanity check, and after that, happy to add interested people as collaborators

@Andrew O'Hara I hit a pretty bad dead end (performance wise) with the current reflection libraries, filed a bug many years ago, but it never went anywhere. I started ripping it out and replacing it with regular Java reflection but that ended up being a lot of work and I never finished that work. So, short answer: yes, I think Klaxon would benefit from a better reflection library, and along the way, maybe make the boundaries a bit clearer in case faster/better reflection libraries come up in the future, so it's easy to swap them

Andrew O'Hara

08/07/2022, 8:08 PM

For reference, here's how kotlinx-reflection can be used with moshi: https://github.com/ZacSweers/MoshiX/tree/main/moshi-metadata-reflect

Larry Meadors

08/07/2022, 10:14 PM

@cedric I sent a pr a few months ago. Does that count? 😎

cedric

08/08/2022, 1:24 AM

Oh... really... who are you on github? TreffnonX?

Larry Meadors

08/08/2022, 5:06 AM

Nope, lmeadors

18 Views

#klaxon Join Slack