Support for multiple alternative names during JSON parsing #597
Conversation
runtime/commonMain/src/kotlinx/serialization/json/JsonAlternativeNames.kt
Outdated
Show resolved
Hide resolved
runtime/commonMain/src/kotlinx/serialization/json/JsonAlternativeNames.kt
Outdated
Show resolved
Hide resolved
runtime/commonMain/src/kotlinx/serialization/json/JsonAlternativeNames.kt
Outdated
Show resolved
Hide resolved
runtime/commonMain/src/kotlinx/serialization/json/internal/StreamingJsonInput.kt
Outdated
Show resolved
Hide resolved
runtime/commonMain/src/kotlinx/serialization/json/internal/TreeJsonInput.kt
Outdated
Show resolved
Hide resolved
runtime/commonMain/src/kotlinx/serialization/json/internal/TreeJsonInput.kt
Outdated
Show resolved
Hide resolved
runtime/commonMain/src/kotlinx/serialization/json/internal/TreeJsonInput.kt
Outdated
Show resolved
Hide resolved
runtime/commonMain/src/kotlinx/serialization/json/internal/TreeJsonInput.kt
Outdated
Show resolved
Hide resolved
Introduce DescriptorSchemaCache class
Stabilize .toString() for SerialClassDescImpl; because HashMaps on different platforms have different iteration order
Hold cache in Json instance instead of passing it everywhere
Now cache map is not created if main name for property is used
sandwwraith
force-pushed
the
alternate_names
branch
from
c98be7d
to
ddd6fcb
Compare
Fix documentation and tests
| * This cache uses ConcurrentHashMap on JVM and regular maps on other platforms. | ||
| * To be able to work with it from multiple threads in Kotlin/Native, use @[ThreadLocal] in appropriate places. | ||
| */ | ||
| internal class DescriptorSchemaCache { |
There was a problem hiding this comment.
This type seems like it would be good to be public. It is something that many formats could reuse.
|
really love to wait for this features merged |
|
hi， When can this be used？ |
|
Postponed for after-1.0.0 (1.1.0 likely) |
Hey there, |
|
Hi there 👋 Same here I am really looking forward to this PR to be merged and released 😞 |
|
I am also looking for this feature. Also it would be nice if aliases could be processed differently. Like I have "durationFrames": 30, "durationMillis": 1000. So json can contain one of the following items. |
|
Unfortunately 1.1.0 Released without this feature, When It can be used ? |
|
@mustafaozhan We plan it for the next release |
|
Superseded by #1375 |
|
@sandwwraith glad to hear that thanks a lot! |
@JsonAlternativeNamesannotationDescriptorSchemaCache– class for associating custom format info with descriptors in effective and type-safe wayFixes #203