zeroc-ice / ice

All-in-one solution for creating networked applications with RPC, pub/sub, server deployment, and more.
https://zeroc.com
GNU General Public License v2.0
2.03k stars 593 forks source link

Documentation issue with Ice.CacheMessageBuffers in C# #2171

Closed externl closed 3 months ago

externl commented 3 months ago

We document that the default for Ice.CacheMessageBuffers is 1 in C#, however it's actually 2 in the code.

https://github.com/zeroc-ice/ice/blob/3.7/csharp/src/Ice/Instance.cs#L931

What about the actually default be for C# and why is it supposedly different than Java's default.

EDIT: Reading the docs a little closer it seems that's there isn't really a difference between 1 and 2 in C#. So it doesn't really matter what we pick.

externl commented 3 months ago

I'm just going to update the docs to say 2 for C#