Re: ws2_32: ioctlsocket should try to check if argp is valid

[Juan Lang]

Hi Jeff, I have a stylistic complaint about your tests in this patch:

+    if(sock == INVALID_SOCKET)
+    {
+        ok(0, "Creating the socket failed: %d, skipping test\n",
WSAGetLastError());
+        return;
+    }
If you intend to skip a test, shouldn't you use skip instead?

+        ret = ioctlsocket(sock, cmds[i], (u_long *)1);
+        if(ret != SOCKET_ERROR)
+           ok(0, "ioctlsocket succeeded unexpectedly\n");
+        else
+        {
I'm not a fan of the ok(0) approach here.  Why not use ok(ret ==
SOCKET_ERROR) instead?
--Juan

ws2_32: ioctlsocket should try to check if argp is valid

[Jeff Zaroyko]

On Wed, Sep 10, 2008 at 2:57 AM, Juan Lang <juan.lang(a)gmail.com> wrote:
> Hi Jeff, I have a stylistic complaint about your tests in this patch:
>
> +    if(sock == INVALID_SOCKET)
> +    {
> +        ok(0, "Creating the socket failed: %d, skipping test\n",
> WSAGetLastError());
> +        return;
> +    }
> If you intend to skip a test, shouldn't you use skip instead?
> +        ret = ioctlsocket(sock, cmds[i], (u_long *)1);
> +        if(ret != SOCKET_ERROR)
> +           ok(0, "ioctlsocket succeeded unexpectedly\n");
> +        else
> +        {
> I'm not a fan of the ok(0) approach here.  Why not use ok(ret ==
> SOCKET_ERROR) instead?
> --Juan

Good point, I'll use skip instead and for your second concern, I guess
I can use the return value of ok(ret == SOCKET_ERROR,.. in an if
expression.

I'll resend.

Thanks
-Jeff