Review: Needs Information
General question: Is there a reason you created ZorbaIOStream rather
than using normal C++ iostreams? It seems like re-inventing the wheel,
and it also makes it harder to use this feature with streams from
other sources.
Also, comments in swig/Item.h are inaccurate -
a
The proposal to merge lp:~zorba-coders/zorba/doxygen_xml_generation into
lp:zorba has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen_xml_generation/+merge/121349
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen
Validation queue job doxygen_xml_generation-2012-08-28T00-20-57.476Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen_xml_generation/+merge/121349
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347
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/~zorba-
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/doxygen_xml_generation-2012-08-28T00-20-57.476Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen_xml_generation/+merge/121349
Your team Zorba Coders is subscribed to branch lp:zorba
The proposal to merge lp:~zorba-coders/zorba/doxygen_xml_generation into
lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/doxygen_xml_generation/+merge/121349
--
https://code.launchpad.net/~zorba-coders/zorba/d
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen_xml_generation/+merge/121349
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.ne
The proposal to merge lp:~zorba-coders/zorba/bug-1040558 into lp:zorba has been
updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/120956
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/120956
You
Validation queue job bug-1040558-2012-08-27T20-44-57.648Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/120956
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~zorba-c
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/bug-1040558-2012-08-27T20-44-57.648Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/120956
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: h
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/120956
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/~zorba-co
The proposal to merge lp:~zorba-coders/zorba/bug-1040558 into lp:zorba has been
updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/120956
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/1209
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bug-1040558/+merge/120956
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/~zorba-co
The proposal to merge lp:~zorba-coders/zorba/jsoniq-static-casts into lp:zorba
has been updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-static-casts/+merge/118961
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-
Voting does not meet specified criteria. Required: Approve > 1, Disapprove < 1,
Needs Fixing < 1, Pending < 1. Got: 1 Approve, 2 Pending.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-static-casts/+merge/118961
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list:
Validation queue job jsoniq-static-casts-2012-08-27T16-42-57.41Z is finished.
The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-static-casts/+merge/118961
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchp
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/jsoniq-static-casts-2012-08-27T16-42-57.41Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-static-casts/+merge/118961
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Review: Approve
JSONItem::unfix contains a "return" statement before an assertion. Not sure if
that's on purpose.
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-static-casts/+merge/118961
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/
The proposal to merge lp:~zorba-coders/zorba/jsoniq-static-casts into lp:zorba
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-static-casts/+merge/118961
--
https://code.launchpad.net/~zorba-coders/zorba/jsoniq-
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/doxygen_xml_generation/+merge/121349
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.ne
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347
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/~zorba-
The proposal to merge lp:~zorba-coders/zorba/structural-any-uri-lazy into
lp:zorba has been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
--
https://code.launchpad.net/~zorba-coders/zorba/struc
Validation queue job structural-any-uri-lazy-2012-08-27T15-25-59.316Z is
finished. The final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https:
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/structural-any-uri-lazy-2012-08-27T15-25-59.316Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
Your team Zorba Coders is subscribed to branch lp:zor
The proposal to merge lp:~zorba-coders/zorba/structural-any-uri-lazy into
lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
--
https://code.launchpad.net/~zorba-coders/zorba
The proposal to merge lp:~zorba-coders/zorba/structural-any-uri-lazy into
lp:zorba has been updated.
Status: Approved => Needs review
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
--
https://code.launchpad.net/~zorba-coders/zorba
The proposal to merge lp:~zorba-coders/zorba/structural-any-uri-lazy into
lp:zorba has been updated.
Commit Message changed to:
Made URI computation lazy in StructuralAnyUri.
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
--
https://
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_867094/+merge/121405
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/~zorba
Sorin Marian Nasoi has proposed merging lp:~zorba-coders/zorba/fix_bug_867094
into lp:zorba.
Requested reviews:
Zorba Coders (zorba-coders)
Related bugs:
Bug #867094 in Zorba: "XQTSX failures due to W3C bugs"
https://bugs.launchpad.net/zorba/+bug/867094
For more details, see:
https://code.
The proposal to merge lp:~zorba-coders/zorba/fix_bug_969251 into lp:zorba has
been updated.
Status: Approved => Merged
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_969251/+merge/108679
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_969251/+merge/1
Validation queue job fix_bug_969251-2012-08-27T09-56-00.342Z is finished. The
final status was:
All tests succeeded!
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_969251/+merge/108679
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing list: https://launchpad.net/~z
Validation queue starting for merge proposal.
Log at:
http://zorbatest.lambda.nu:8080/remotequeue/fix_bug_969251-2012-08-27T09-56-00.342Z/log.html
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_969251/+merge/108679
Your team Zorba Coders is subscribed to branch lp:zorba.
--
Mailing l
The proposal to merge lp:~zorba-coders/zorba/fix_bug_969251 into lp:zorba has
been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_969251/+merge/108679
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_969251/+m
Review: Approve
Thanks Sorin ;-)
--
https://code.launchpad.net/~zorba-coders/zorba/fix_bug_969251/+merge/108679
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://laun
The proposal to merge lp:~zorba-coders/zorba/structural-any-uri-lazy into
lp:zorba has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
--
https://code.launchpad.net/~zorba-coders/zorba
Review: Approve
--
https://code.launchpad.net/~zorba-coders/zorba/structural-any-uri-lazy/+merge/104918
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.n
Hi Ghislain,
The point of the non-const param is to call take() inside the constructor, thus
saving a (synchronized) ref counting. I have now added this take() call in the
StructuralAnyURIItem constructor.
You are right about the static casts. I thought that getAnyUriTypeCode() was a
zorba::It
37 matches
Mail list logo