qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 2/3] vl.c: Fold constant string into printf rath


From: Andreas Färber
Subject: Re: [Qemu-devel] [PATCH 2/3] vl.c: Fold constant string into printf rather than using %s
Date: Mon, 12 Dec 2011 10:40:42 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:8.0) Gecko/20111105 Thunderbird/8.0

Am 12.12.2011 09:21, schrieb Michael Ellerman:
> In help() we do what boils down to:
> 
>   printf("%s", "qemu");
> 
> This seems to be an artifact of be995c27640a82c7056b6f53d02ec823570114e5
> ("removed unused code"), which removed some ifdef'ery that used to print
> a different name depending on CONFIG_SOFTMMU.
> 
> But now that is gone and we always use "qemu" we may as well just put
> that in the format string.

I would rather propose to save argv[0] and use that. By now "qemu" is
not even correct for i386.

Andreas

-- 
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg



reply via email to

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