Hi,

On Wed, Aug 04, 2010 at 04:20:06AM -0300, Diego Nieto Cid wrote:

> @@ -123,7 +123,7 @@ debug_printf (const char *f, ...)
>  
>    va_start (ap, f);
>  #ifdef XKB_DEBUG  
> -  ret = printf (f, ap);
> +  ret = vfprintf (stderr, f, ap);
>  #endif
>    va_end (ap);

Looks right :-)

I guess the two changes (using v[s]printf(), and using stderr) *could*
be split into two patches -- but this probably would be overkill. I
guess it's fine as it is :-)

-antrik-

Reply via email to