[PATCH] IPHLPAPI.DLL: Implement functions ConvertLengthToIpv4Mask and ConvertIpv4MaskToLength

Huw Davies huw at codeweavers.com
Mon Mar 19 08:39:34 CDT 2018


On Mon, Mar 19, 2018 at 11:55:38AM +0100, Roger Zoellner wrote:
> ---
>  dlls/iphlpapi/iphlpapi.spec   |  4 ++--
>  dlls/iphlpapi/iphlpapi_main.c | 48 +++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 50 insertions(+), 2 deletions(-)

Hi Roger,

Thank you for your contribution.  We actually have another implementation
of ConvertLengthIpv4Mask() that's almost viable (see wine-devel for the
details), so I suggest you concerntrate on something else.  Your
ConvertIpv4MaskToLength implementation would be a good choice to work
on, so I'd resubmit just the bits that touch that.

You will need to include the following:

A Signed-off-by: line.
Some tests.
Add the prototype of ConvertIpv4MaskToLength in include/netioapi.h


> diff --git a/dlls/iphlpapi/iphlpapi.spec b/dlls/iphlpapi/iphlpapi.spec
> index b6c9aef2f8..167c646de4 100644
> --- a/dlls/iphlpapi/iphlpapi.spec
> +++ b/dlls/iphlpapi/iphlpapi.spec
> @@ -22,8 +22,8 @@
>  @ stdcall ConvertInterfaceNameToLuidA( str ptr )
>  @ stdcall ConvertInterfaceNameToLuidW( wstr ptr )
>  #@ stub ConvertInterfacePhysicalAddressToLuid
> -#@ stub ConvertIpv4MaskToLength
> -#@ stub ConvertLengthToIpv4Mask
> +@ stdcall ConvertIpv4MaskToLength ( long ptr )
> +@ stdcall ConvertLengthToIpv4Mask( long ptr)

Also, watch out for inconsistent white-space.

Huw.



More information about the wine-devel mailing list