The changes you asked for are done

> > Added the error throw and also changed getExternalVariables to use iterators
> > instead of a vector
> 
> Juan, XPST0008 is not a bad choice for the error code, but in this case, the
> error does not come from an xquery program but from the zorba c++ api. So, you
> must create a new error code. To do this you must modify the
> src/diagnostics/diagnostic_en.xml file. There is a list of API errors there
> already (starting at line 1727). Add your own error there.
> 
> Regarding the getExternalVariables method:
> 
> 1. It doesn't need to return a bool anymore
> 2. There is already a VectorIterator class in src/api/vectoriterator.h. Please
> use that one (it is a little more lightweight than creating a temp seq and
> getting a temp seq iterator).
-- 
https://code.launchpad.net/~zorba-coders/zorba/xqxq-api-changes/+merge/79589
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to     : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp

Reply via email to