Closed desaikd closed 3 months ago
Related PR #791
Description of changes: This PR works on adding is_human_readable() for Serializer and Deserializer in serde implementation of Ion.
is_human_readable()
Serializer
Deserializer
List of changes:
is_human_readable
ValueSerializer
WriteConfig
ValueDeserializer
SystemReader
next_item()
Test: Added a unit test that converts to/from binary and text as well as compares for roundtrip.
Note for reviewer: Miri test here is failing due to writer code in lazy writer and is unrelated to current change.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Related PR #791
Description of changes: This PR works on adding
is_human_readable()forSerializerandDeserializerin serde implementation of Ion.List of changes:
is_human_readableproperty inValueSerializerbased onWriteConfig.is_human_readableproperty inValueDeserializerbased onSystemReader'snext_item().Test: Added a unit test that converts to/from binary and text as well as compares for roundtrip.
Note for reviewer: Miri test here is failing due to writer code in lazy writer and is unrelated to current change.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.