Re: [PATCH v2 0/6] MR3994: include: Introduce wine_dbgstr_fourcc and debugstr_fourcc.

[Zebediah Figura (＠zfigura)]

Zebediah Figura (@zfigura) commented about include/wine/debug.h:
>                           id->Data4[4], id->Data4[5], id->Data4[6], id->Data4[7] );
>
> }
>
> +static inline const char *wine_dbgstr_fourcc( unsigned int fourcc )
> +{
> - char str[4] = { fourcc, fourcc >> 8, fourcc >> 16, fourcc >> 24 };
> - if (!fourcc)
> -
>    return "''";
> - if ((isprint( str[0] ) || str[0] == ' ') && (isprint( str[1] ) || str[1] == ' ') &&
> -
>    (isprint( str[2] ) || str[2] == ' ') && (isprint( str[3] ) || str[3] == ' '))

isprint() already includes space.