[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 26/30] qemu-img: Use user_creatable_process_cmdline() for
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v3 26/30] qemu-img: Use user_creatable_process_cmdline() for --object |
Date: |
Mon, 15 Mar 2021 15:15:52 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Kevin Wolf <kwolf@redhat.com> writes:
> Am 13.03.2021 um 13:30 hat Markus Armbruster geschrieben:
>> Paolo Bonzini <pbonzini@redhat.com> writes:
>>
>> > On 13/03/21 08:40, Markus Armbruster wrote:
>> >>> + if (!user_creatable_add_from_str(optarg, &local_err)) {
>> >>> + if (local_err) {
>> >>> + error_report_err(local_err);
>> >>> + exit(2);
>> >>> + } else {
>> >>> + /* Help was printed */
>> >>> + exit(EXIT_SUCCESS);
>> >>> + }
>> >>> + }
>> >>> + break;
>> >>> }
>> >>> - } break;
>> >>> case OPTION_IMAGE_OPTS:
>> >>> image_opts = true;
>> >>> break;
>> >> Why is this one different? The others all call
>> >> user_creatable_process_cmdline().
>> >>
>> >>
>> >
>> > It's to exit with status code 2 instead of 1.
>>
>> I see. Worth a comment?
>
> There is a comment at the start of the function (which is just a few
> lines above) that explains the exit codes:
>
> * Compares two images. Exit codes:
> *
> * 0 - Images are identical or the requested help was printed
> * 1 - Images differ
> * >1 - Error occurred
I had in mind a comment that helps me over the "why is this not using
user_creatable_process_cmdline()" hump. Like so:
case OPTION_OBJECT:
{
/*
* Can't use user_creatable_process_cmdline(), because
* we need to exit(2) on error.
*/
... open-coded variation of
user_creatable_process_cmdline() ...
}
Entirely up to you.
- [PATCH v3 22/30] qom: Factor out user_creatable_process_cmdline(), (continued)
[PATCH v3 30/30] qom: Drop QemuOpts based interfaces, Kevin Wolf, 2021/03/08
[PATCH v3 15/30] qapi/qom: Add ObjectOptions for confidential-guest-support, Kevin Wolf, 2021/03/08
Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add, Peter Krempa, 2021/03/10
- Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add, Paolo Bonzini, 2021/03/10
- Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add, Peter Krempa, 2021/03/10
- Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add, Kevin Wolf, 2021/03/10
- Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add, Peter Krempa, 2021/03/11
- Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add, Paolo Bonzini, 2021/03/11
- Re: [PATCH v3 00/30] qapi/qom: QAPIfy --object and object-add, Kevin Wolf, 2021/03/11