Review: Needs Fixing

Looks like you forgot to add xquery.cpp (test case?).

You added the new parse() virtual method to the public API not at the end of 
the XQuery class.

I don't like our XQuery API (what's the point behind creating an XQuery object 
that doesn't represent a query?), but I guess this new parse() function is 
consistent with it. But would it be useful to also have a function to retrieve 
a ModuleInfo from an XQuery that *has* been associated with a query, eg. from 
Zorba::compileQuery() ?

Your team Zorba Coders is subscribed to branch lp:zorba.

Mailing list:
Post to     :
Unsubscribe :
More help   :

Reply via email to