[PATCH 4/4 v4] xf86Helper: use LogHdrMessageVerb in xf86VDrvMsgVerb
walter harms
wharms at bfs.de
Fri Aug 5 07:35:33 PDT 2011
Am 05.08.2011 16:20, schrieb Daniel Kurtz:
> LogHdrMessageVerb allows passing a parameterized header to insert in a log
> message between MessageType and the formatted message body string.
>
> Signed-off-by: Daniel Kurtz <djkurtz at chromium.org>
> Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>
> ---
> hw/xfree86/common/xf86Helper.c | 25 ++++++-------------------
> 1 files changed, 6 insertions(+), 19 deletions(-)
>
> diff --git a/hw/xfree86/common/xf86Helper.c b/hw/xfree86/common/xf86Helper.c
> index f42d0f9..a64b271 100644
> --- a/hw/xfree86/common/xf86Helper.c
> +++ b/hw/xfree86/common/xf86Helper.c
> @@ -1027,36 +1027,23 @@ xf86EnableDisableFBAccess(int scrnIndex, Bool enable)
> }
> }
>
> -/* Print driver messages in the standard format */
> -
> -#undef PREFIX_SIZE
> -#define PREFIX_SIZE 14
> -
> +/* Print driver messages in the standard format of
> + (<type>) <screen name>(<screen index>): <message> */
> void
> xf86VDrvMsgVerb(int scrnIndex, MessageType type, int verb, const char *format,
> va_list args)
> {
> - char *tmpFormat;
> + const char *scrnName = NULL;
>
> /* Prefix the scrnIndex name to the format string. */
> if (scrnIndex >= 0 && scrnIndex < xf86NumScreens &&
> xf86Screens[scrnIndex]->name) {
> - tmpFormat = malloc(strlen(format) +
> - strlen(xf86Screens[scrnIndex]->name) +
> - PREFIX_SIZE + 1);
> - if (!tmpFormat)
> - return;
> -
> - snprintf(tmpFormat, PREFIX_SIZE + 1, "%s(%d): ",
> - xf86Screens[scrnIndex]->name, scrnIndex);
> -
> - strcat(tmpFormat, format);
> - LogVMessageVerb(type, verb, tmpFormat, args);
> - free(tmpFormat);
> + scrnName = xf86Screens[scrnIndex]->name;
why do you need "scrnName" ?
re,
wh
> + LogHdrMessageVerb(type, verb, format, args, "%s(%d): ", scrnName,
> + scrnIndex);
> } else
> LogVMessageVerb(type, verb, format, args);
> }
> -#undef PREFIX_SIZE
>
> /* Print driver messages, with verbose level specified directly */
> void
More information about the xorg-devel
mailing list