Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1047547 into lp:zorba

2012-09-11 Thread Ghislain Fourny
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1047547 into lp:zorba

2012-09-11 Thread Ghislain Fourny
The proposal to merge lp:~zorba-coders/zorba/bug-1047547 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1047547 into lp:zorba

2012-09-11 Thread Zorba Build Bot
Attempt to merge into lp:zorba failed due to conflicts: text conflict in ChangeLog -- https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

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

2012-09-11 Thread Paul J. Lucas
Why does ztd::alloc_sizeof( theKeys ) in src/store/naive/json_items.cpp:245 return 0? Because a specialization for HashMap wasn't ever defined. I really wish we'd get away from using our own non-standard hashmaps. There really is no point. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1047547 into lp:zorba

2012-09-11 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/bug-1047547 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1047547 into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1047547-2012-09-11T15-10-43.456Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1047547 into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue job bug-1047547-2012-09-11T15-10-43.456Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1047547 into lp:zorba

2012-09-11 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1047547 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1047547/+merge/123396

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-1046415 into lp:zorba

2012-09-11 Thread Matthias Brantner
Review: Needs Information - Is the empty string a valid name? Should we raise an error if the name of a pair is an empty string? Currently, the following query works fine: { : 1}() - What should happen if the key expression evaluates to null. Currently, the following query returns 1; { null :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1046411 into lp:zorba

2012-09-11 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-1046411/+merge/123654 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1046411 into lp:zorba

2012-09-11 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/bug-1046411 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1046411/+merge/123654 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1046411 into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bug-1046411-2012-09-11T16-04-36.756Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bug-1046411/+merge/123654 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1046411 into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue job bug-1046411-2012-09-11T16-04-36.756Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bug-1046411/+merge/123654 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-1046411 into lp:zorba

2012-09-11 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bug-1046411 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-1046411/+merge/123654 -- https://code.launchpad.net/~zorba-coders/zorba/bug-1046411/+merge/123654

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2012-09-11 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/pjl-misc into lp:zorba. Requested reviews: Paul J. Lucas (paul-lucas) For more details, see: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 Fixed pragma warning. --

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2012-09-11 Thread Paul J. Lucas
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 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-coders

Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2012-09-11 Thread Matthias Brantner
Review: Approve -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 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-coders

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2012-09-11 Thread Matthias Brantner
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 Your team

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/pjl-misc-2012-09-11T18-11-38.544Z/log.html -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue job pjl-misc-2012-09-11T18-11-38.544Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders

[Zorba-coders] [Merge] lp:~paul-lucas/zorba/pjl-misc into lp:zorba

2012-09-11 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 -- https://code.launchpad.net/~paul-lucas/zorba/pjl-misc/+merge/123802 Your team Zorba

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-09-11 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bang_operator-2012-09-11T23-04-10.294Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-09-11 Thread Zorba Build Bot
The attempt to merge lp:~zorba-coders/zorba/bang_operator 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 bang_operator-2012-09-11T23-04-10.294Z is finished. The

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bang_operator into lp:zorba

2012-09-11 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/bang_operator into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bang_operator/+merge/121347 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-950621 into lp:zorba

2012-09-11 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 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 :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bug-950621 into lp:zorba

2012-09-11 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bug-950621 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655 -- https://code.launchpad.net/~zorba-coders/zorba/bug-950621/+merge/123655

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-1046415 into lp:zorba

2012-09-11 Thread Chris Hillery
As far as I can make out, an empty string is a valid key for an object. jsonlint.com at least agrees. null is not a valid key, and we should raise an error if such a case arises. Also, the fact that using the *string* null as an accessor worked in that case frightens me. --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba

2012-09-11 Thread Chris Hillery
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/bugs-1022495-and-1022494/+merge/114185 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 :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba

2012-09-11 Thread Chris Hillery
Review: Needs Fixing More modules need changing, coming soon. -- https://code.launchpad.net/~zorba-coders/zorba/bugs-1022495-and-1022494/+merge/114185 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-1046415 into lp:zorba

2012-09-11 Thread Matthias Brantner
As far as I can make out, an empty string is a valid key for an object. jsonlint.com at least agrees. Fair enough. null is not a valid key, and we should raise an error if such a case arises. Also, the fact that using the *string* null as an accessor worked in that case frightens me.

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba

2012-09-11 Thread Chris Hillery
Review: Approve Should be all done now. -- https://code.launchpad.net/~zorba-coders/zorba/bugs-1022495-and-1022494/+merge/114185 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba

2012-09-11 Thread Chris Hillery
The proposal to merge lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bugs-1022495-and-1022494/+merge/114185 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bugs-1022495-and-1022494-2012-09-12T04-38-44.886Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/bugs-1022495-and-1022494/+merge/114185 Your team Zorba Coders is subscribed to branch

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba

2012-09-11 Thread Zorba Build Bot
Validation queue job bugs-1022495-and-1022494-2012-09-12T04-38-44.886Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/bugs-1022495-and-1022494/+merge/114185 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba

2012-09-11 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/bugs-1022495-and-1022494 into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/bugs-1022495-and-1022494/+merge/114185 --