Re: [PATCH 0/1] MR4853: iphlpapi: Implement GetRTTAndHopCount().

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/iphlpapi/iphlpapi_main.c:
> - RETURNS
> - Success: TRUE
> - Failure: FALSE
> -
> -
> -
> - FIXME
> -
> - Stub, returns FALSE.
> */
>
> -BOOL WINAPI GetRTTAndHopCount(IPAddr DestIpAddress, PULONG HopCount, ULONG MaxHops, PULONG RTT)
> -{
> - FIXME("(DestIpAddress 0x%08lx, HopCount %p, MaxHops %ld, RTT %p): stub\n",
> - DestIpAddress, HopCount, MaxHops, RTT);
> - return FALSE;
>
> +BOOL WINAPI GetRTTAndHopCount( IPAddr DestIpAddress, PULONG HopCount, ULONG MaxHops, PULONG RTT )
> +{

Could we change the parameter names to be snake_case - e.g. `dest_addr`, `hop_count`? Also `PULONG` -> `ULONG *`.

There is also trailing whitespace after the opening brace.  There are other instances of trailing whitespace below.