json icon indicating copy to clipboard operation
json copied to clipboard

Remove `allocator` field in `Deserializer`

Open ibokuri opened this issue 2 years ago • 0 comments

I think the initial reasoning for the field was so that deserialization and visitor-related operations could have different allocators. But I'm not even sure if that makes sense or is a good use-case.

~Plus, in deserializeString and deserializeEnum, I think it makes more sense if the visitor duplicates the passed-in string instead of the deserializer since the allocated string ends up being either the value produced by the visitor or a part of the produced value.~

ibokuri avatar Mar 14 '22 20:03 ibokuri