Re: [PATCH 0/1] MR5831: dnsapi: Add a stub for DnsServiceBrowse

[Alex Henrie (＠alexhenrie)]

Alex Henrie (@alexhenrie) commented about include/windns.h:
>  ULONG Flags;
>
> } DNS_CACHE_ENTRY, *PDNS_CACHE_ENTRY;
>
> +typedef struct _DNS_SERVICE_BROWSE_CALLBACK {
> +} DNS_SERVICE_BROWSE_CALLBACK, *PDNS_SERVICE_BROWSE_CALLBACK;
> -
>
> +typedef struct _DNS_SERVICE_BROWSE_REQUEST {
> - ULONG  Version;
> - ULONG  InterfaceIndex;
> - PCWSTR QueryName;
> - union {
> - PDNS_SERVICE_BROWSE_CALLBACK pBrowseCallback;
> - DNS_QUERY_COMPLETION_ROUTINE *pBrowseCallbackV2;
> - };
> - PVOID  pQueryContext;

In new code it is preferable to write `void *` instead of `PVOID` and `const WCHAR *` instead of `PCWSTR`. Also, please try to match the style of the previous structs in this file: Four-space indentation, `{` on its own line, one space between the member type and the member name.