On Mon, Nov 29, 2010 at 20:57:43 -0800, Alan Coopersmith wrote:

> +/* Old api, now deprecated, may be removed in a future release */
> +char *
> +Xvprintf(const char *format, va_list va)
> +{
> +    char *ret;
> +
> +    Xvasprintf(&ret, format, va);
> +
>      return ret;
>  }
>  
> @@ -68,7 +191,7 @@ char *Xprintf(const char *format, ...)
>      char *ret;
>      va_list va;
>      va_start(va, format);
> -    ret = Xvprintf(format, va);
> +    Xvasprintf(&ret, format, va);
>      va_end(va);
>      return ret;
>  }

Shouldn't these two check for -1 return from Xvasprintf?

Cheers,
Julien
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to