search

metosin / jsonista

Clojure library for fast JSON encoding and decoding.
https://cljdoc.org/d/metosin/jsonista
Eclipse Public License 2.0
422 stars 30 forks source link

Reflection warnings #50

Closed FieryCod closed 3 years ago

FieryCod commented 3 years ago

I'm receiving some reflection warnings, when using jsonista. I think that changing this is reasonable, because some libraries which use jsonista with GraalVM now have to providereflection-config.json. 

Reflection warning, jsonista/core.clj:139:19 - call to method registerModule can't be resolved (target class is unknown).
Reflection warning, jsonista/core.clj:140:19 - call to method registerModule can't be resolved (target class is unknown).
Reflection warning, jsonista/core.clj:142:39 - call to method enable can't be resolved (target class is unknown).
Reflection warning, jsonista/core.clj:143:44 - call to method enable can't be resolved (target class is unknown).
Reflection warning, jsonista/core.clj:144:71 - reference to field getFactory can't be resolved.
Reflection warning, jsonista/core.clj:144:71 - call to method enable can't be resolved (target class is unknown).
Reflection warning, jsonista/core.clj:146:8 - call to method registerModule can't be resolved (target class is unknown).
Reflection warning, jsonista/core.clj:146:8 - call to method registerModule can't be resolved (target class is unknown).
Reflection warning, jsonista/core.clj:147:6 - call to method disable can't be resolved (target class is unknown).

Would you like to have a PR for fixing this?

viesti commented 3 years ago

Thank you for noticing! I think a PR is very well welcome :)

FieryCod commented 3 years ago

It's already fixed there: https://github.com/metosin/jsonista/commit/90f447aac858a18194dcd95f01ca40062f41866f Could you please roll out a new release :)?

viesti commented 3 years ago

Released in 0.3.2 @FieryCod !