The downside of building this functionality into Zorba: In release mode, the
.xq file isn't read, so the implemented-in-XQuery functions don't exist. I
don't remember the right solution, though.
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uri_module/+merge/96005
Your team Zorba Cod
The attempt to merge lp:~zorba-coders/zorba/feature-uri_module into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job feature-uri_module-2012-03-06T00-39-09.268Z is
finishe
The attempt to merge lp:~zorba-coders/zorba/feature-uri_module into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job feature-uri_module-2012-03-05T23-11-19.724Z is
finishe
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uri_module/+merge/96005
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/~zo
There are additional revisions which have not been approved in review. Please
seek review and approval of these new revisions.
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uri_module/+merge/96005
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://laun
Review: Approve
Typo: "characeters" in the doc of all three uri.xq functions. Other than that,
changes look good. I'll approve now so you can merge after correcting the typo.
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uri_module/+merge/96005
Your team Zorba Coders is subscribed to
- fixed the parameter for the transcoding stream
- changed the code such that plus-decoding is done before percent-decoding as
stated in the docs
- added a note about the assumptions on the input to be ascii-only
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uri_module/+merge/96005
Yo
Two corrections:
- The transcode::stream is given an explicit source encoding of ISO-8859-1.
Surely it should be passed lCharset?
- The XQDoc for uri:decode() says that '+' will be replaced with ' ' before
percent-decoding. It is in fact done afterwards. I think actually it should be
done fi
The attempt to merge lp:~zorba-coders/zorba/feature-uri_module into lp:zorba
failed. Below is the output from the failed tests.
CMake Error at /home/ceej/zo/testing/zorbatest/tester/TarmacLander.cmake:274
(message):
Validation queue job feature-uri_module-2012-02-24T03-31-06.652Z is
finishe
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/feature-uri_module/+merge/94492
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/~zo
10 matches
Mail list logo
