Re: [PATCH v6 0/1] MR3347: klist: Add a program that lists Kerberos tickets.

[Hans Leidekker (＠hans)]

Hans Leidekker (@hans) commented about programs/klist/main.c:
> +    EXPAND_KERB_ETYPES(encryption_type)
> +}
> +
> +static BOOL get_process_logon_id(LUID *const logon_id)
> +{
> +    HANDLE token_handle;
> +    TOKEN_STATISTICS token_statistics;
> +    DWORD token_statistics_len;
> +    BOOL err;
> +
> +    err = OpenProcessToken(GetCurrentProcess(), TOKEN_QUERY, &token_handle);
> +    if (!err)
> +    {
> +        WINE_ERR("OpenProcessToken failed\n");
> +        return err;
> +    }
> The WINE_ prefix is not needed anymore. Returning FALSE would be a little clearer.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3347#note_39272