[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 27/29] libqtest: Make qtest_init() accept for
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v6 27/29] libqtest: Make qtest_init() accept format string |
Date: |
Thu, 7 Sep 2017 13:00:48 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 09/05/2017 07:46 AM, Thomas Huth wrote:
> On 01.09.2017 20:03, Eric Blake wrote:
>> Several callers were formatting a string into a temporary
>> variable before calling qtest_init(); factor that into the
>> common code.
>>
>> Signed-off-by: Eric Blake <address@hidden>
>> ---
>> -QTestState *qtest_init(const char *extra_args);
>> +QTestState *qtest_init(const char *extra_args, ...) GCC_FMT_ATTR(1, 2);
>
> Maybe the parameter should be called "fmt" or "format" now instead?
> (similar to printf() and friends)
Sure.
>
>> /**
>> * qtest_init_without_qmp_handshake:
>> @@ -518,7 +519,7 @@ void qtest_add_abrt_handler(GHookFunc fn, const void
>> *data);
>> */
>> static inline QTestState *qtest_start(const char *args)
>> {
>> - global_qtest = qtest_init(args);
>> + global_qtest = qtest_init("%s", args);
>
> That's a little bit sad that there are some spots which need an explicit
> "%s" now ... maybe it would be nicer to add a new function called
> qtest_initf() instead and keep the old one (without the "f") as it is?
Indeed, and looking back over my series, I see we could also benefit
from a va_list form.
Also, since you already questioned me on the churn from qtest_start() =>
qtest_init() => qtest_start(), I can make less churn in the next spin by
converging on qtest_start() from the get-go.
So I will go with qtest_start() (plain string), qtest_startf()
(formatted like sprintf), and qtest_vstartf() (formatted like vsprintf).
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
[Qemu-devel] [PATCH v6 26/29] fw_cfg-test: Drop dependence on global_qtest, Eric Blake, 2017/09/01
[Qemu-devel] [PATCH v6 28/29] libqtest: Remove qtest_start() and qtest_end() shortcuts, Eric Blake, 2017/09/01
[Qemu-devel] [PATCH v6 29/29] libqtest: Rename qtest_init() to qtest_start(), Eric Blake, 2017/09/01