[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 6/6] qemu-option: warn for short-form boolean options
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 6/6] qemu-option: warn for short-form boolean options |
Date: |
Tue, 10 Nov 2020 09:32:40 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.1 (gnu/linux) |
Paolo Bonzini <pbonzini@redhat.com> writes:
> Il lun 9 nov 2020, 22:19 Markus Armbruster <armbru@redhat.com> ha scritto:
>
>> This function now warns, except for "help" and "?". The exception
>> applies even when we treat "help" and "?" as sugar for "help=on" and
>> "?=on" because opts_accepts_any().
>>
>
> Right, because again help_wanted will be false for non-validated
> QemuOptsList.
>
>> Summary: only qemu_opts_parse_noisily() warns. This is airtight only if
>> all user input flows through qemu_opts_parse_noisily().
>
>
> HMP doesn't. But that's too hard to change now, and it's not considered as
> much of a stable interface as the command line.
>
> Anyway I am not going to push this for 5.2. Thanks for the speedy reviews
> anyway!
You're welcome! It was worth a try. We can try again for 6.0 without
time pressure.
- Re: [PATCH v2 4/6] qemu-option: clean up id vs. list->merge_lists, (continued)
[PATCH v2 5/6] qemu-option: move help handling to get_opt_name_value, Paolo Bonzini, 2020/11/09
[PATCH v2 6/6] qemu-option: warn for short-form boolean options, Paolo Bonzini, 2020/11/09
Re: [PATCH v2 for-5.2 0/6] Deprecate or forbid crazy QemuOpts cases, no-reply, 2020/11/09