The proposal to merge lp:~zorba-coders/zorba/skiplimit into lp:zorba has been
updated.
Status: Approved = Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/skiplimit/+merge/173126
--
https://code.launchpad.net/~zorba-coders/zorba/skiplimit/+merge/173126
Review: Needs Information
1. I think Dana will have to approve the adding of proprietary syntax to do
what is essentially syntactic sugar for fn:subsequence().
2. Why allow multiple offset/limit clauses when, as far as I can understand,
only the last offset/limit clause takes effect?
--
1. I make an email loop to get Dana's approval.
2. There are couple of options, I wasn't sure which one to pick:
a) (offset ExprSingle)? (limit ExprSingle)?
b) (offset ExprSingle) | (limit ExprSingle) | (limit ExprSingle offset
ExprSingle) | (offset ExprSingle limit ExprSingle)
c) ((offset
See the updated task description on the bug report. Also, see the updated
coding guidelines for p and error codes.
Documentation and code problems fixed, please check it again.
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1188062/+merge/169946
Your team Zorba Coders is subscribed to
After that, I'm afraid we've got some more substantive changes to be made to
JSON-ify this module. I'll update the bug with details. If you'd like, we
could merge this first pass (after you've addressed the coding guidelines
issues) and then have a separate proposal for the JSONiq updates;
5 matches
Mail list logo
