KalleOlaviNiemitalo commented on PR #3242: URL: https://github.com/apache/avro/pull/3242#issuecomment-2484891553
If the method becomes public, then it should have argument null checks and trimming like in Parse(string json). To avoid redundant checks and trimming when the caller has already done those, one could rename the existing `internal static Schema Parse(string json, SchemaNames names, string encspace)` to ParseInternal and add a public Parse wrapper with the checks and trimming. -- 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]
