← Back to team overview

zorba-coders team mailing list archive

Re: [Merge] lp:~zorba-coders/zorba/feature-empty_selectors into lp:zorba

 

> The current XQuery 3.0 spec does not talk about implicit iteration for dyn func invocation. But I remember Ghislain syaing that he will try to add implicit iteration to the spec. Don't know if he tried and failed, or did not try at all, or whether he actually succeeded and it just has not been made public yet.
> 
> But no matter what, this is a case where I wouldn't mind a slight divergence from the official spec as the resulting implementation is much cleaner and actually more general as well. 
I have removed the implicit iteration from the dynamic function invocation for now.
When will the hof branch be merged?

> Furthermore, I think we should get rid of the ZORBA_WITH_JSONIQ compiler flag and use an option to enable/disable jsoniq. So, we can get back to 100% XQuery conformance by disabling jsoniq.
I have added such a flag to the Zorba_CompilerOptions in the parser branch.
That's at least a start and we can take it from there. What do you think?


>> On Mar 16, 2013, at 11:32 AM, Markos Zaharioudakis <markos_za@xxxxxxxxx>
>> wrote:
>> 
>>> Review: Needs Fixing
>>> 
>>> The implicit iteration should always be done in the translator, not in the
>> runtime (actually, that's how it is in the hof branch). So the new do-loop in
>> dynamic_function_iterator.cpp should be removed.
>>> 
>>> --
>>> https://code.launchpad.net/~zorba-coders/zorba/feature-
>> empty_selectors/+merge/153615
>>> You proposed lp:~zorba-coders/zorba/feature-empty_selectors for merging.
> -- 
> https://code.launchpad.net/~zorba-coders/zorba/feature-empty_selectors/+merge/153615
> You proposed lp:~zorba-coders/zorba/feature-empty_selectors for merging.


-- 
https://code.launchpad.net/~zorba-coders/zorba/feature-empty_selectors/+merge/153615
Your team Zorba Coders is subscribed to branch lp:zorba.


References