Aric Stewart <aric(a)codeweavers.com> writes:
> + if (!caps->IsRange)
> + {
> + static char out[30];
> + unsigned int i;
> + strcpy(out, "[ ");
> + for (i = 0; i < caps->usage_count; i++)
> + strcat(out, wine_dbg_sprintf("0x%x ", caps->u.NotRange.Usage[i]));
> + strcat(out,"]");
> + return out;
Having a static buffer defeats the purpose of using wine_dbg_sprintf.
> +static void debug_feature(struct feature *feature)
> +{
> + if (!feature)
> + return;
> + TRACE("[\n\t type %s [%i]; %s; %s; %s; %s; %s; %s; %s; %s; %s\n\t %s\n]\n",
Please don't put \n in traces, that messes up the output. If you truly
need multiple lines, use separate TRACE statements.
--
Alexandre Julliard
julliard(a)winehq.org