[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v2] ns16550: Add command line parsing adjustments



>>> On 09.01.17 at 17:58, <ian.jackson@xxxxxxxxxxxxx> wrote:
> Jan Beulich writes ("Re: [PATCH v2] ns16550: Add command line parsing 
> adjustments"):
>> Right, I had been thinking of this too, but then wasn't sure to
>> suggest such a mixture. Perhaps the first two options could
>> even remain completely nameless then (I think those are the
>> ones which had been there from the beginning)?
> 
> We have to support comma-separated lists for backward compatibility,
> anyway.  It's probably easier to support
>   ( existing positional parameter ',' )*  ( name '=' value ',' )*
> than to split the whole thing into two parsers.

Just seen that, which of course is fine.

> So I just suggested to Swapnil on irc that they should implement their
> additional two parameters as named parameters which could be suffixed
> onto the existing comma-separated string.  I don't want to ask them to
> implement a complete conversion, but once the framework is there we
> can easily support (for example) irq setting both positionally and by
> name.

Right - limiting this to just their addition was my suggestion too,
if he's not up to doing the full thing.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.