Re: [PATCH v6 0/2] MR3473: whoami: Refactor and add more commands.

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about programs/whoami/main.c:
> }
>
> -int __cdecl wmain(int argc, WCHAR *argv[])
> +static BOOL get_user_name(EXTENDED_NAME_FORMAT name_format, WCHAR **name)
>  {
> - WCHAR *buf = NULL;
> - ULONG size = 0;
> - ULONG size;
> BOOL result;
> - if (argc > 1)
> - result = GetUserNameExW(name_format, NULL, &size);
> - if (result || GetLastError() != ERROR_MORE_DATA)
> {
> -
>    int i;
> -
>    ERR("GetUserNameExW expected ERROR_MORE_DATA, error %ld\n", GetLastError());

Printing these errors isn't very useful. One catch-all ERR() like you have when get_user_name() fails should be enough.