Re: [v2] msvcrt: Introduce vf[w]printf_helper function.

[Piotr Caban]

Hi,

On 10/02/17 21:46, Gijs Vermeulen wrote:
> -int CDECL MSVCRT_vfprintf(MSVCRT_FILE* file, const char *format, __ms_va_list valist)
> +static int vfprintf_vfwprintf_helper(MSVCRT_FILE* file, const char *format, const MSVCRT_wchar_t *format_wide,
> +        __ms_va_list valist, MSVCRT__locale_t locale, DWORD options, BOOL wide)
> I would prefer to have a separate helper for vfprintf and vfwprintf 
> functions. Also please reorder the parameters so it's more similar to 
> _stdio_common_vfprintf.
> +    if(!MSVCRT_CHECK_PMT( file != NULL )) return -1;
> +    if(!(wide && MSVCRT_CHECK_PMT( format != NULL ))) return -1;
> This doesn't work as expected. It should look like this:
> if(!wide && !MSVCRT_CHECK_PMT(...

Thanks,
Piotr