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

2011-09-28 Thread Federico Cavalieri
Federico Cavalieri has proposed merging lp:~fcavalieri/zorba/bugs into lp:zorba. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Updated Changelog to reflect fixes in revisions 10470, 10471, 10472, 10473

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

2011-09-28 Thread Federico Cavalieri
The proposal to merge lp:~fcavalieri/zorba/bugs into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is s

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

2011-09-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bugs-2011-09-28T07-02-16.793Z/log.html -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.

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

2011-09-28 Thread Zorba Build Bot
Validation queue job bugs-2011-09-28T07-02-16.793Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to

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

2011-09-28 Thread Zorba Build Bot
Voting does not meet specified criteria. Required: Approve > 0, Disapprove < 1. Got: 1 Pending. -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to : zorba-cod

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

2011-09-28 Thread Zorba Build Bot
The proposal to merge lp:~fcavalieri/zorba/bugs into lp:zorba has been updated. Status: Approved => Needs review For more details, see: https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is s

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

2011-09-28 Thread Markos Zaharioudakis
Review: Approve -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 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 More

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

2011-09-28 Thread Markos Zaharioudakis
The proposal to merge lp:~fcavalieri/zorba/bugs into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is s

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

2011-09-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/bugs-2011-09-28T08-25-42.98Z/log.html -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.n

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

2011-09-28 Thread Zorba Build Bot
Validation queue job bugs-2011-09-28T08-25-42.98Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-coders Post to

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

2011-09-28 Thread noreply
The proposal to merge lp:~fcavalieri/zorba/bugs into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 -- https://code.launchpad.net/~fcavalieri/zorba/bugs/+merge/77287 Your team Zorba Coders is subscri

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

2011-09-28 Thread Gabriel Petrovay
The proposal to merge lp:~gabipetrovay/zorba/curl_dll_fix into lp:zorba has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~gabipetrovay/zorba/curl_dll_fix/+merge/76978 -- https://code.launchpad.net/~gabipetrovay/zorba/curl_dll_fix/+merge/76

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

2011-09-28 Thread Zorba Build Bot
Validation queue starting for merge proposal. Log at: http://zorbatest.lambda.nu:8080/remotequeue/curl_dll_fix-2011-09-28T12-05-40.187Z/log.html -- https://code.launchpad.net/~gabipetrovay/zorba/curl_dll_fix/+merge/76978 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list:

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

2011-09-28 Thread Zorba Build Bot
Validation queue job curl_dll_fix-2011-09-28T12-05-40.187Z is finished. The final status was: All tests succeeded! -- https://code.launchpad.net/~gabipetrovay/zorba/curl_dll_fix/+merge/76978 Your team Zorba Coders is subscribed to branch lp:zorba. -- Mailing list: https://launchpad.net/~zorba-

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

2011-09-28 Thread noreply
The proposal to merge lp:~gabipetrovay/zorba/curl_dll_fix into lp:zorba has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~gabipetrovay/zorba/curl_dll_fix/+merge/76978 -- https://code.launchpad.net/~gabipetrovay/zorba/curl_dll_fix/+merge/76978 Yo

Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/external_func_param into lp:zorba

2011-09-28 Thread Markos Zaharioudakis
3 questions: 1. Should the destroy() method be called also when the param is replaced by a second invocation of the addExternalFunctionParam? 2. Should we check that the func_param is not NULL before calling its destroy() method? 3. Is there any need for a removeExternalFunctionParam()? For xq

[Zorba-coders] [Bug 855314] Re: Assertion failed in typemanager with for and el constuctor

2011-09-28 Thread Markos Zaharioudakis
** Changed in: zorba Status: Fix Committed => Fix Released -- You received this bug notification because you are a member of Zorba Coders, which is subscribed to the bug report. https://bugs.launchpad.net/bugs/855314 Title: Assertion failed in typemanager with for and el constuctor Sta

Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/external_func_param into lp:zorba

2011-09-28 Thread Markos Zaharioudakis
One more: The method dynamic_context::getExternalFunctionParameter(const std::string& aName) does not check the val.type. Why not? (The other getExternalFunctionParameter method does check the val.type). -- https://code.launchpad.net/~matthias-brantner/zorba/external_func_param/+merge/77255 You

[Zorba-coders] [Merge] lp:~matthias-brantner/zorba/external_func_param into lp:zorba

2011-09-28 Thread Matthias Brantner
Matthias Brantner has proposed merging lp:~matthias-brantner/zorba/external_func_param into lp:zorba. Requested reviews: Markos Zaharioudakis (markos-za) For more details, see: https://code.launchpad.net/~matthias-brantner/zorba/external_func_param/+merge/77410 Added ExternalFunctionParameter

Re: [Zorba-coders] [Merge] lp:~matthias-brantner/zorba/external_func_param into lp:zorba

2011-09-28 Thread Matthias Brantner
1. destroy is now called if the parameter is replaced 2. I put a check for func_param != NULL in place 3. It might be useful but the other hashmap causes some problems. For example, get is not constant and the dctx_value_t is currently not serializable. 4. I have put a check for value.type in p