You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Yes can confirm this; it looks like this was indeed introduced by 3e3266cf48f132928225e1561a6ae4cb5503d08f (by accident):
There is only JsonWriter.value(long), so the adapters for all other integral primitive types (byte, short, int) actually call value(long).
Sorry for that. Maybe it would make sense to (partially) revert that commit.
It seems AtomicInteger is affected as well, but it's adapter always behaved this way, that a Long instead of the AtomicInteger is stored in the JsonPrimitive. However, this might be a good thing since it is mutable and might cause confusing behavior if it was stored in JsonPrimitive.
There is only
JsonWriter.value(long)
, so the adapters for all other integral primitive types (byte
,short
,int
) actually callvalue(long)
.Sorry for that. Maybe it would make sense to (partially) revert that commit.
It seems
AtomicInteger
is affected as well, but it's adapter always behaved this way, that aLong
instead of theAtomicInteger
is stored in theJsonPrimitive
. However, this might be a good thing since it is mutable and might cause confusing behavior if it was stored inJsonPrimitive
.Originally posted by @Marcono1234 in #2680 (comment)
The text was updated successfully, but these errors were encountered: