On Tuesday 14 April 2009 12:39:51 Ge van Geldorp wrote:
Hi Ge,
Why did you remove:
- if (0) /* Crashes on Windows */
rc = pGetUserNameExW(NameSamCompatible, NULL, NULL);
The point of these additions was to show/document that Windows crashes when called like this.
Because Alexandre told me for an earlier patch to remove tests that crash on Windows: http://www.winehq.org/pipermail/wine-devel/2009-January/072010.html
What Paul meant is that as this test is behind an if(0), it's not going to be run anyway. So all it does is to serve as documentation of Windows behaviour. Arguably, this could just be a comment, ideally in the implementation of GetUserNameExW, but in general it's nice to have this sort of information.
Cheers, Kai