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
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
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.
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
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
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
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
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
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
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
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
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
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:
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-
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
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
** 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
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
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
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
20 matches
Mail list logo