[2/2] secur32/tests: Fix GetUserNameEx() tests (try 2)
Kai Blin
kai.blin at gmail.com
Tue Apr 14 06:17:13 CDT 2009
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
--
Kai Blin
WorldForge developer http://www.worldforge.org/
Wine developer http://wiki.winehq.org/KaiBlin
Samba team member http://www.samba.org/samba/team/
--
Will code for cotton.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20090414/a8037b0a/attachment.pgp>
More information about the wine-devel
mailing list