Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Ghislain Fourny
Hi Matthias, I am not sure I understand what you are suggesting. The reason for non-const is that temporary parse nodes (in that case a lookup expression node) use regular pointers that they own to child parse nodes, i.e., the destructor deletes the object pointed to at the end. However, these

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Ghislain Fourny
After looking more in depth, I realized that there is an inconsistency across parsenodes. Some of them have const exprnode* pointers on their children (like JSONObjectLookup had), some others regular exprnode* pointers (like JSONObjectLookup has now). The latter seem to correspond to places

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Stage BuildZorbaUbuntu failed. Check compiler output at http://jenkins.lambda.nu/job/BuildZorbaUbuntu/336/parsed_console to view the results. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Your

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Ghislain Fourny
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 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/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Voting criteria failed for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 : Votes: {'Approve': 2, 'Needs commit message': 1} -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Your team Zorba Coders is

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Your

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Stage TestZorbaUbuntu failed. 1 tests failed (8490 total tests run). Check test results at http://jenkins.lambda.nu/job/TestZorbaUbuntu/296/testReport/ to view the results. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Your

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Stage TestZorbaUbuntu failed. 2 tests failed (8490 total tests run). Check test results at http://jenkins.lambda.nu/job/TestZorbaUbuntu/297/testReport/ to view the results. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Your

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue result for https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Stage TestZorbaUbuntu failed. 1 tests failed (8490 total tests run). Check test results at http://jenkins.lambda.nu/job/TestZorbaUbuntu/298/testReport/ to view the results. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Your

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue starting for the following merge proposals: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Progress dashboard at http://jenkins.lambda.nu/view/ValidationQueue -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 Your

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-update-syntax into lp:zorba

2013-09-13 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/new-update-syntax into lp:zorba has been updated. Status: Work in progress = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-update-syntax/+merge/184308 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba

2013-09-13 Thread Zorba Build Bot
Validation queue succeeded - proposal merged! -- https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 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/new-jsoniq-update into lp:zorba

2013-09-13 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/new-jsoniq-update into lp:zorba has been updated. Status: Needs review = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/new-jsoniq-update/+merge/185295 --

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

2013-09-13 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~zorba-coders/zorba/diagnostic-namespaces into lp:zorba. Commit message: You can now add more namespaces/prefixes to diagnostics_*.xml files. The upshot is that core modules can have errors in their own namespaces. Requested reviews: Paul J. Lucas

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

2013-09-13 Thread Paul J. Lucas
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/diagnostic-namespaces/+merge/185610 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 :