NathanielAB commented on code in PR #3209:
URL: https://github.com/apache/avro/pull/3209#discussion_r1792970591
##########
lang/csharp/src/apache/main/IO/JsonDecoder.cs:
##########
@@ -45,6 +45,14 @@ private class ReorderBuffer
public JsonReader OrigParser { get; set; }
}
+ private class AvroJsonTextReader : JsonTextReader
Review Comment:
Applied your suggestion with regards to making the class `sealed`. Agreed
about the reader not being able to be `AvroJsonTextReader`, it's unfortunate
its being used differently in `DoAction`, but the same fix has also been
applied to that reader.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
