[PATCH xfs 2/3] Define VErrorF and then try to get libxtrans to use our error log

Alan Coopersmith alan.coopersmith at oracle.com
Fri Feb 14 22:31:41 PST 2014


On 01/29/14 01:14 PM, Keith Packard wrote:
>   void
> -NoticeF(const char *f, ...)
> +VErrorF(const char *f, va_list args)
>   {
> -    /* XXX should Notices just be ignored if not using syslog? */
> -    va_list args;
> -    va_start(args, f);
>   #ifdef USE_SYSLOG
>       if (UseSyslog) {
>   	vsyslog(LOG_NOTICE, f, args);
> @@ -160,6 +157,15 @@ NoticeF(const char *f, ...)
>       fprintf(stderr, "%s notice: ", progname);
>       vfprintf(stderr, f, args);
>   #endif /* USE_SYSLOG */
> +}
> +
> +void
> +NoticeF(const char *f, ...)
> +{
> +    /* XXX should Notices just be ignored if not using syslog? */
> +    va_list args;
> +    va_start(args, f);
> +    VErrorF(f, args);
>       va_end(args);
>   }

Shouldn't that have been done with ErrorF instead of NoticeF, to keep
them with the error prefix & syslog level?   Seems strange to have
VErrorF come through at a different level than ErrorF.


-- 
	-Alan Coopersmith-              alan.coopersmith at oracle.com
	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list