kbilsted / StatePrinter

Automating unit testing and ToString() coding
Apache License 2.0
93 stars 32 forks source link

Fix a race condition in HarvestInfoCache.TryAdd(). #51

Closed romkatv closed 7 years ago

romkatv commented 7 years ago

HarvestInfoCache.TryAdd() can be called concurrently, in which case it can throw ArgumentException when adding a type that has already been added by another thread.

romkatv commented 7 years ago

Thanks for accepting the PR, Kasper. Could you push a new version to NuGet?

kbilsted commented 7 years ago

Done. I even added some hefty performance improvements :-)