2 ms·
>a massive mistake to let double.ToString() use the current culture rather than the invariant culture. I would imagine that's a carryover from the Win32/Client
by Clubber 2y ago
>a massive mistake to let double.ToString() use the current culture rather than the invariant culture.
I would imagine that's a carryover from the Win32/Client-Server days when that would have been a better choice.
Is that annoying? Yea. Is that annoying enough to force companies to collectively spend billions to look through their decades old codebases for double.ToString() and add culture arguments? Also keep in mind, this is a runtime issue, so the time to fix would be much more than if it were a compile issue. I would say no.
- neonsunset 2y agoNowadays you just apply https://learn.microsoft.com/en-us/dotnet/core/runtime-config/globalization#examples https://learn.microsoft.com/en-us/dotnet/core/runtime-config... and call it a day. It is also added as a default to all AOT templates.
- alkonaut 2y agoThat's a great idea (and after the fact, much better than changing the API). On day 1 it should have been easy though.