Eric Blake <ebl...@redhat.com> writes:
> On 06/13/2015 08:20 AM, Markus Armbruster wrote:
>> qerror_report() is a transitional interface to help with converting
>> existing monitor commands to QMP. It should not be used elsewhere.
>> Replace by error_report().
>>
>> Signed-off-by: Markus Armbruster <arm...@redhat.com>
>> ---
>> tpm.c | 10 ++++------
>> 1 file changed, 4 insertions(+), 6 deletions(-)
>>
>> diff --git a/tpm.c b/tpm.c
>> index 99a9895..224e65e 100644
>> --- a/tpm.c
>> +++ b/tpm.c
>> @@ -140,23 +140,21 @@ static int configure_tpm(QemuOpts *opts)
>>
>> id = qemu_opts_id(opts);
>> if (id == NULL) {
>> - qerror_report(ERROR_CLASS_GENERIC_ERROR, QERR_MISSING_PARAMETER,
>> "id");
>> + error_report(QERR_MISSING_PARAMETER, "id");
>
> Ah, this fixes a hunk difference I pointed out in 5/11. If you need to
> rebase, I'd consider rearranging to put this patch first, so that the
> mechanical 5/11 is even more mechanical (limited to just error_setg
> conversion).
While that would make PATCH 05 a bit more mechanical, this one would
become quite a bit uglier, because I can't directly pass
QERR_MISSING_PARAMETER to any of the error_report() functions.
I'd either have to create an Error to pass to error_report_err(), or
drop the unwanted error class by wrapping parenthesis around
QERR_MISSING_PARAMETER to turn its comma into a comma operator. Ugh.
> But that's not enough reason in itself for a respin, so:
>
> Reviewed-by: Eric Blake <ebl...@redhat.com>
Thanks!
