neuecc / Utf8Json

Definitely Fastest and Zero Allocation JSON Serializer for C#(NET, .NET Core, Unity, Xamarin).
MIT License
2.36k stars 267 forks source link

Fix #128 #129

Open doominator42 opened 5 years ago

doominator42 commented 5 years ago

PR for issue #128.

AGeorgy commented 4 years ago

Oh, please! Why is it, steel not in master? @neuecc

GrowSharp commented 4 years ago

Can we get this into the master already please.?

penguinawesome commented 4 years ago

Please merge this changes. This fix is important. @neuecc

jensbrak commented 4 years ago

I ended up here because I got the ArgumentException "Key was already exists. Key:" exception when trying to serialize a class with an enum and googled it.

Part from the functional aspects of the exception, fix the grammar in the exception argument as well. Not that it really matters but it looks kinda bad. Suggestion: "Key already exist. Key:" or perhaps "Duplicate Key"...