Re: [Xen-devel] [PATCH 10/14] docs: update xl-disk-configuration.txt to describe new syntax

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

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"?

Yes.

> 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.

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel