[PATCH] xfree86 : Simplify xf86Msg{,Verb}

Peter Hutterer peter.hutterer at who-t.net
Sun Aug 1 20:42:04 PDT 2010


On Sun, Aug 01, 2010 at 04:01:27PM -0700, Jesse Adkins wrote:
> Previously, the functions would call xf86VDrvMsgVerb with a screen of -1
>  despite their comments saying they were for "non-driver messages".
> They now call LogVMessageVerb, which is what xf86VDrvMsgVerb does anyway
>  when it has a screen == -1.
> Signed-off-by: Jesse Adkins <jesserayadkins at gmail.com>
> ---
>  hw/xfree86/common/xf86Helper.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/hw/xfree86/common/xf86Helper.c b/hw/xfree86/common/xf86Helper.c
> index 07f9f0a..90e0c67 100644
> --- a/hw/xfree86/common/xf86Helper.c
> +++ b/hw/xfree86/common/xf86Helper.c
> @@ -1267,7 +1267,7 @@ xf86MsgVerb(MessageType type, int verb, const char *format, ...)
>      va_list ap;
>  
>      va_start(ap, format);
> -    xf86VDrvMsgVerb(-1, type, verb, format, ap);
> +    LogVMessageVerb(type, verb, format, ap);
>      va_end(ap);
>  }
>  
> @@ -1278,7 +1278,7 @@ xf86Msg(MessageType type, const char *format, ...)
>      va_list ap;
>  
>      va_start(ap, format);
> -    xf86VDrvMsgVerb(-1, type, 1, format, ap);
> +    LogVMessageVerb(type, 1, format, ap);
>      va_end(ap);
>  }
>  
> -- 
> 1.7.0.4
> 

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Cheers,
  Peter


More information about the xorg-devel mailing list