Re: [PATCH v2] qapi, qemu-options: make all parsing visitors parse boolean options the same

[Paolo Bonzini]

On 04/11/20 08:35, Markus Armbruster wrote:
> > +               "boolean (on/off, yes/no, true/false, y/n)");
> Recommend to have the error message only mention the preferred form.  I
> like the laconic "'on' or 'off'".  It's really all the user needs to
> know.

I went for "boolean (on/off) value".

> Avoiding the full_name() on success isn't hard:
>
>        if (!qapi_bool_parse(name, str, obj, NULL)) {
>            error_setg(errp, QERR_INVALID_PARAMETER_VALUE,
>                       full_name(qiv, name), "'on' or 'off'");
>            return false;
>        }
>        return true;

Can't refuse.

> >      case QEMU_OPT_BOOL:
> > -        return parse_option_bool(opt->name, opt->str, &opt->value.boolean,
> > -                                 errp);
> > +        return qapi_bool_parse(opt->name, opt->str, &opt->value.boolean, errp);
>
> Please break the line the same way as before.

Why? It's not even 80 characters.

Paolo