Jon Skeet
Jon Skeet
There are a few things I'd probably want to change here - I wonder if the best thing is for me to create an PR from scratch rather than leave...
Great. I'll try to come up with something next week.
@JasonLunn: I haven't had any feedback, and don't have the time to chase at the moment I'm afraid.
@mkruskal-google: I'll definitely add tests before we merge :) I'd like feedback on the API surface from potential consumers such as @JamesNK before writing those tests though.
I don't mind changing the default limit, but I wouldn't want to do so without input from the rest of the Protobuf team. @mkruskal-google do you have thoughts here? Does...
@mkruskal-google: It's already configurable on CodedInputStream, but that's *relatively* fiddly to use. There was a proposal to introduce a new SerializationOptions, but that wasn't going to be backwardly compatible. I...
I'm going to have another look at the gRPC issue, and see how gRPC performs parsing, to see if there's an alternative that doesn't take C# out of kilter with...
Okay, I've had a look. It's definitely *feasible*, but it's somewhat fiddly. Fortunately only in a relatively small number of places, but even so, we need to think carefully before...
This is on our "to-do" list, but is far from trivial unfortunately. (It's good to get concrete reasons from customers - that definitely helps us in terms of planning and...
@amanda-tarafa I suggest that as we don't really *need* the Microsoft.CodeAnalysis.CSharp bump, we just close this until we're ready to update to .NET 8.