Updating Cache.Add documentation - Fails silently if key already exists rather than throwing an exception

[prasanthlouis]

I think it'd be more clear if it was explained that using the Add method to add to an already existing key fails "silently" rather than throwing an exception, or something along those lines. When I read the word "fails", I was expecting it was going throw an exception just like how dictionaries work. What do you guys think? If not "silently", then maybe something along those lines.

Summary

Describe your changes here.

Fixes #Issue_Number (if available)

[Thraka]

This looks good to me. However, I can't seem to find the source code for the CacheStoreProvider type (which is what is implementing the Add method) on Reference Source to validate this indeed behaves this way...

[mairaw]

Wouldn't be this the implementation of this method?
https://referencesource.microsoft.com/#System.Web/Cache/cache.cs,26152879ad8a322a

[mairaw]

And for CacheStoreProvider, that's available only through the internal version:
http://index/?query=cache.add&rightProject=System.Web&file=Cache%5CCacheStoreProvider.cs&rightSymbol=kvw1ervp0adr

[Thraka]

Yes it was, but it's just forwards to ObjectCache.Add which is a variable of an abstract base class. Anyhoo I tracked it down and verified it does just exit and doesn't throw anything.

Great catch @prasanthlouis

Thanks!