qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] trivial: remove trailing newl


From: Stefan Hajnoczi
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH v2] trivial: remove trailing newline from error_report
Date: Thu, 9 Jul 2015 11:26:23 +0100
User-agent: Mutt/1.5.23 (2014-03-12)

On Mon, Jun 29, 2015 at 04:56:26PM -0400, John Snow wrote:
> Minor cleanup.
> 
> Signed-off-by: John Snow <address@hidden>
> ---
>  hw/ide/ahci.c      | 2 +-
>  hw/vfio/platform.c | 2 +-
>  net/tap-linux.c    | 4 ++--
>  3 files changed, 4 insertions(+), 4 deletions(-)

Reviewed-by: Stefan Hajnoczi <address@hidden>

Laszlo Ersek suggested a compile-time check (e.g. BUILD_BUG_ON()).  Have
you looked into this?

Perhaps something along the lines of:

BUILD_BUG_ON(__builtin_constant_p(fmt) &&
             __builtin_strlen(fmt) > 0 &&
             fmt[__builtin_strlen(fmt) - 1] == '\n')

Attachment: pgpHlLM_yOi3J.pgp
Description: PGP signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]