"Alexandre Julliard" julliard@winehq.org wrote:
The system locale defines the current codepages, so it affects MultiByteToWideChar and similar APIs. The user locale affects resource loading. It's likely that there are places in Wine that don't handle the distinction properly, but I don't think it's reasonable to demand tests for every single API before we can consider the patch.
There is no need to test every API, but there are APIs we need to require mandatory testing:
GetSystemDefaultLCID GetUserDefaultLCID GetACP FindResourceA FindResourceExA CompareStringA GetLocaleInfoA
Particularly since without the patch there is no way to make user and system locales different, so there is no way to test anything at all.
Right, that's why I'm asking to revert patch and commit it simultaneously with a test case and possible fixes for bugs revealed by the tests.