[2/5] resend ws2_32/tests: add tests for Inet_Ntop and inet_ntoa

Paul Vriens paul.vriens.wine at gmail.com
Mon Mar 9 02:12:10 CDT 2009


Jeff Latimer wrote:
> Note that Inet_Ntop is only available in Vista and Windows 2008.  This 
> patch removes the version testing.  This one removes the redundant test 
> after GetModuleHandle and adds win_skip().
> ---
>  dlls/ws2_32/tests/sock.c |   68 
> ++++++++++++++++++++++++++++++++++++++++++++++
>  1 files changed, 68 insertions(+), 0 deletions(-)
> 
> 
> 
> ------------------------------------------------------------------------
> 
> 
Hi,

There is no need to have two if (pInetNtop) statements right after each other, 
this could be just one.

Maybe it's even better that before you start using pInetNtop, you do:

if (!pInetNtop)
{
     win_skip("InetNtop not present, not executing tests\n");
     return;
}

You than don't need to indent the code anymore.

-- 
Cheers,

Paul.



More information about the wine-devel mailing list