Claudio Fontana wrote:
Hello all,
I posted previously a patch to wine-patches here:
http://www.winehq.org/pipermail/wine-patches/2006-December/034022.html
However, I now understand that although it solves the issue I had with the specific program, the change is incomplete, since in several other places, sections and keys are only considered meaningful when not equal to the empty string. In the Microsoft implementation, the unnamed section is valid, as are zero length keys.
Thus, I think a broader scope change is required. Before I start working on a new patch, do you acknowledge that the current implementation is incorrect, and a patch fixing it by not requiring name[0] to be != '\0' is likely to be accepted? (btw, I've noticed you switched to git, so I will be using that instead now).
Hi Claudio,
The patch looks good, apart from the specific version check for win95. Do you think you could add some tests to our test framework (dlls/kernel32/tests) to test for this issue that you've found. Then the tests could be run on a variety of platforms to see if it is only win95 that has that specific behaviour or if the original author of the comment was wrong about that.
The tests that fail on Wine will then determine what needs to be fixed.
Thanks,