← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/bug-867363-disable-http into lp:zorba

 

> 1. I don't see the zorba cmd doing anything with the --disable-http-resolution
> option, if that option is actually specified in the command line.

See the second diff in zorbacmdproperties_base.h. It just explicitly sets the disable-http-resolution option on the static context by using the same code as --option.


> 2. If a user uses the option inside an xquery module, it will have no effect
> because options are processed after the imports. Such a user will then be very
> confused. We should probably consider moving the processing of options before
> the imports, but until we do this, we should throw an error if the disable-
> http-resolution option is encountered inside a module prolog.

I'll implement something for this.

> The error (and
> the documentation) should say that this option is applicable only at the C++
> api.

Yes, you're right; I'll fix that too.
-- 
https://code.launchpad.net/~zorba-coders/zorba/bug-867363-disable-http/+merge/133032
Your team Zorba Coders is subscribed to branch lp:zorba.


Follow ups

References