On Fri, 2011-05-20 at 11:21 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [PATCH 10/14] docs: update
> xl-disk-configuration.txt to describe new syntax"):
> > I think it would be ok if you correct the * placements etc. Having some
> > sort of (semi-)formal syntax declaration in addition to the wordy
> > description is useful IMHO.
> > Would the syntax be clearer as a BNF style thing rather than a regexp-y
> > thing? e.g. this would allow "[<target>, [<format>, [<vdev>,
> > [<access>]]]]," to be declared as "POSITIONAL_PARAMETERS := ..." making
> > the syntax a little more self-documenting and "KEY := ..." could be
> > defined with reference to the relevant following sections etc.
> The trouble with bnf is that it all comes out of order -
> self-assembly, so to speak.
> > Should the second "<key>=<value>" usage also have a "|flag"?
> > I presume "key" can also == "target" and can be the non-ultimate key so
> > long as the value does not contain any problematic characters. In this
> > case the explicit target= needs to be marked as optional.
> No, general "key" may not be "target". "target=" always eats the rest
> of the string.
Oh, yes, that's pretty obvious now I think about it...
Xen-devel mailing list