Re: [Zorba-coders] [Merge] lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module

2012-07-03 Thread David Graf
David - Matthias is on vacation this week, so if possible I'd like to review this change instead and get it merged. That said - can you explain to me the central part of the change, which is the Base64 dance done inside of if (lContainsNonAscii) ? Better yet, please put a comment there

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Ghislain Fourny
Ghislain Fourny has proposed merging lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba. Requested reviews: Ghislain Fourny (gislenius) Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 Fixed a

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Ghislain Fourny
The proposal to merge lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba has been updated. Commit Message changed to: Fixed a bug preventing from compiling Zorba single-threadedly. For more details, see: https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Ghislain Fourny
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 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/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Ghislain Fourny
The proposal to merge lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/sync-code-hashmap-fix-2012-07-03T09-02-57.271Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 Your team Zorba Coders is subscribed to branch lp:zorba.

Re: [Zorba-coders] [Merge] lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module

2012-07-03 Thread Dennis Knochenwefel
Review: Approve -- https://code.launchpad.net/~davidagraf/zorba/bug-867248/+merge/113059 Your team Zorba Coders is subscribed to branch lp:zorba/email-module. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-coders@lists.launchpad.net Unsubscribe :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue job sync-code-hashmap-fix-2012-07-03T09-02-57.271Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Zorba Build Bot
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/sync-code-hashmap-fix/+merge/113175 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Ghislain Fourny
The proposal to merge lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Ghislain Fourny
Thanks Markos, the function is now virtual ;-) -- https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to :

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/unregister-delete-or-destroy-2012-07-03T09-39-05.169Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 Your team Zorba Coders is subscribed to

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue job unregister-delete-or-destroy-2012-07-03T09-39-05.169Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve 1, Disapprove 1, Needs Fixing 1, Pending 1. Got: 1 Approve, 2 Needs Fixing. -- https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 Your team Zorba Coders is subscribed to branch lp:zorba. --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Zorba Build Bot
The proposal to merge lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba has been updated. Status: Approved = Needs review For more details, see: https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 --

[Zorba-coders] [Merge] lp:~davidagraf/zorba/windows-build-fix into lp:zorba

2012-07-03 Thread David Graf
David Graf has proposed merging lp:~davidagraf/zorba/windows-build-fix into lp:zorba. Requested reviews: David Graf (davidagraf) Dennis Knochenwefel (dennis-knochenwefel) For more details, see: https://code.launchpad.net/~davidagraf/zorba/windows-build-fix/+merge/113196 windows build fix

[Zorba-coders] [Merge] lp:~davidagraf/zorba/windows-build-fix into lp:zorba

2012-07-03 Thread David Graf
The proposal to merge lp:~davidagraf/zorba/windows-build-fix into lp:zorba has been updated. Commit Message changed to: windows build fix For more details, see: https://code.launchpad.net/~davidagraf/zorba/windows-build-fix/+merge/113196 --

Re: [Zorba-coders] [Merge] lp:~davidagraf/zorba/windows-build-fix into lp:zorba

2012-07-03 Thread David Graf
Review: Approve -- https://code.launchpad.net/~davidagraf/zorba/windows-build-fix/+merge/113196 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:~davidagraf/zorba/windows-build-fix into lp:zorba

2012-07-03 Thread Dennis Knochenwefel
Review: Approve -- https://code.launchpad.net/~davidagraf/zorba/windows-build-fix/+merge/113196 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:~davidagraf/zorba/windows-build-fix into lp:zorba

2012-07-03 Thread David Graf
The proposal to merge lp:~davidagraf/zorba/windows-build-fix into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~davidagraf/zorba/windows-build-fix/+merge/113196 --

[Zorba-coders] [Merge] lp:~davidagraf/zorba/windows-build-fix into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue job windows-build-fix-2012-07-03T11-15-09.303Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~davidagraf/zorba/windows-build-fix/+merge/113196 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~davidagraf/zorba/windows-build-fix into lp:zorba

2012-07-03 Thread noreply
The proposal to merge lp:~davidagraf/zorba/windows-build-fix into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~davidagraf/zorba/windows-build-fix/+merge/113196 --

Re: [Zorba-coders] [Merge] lp:~davidagraf/zorba/bug-867248 into lp:zorba/email-module

2012-07-03 Thread David Graf
I'm afraid I still don't understand what this encoding is doing. It sounds like if the subject of an email contains one non-ASCII character, that the entire subject line will be encoded as base-64. Is that really the intent? Won't that render it completely unreadable in most circumstances? I

[Zorba-coders] [Bug 1020588] [NEW] Illegal updating expressions

2012-07-03 Thread Christian Gruen
Public bug reported: a/, delete node a/ Expected: XUST0001 Returned: a/ Hope this helps, Christian ** Affects: zorba Importance: Undecided Status: New -- You received this bug notification because you are a member of Zorba Coders, which is the registrant for Zorba.

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 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/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 --

Re: [Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 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/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/unregister-delete-or-destroy-2012-07-03T15-05-59.471Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 Your team Zorba Coders is subscribed to

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Markos Zaharioudakis
The proposal to merge lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba has been updated. Status: Needs review = Approved For more details, see: https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue job unregister-delete-or-destroy-2012-07-03T15-05-59.471Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba

2012-07-03 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/unregister-delete-or-destroy into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/unregister-delete-or-destroy/+merge/100814 --

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/sync-code-hashmap-fix-2012-07-03T15-32-56.266Z/log.html -- https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 Your team Zorba Coders is subscribed to branch lp:zorba.

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread Zorba Build Bot
Validation queue job sync-code-hashmap-fix-2012-07-03T15-32-56.266Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

[Zorba-coders] [Merge] lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba

2012-07-03 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/sync-code-hashmap-fix into lp:zorba has been updated. Status: Approved = Merged For more details, see: https://code.launchpad.net/~zorba-coders/zorba/sync-code-hashmap-fix/+merge/113175 --

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

2012-07-03 Thread Rodolfo Ochoa
Review: Approve -- https://code.launchpad.net/~davidagraf/zorba/bug-1013075/+merge/110307 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:~davidagraf/zorba/bug-1013075 into lp:zorba

2012-07-03 Thread Rodolfo Ochoa
Review: Needs Fixing Is not working on my VS, I have: Error 146 error LNK1181: cannot open input file '..\..\src\Release\zorba_simplestore.lib' -- https://code.launchpad.net/~davidagraf/zorba/bug-1013075/+merge/110307 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing

[Zorba-coders] [Bug 979147] Re: test/rbkt/zorba/index/numbers fails in Release mode

2012-07-03 Thread Markos Zaharioudakis
There is no bug in zorba. This is due to g++ handling numbers and arithmetic ops differently, depending on the optimization level and/or platform. In fact, there are probably g++ bugs in this area. For example, the result changes if I move a printout statement above or below another line. **

[Zorba-coders] [Bug 1020588] Re: Illegal updating expressions

2012-07-03 Thread Chris Hillery
I confirm that the query produces the result Christian states. I believe he is correct that it should raise XUST0001 based on section 2.5.4 Comma Expression in the XQuery Update spec. ** Changed in: zorba Importance: Undecided = Medium ** Changed in: zorba Assignee: (unassigned) = Markos