[Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-950456 into lp:zorba

2012-03-08 Thread Paul J. Lucas
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/bug-950456 into 
lp:zorba.

Requested reviews:
  Paul J. Lucas (paul-lucas)
Related bugs:
  Bug #950456 in Zorba: Memory leak in http-client module
  https://bugs.launchpad.net/zorba/+bug/950456

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693

Added delete theSerStream;
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'modules/com/zorba-xquery/www/modules/http-client.xq.src/http_request_handler.cpp'
--- modules/com/zorba-xquery/www/modules/http-client.xq.src/http_request_handler.cpp	2011-06-14 14:21:49 +
+++ modules/com/zorba-xquery/www/modules/http-client.xq.src/http_request_handler.cpp	2012-03-09 01:04:20 +
@@ -39,6 +39,7 @@
   theSerStream(NULL),
   thePost(NULL),
   theLast(NULL),
+  theLastSerializerOptions(NULL),
   theIsHeadRequest(false)
   {
 theHeaderLists.push_back(NULL);
@@ -57,6 +58,7 @@
 if (thePost != NULL) {
   curl_formfree(thePost);
 }
+delete theSerStream;
   }
 
   void HttpRequestHandler::begin()
@@ -288,3 +290,4 @@
 
 } // namespace http_client
 } // namespace zorba
+/* vim:set et sw=2 ts=2: */

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-950456 into lp:zorba

2012-03-08 Thread Paul J. Lucas
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
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 help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-950456 into lp:zorba

2012-03-08 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
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 help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-950456 into lp:zorba

2012-03-08 Thread Matthias Brantner
The proposal to merge lp:~paul-lucas/zorba/bug-950456 into lp:zorba has been 
updated.

Status: Needs review = Approved

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
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 help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-950456 into lp:zorba

2012-03-08 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/bug-950456-2012-03-09T01-14-13.39Z/log.html
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
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 help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-950456 into lp:zorba

2012-03-08 Thread Zorba Build Bot
Validation queue job bug-950456-2012-03-09T01-14-13.39Z is finished. The final 
status was:

All tests succeeded!
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
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 help   : https://help.launchpad.net/ListHelp


[Zorba-coders] [Merge] lp:~paul-lucas/zorba/bug-950456 into lp:zorba

2012-03-08 Thread noreply
The proposal to merge lp:~paul-lucas/zorba/bug-950456 into lp:zorba has been 
updated.

Status: Approved = Merged

For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
-- 
https://code.launchpad.net/~paul-lucas/zorba/bug-950456/+merge/96693
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 help   : https://help.launchpad.net/ListHelp