Code review comment for lp:~zorba-coders/zorba/new-jsoniq

Revision history for this message
Ghislain Fourny (gislenius) wrote :

> > > Why would you need a qualified value if the option itself is already
> > qualified?
> >
> > I guess because the option name and the option value may not live in the
> same
> > namespace.
>
> Why not? It's not as if you do schema validation on the options and their
> values. The option needs to be qualified to distinguish it from XQuery
> options, but the value needs not. And the validation is done entirely in the
> code so there is no schema validation.

Technically, it is up to us - does the op:enable option accept QName or string values? If it is strings, I agree. If it is QNames, the namespace is important I think.

« Back to merge proposal