[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: |
Kevin Wolf |
Subject: |
Re: [PATCH v3 26/30] qemu-img: Use user_creatable_process_cmdline() for --object |
Date: |
Mon, 15 Mar 2021 12:38:54 +0100 |
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
Kevin
- [PATCH v3 16/30] qapi/qom: Add ObjectOptions for input-*, (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