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

Piotr Caban piotr.caban at gmail.com
Fri Oct 6 12:41:59 CDT 2017


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



More information about the wine-devel mailing list