zorba-coders team mailing list archive

[Till Westmann <till.westmann@xxxxxxxxxx>]

> - JNTY0023 doesn't exist in the spec

I've sent an e-mail to the list to add it to the spec.

> - jn:decode-from-roundtrip should return item()*, e.g. it could also return
> atomics
> - Probably not related to this proposal but it's strange that the following
> example doesn't raise a sequence-type matching error (in this branch)
>   jn:decode-from-roundtrip({ "Q{http://jsoniq.org/roundtrip}type"; :
> "xs:dateTime", "Q{http://jsoniq.org/roundtrip}value"; :
> "2012-09-21T09:44:14.753-07:00" }) instance of xs:dateTime
> 
> - jn:encode-for-roundtrip should accept item()*

Thinking more about this both functions should probably accept and return item()* as simple atomic values 5 should just go through untouched.
I've changed the signature to item()* for now and sent an e-mail to the list for clarification.

> - the exception raised in JSONEncodeForRoundtripIterator::encodeNode doesn't
> have a query location. It could have one by directly raising an
> XQueryException or catching and rethrowing it in nextImpl with the location
> added.

fixed

> - ChangeLog entry missing

fixed
-- 
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-roundtrip/+merge/124316
Your team Zorba Coders is subscribed to branch lp:zorba.