[Zorba-coders] [Merge] lp:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

2012-09-26 Thread noreply
The proposal to merge lp:~zorba-coders/zorba/fix-leak_flwor_clause into 
lp:zorba has been updated.

Status: Approved => Merged

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
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:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

2012-09-26 Thread Zorba Build Bot
Validation queue job fix-leak_flwor_clause-2012-09-26T20-25-40.972Z is 
finished. The final status was:

All tests succeeded!
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
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:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

2012-09-26 Thread Markos Zaharioudakis
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
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:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

2012-09-26 Thread Zorba Build Bot
Validation queue starting for merge proposal.
Log at: 
http://zorbatest.lambda.nu:8080/remotequeue/fix-leak_flwor_clause-2012-09-26T20-25-40.972Z/log.html
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
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:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

2012-09-26 Thread Matthias Brantner
The proposal to merge lp:~zorba-coders/zorba/fix-leak_flwor_clause into 
lp:zorba has been updated.

Status: Needs review => Approved

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
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:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

2012-09-26 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
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:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

2012-09-26 Thread Matthias Brantner
Matthias Brantner has proposed merging 
lp:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba.

Commit message:
fix memory leak for test/rbkt/zorba/flwor/flwor18 by adding a virtual 
destructor to the flwor_clause.

Requested reviews:
  Matthias Brantner (matthias-brantner)
  Markos Zaharioudakis (markos-za)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
-- 
https://code.launchpad.net/~zorba-coders/zorba/fix-leak_flwor_clause/+merge/126539
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'src/compiler/expression/flwor_expr.cpp'
--- src/compiler/expression/flwor_expr.cpp	2012-09-19 18:18:02 +
+++ src/compiler/expression/flwor_expr.cpp	2012-09-26 20:24:22 +
@@ -38,8 +38,15 @@
 namespace zorba
 {
 
+/***
+
+/
 DEF_EXPR_ACCEPT (flwor_expr)
 
+flwor_clause::~flwor_clause()
+{
+}
+
 
 /***
 

=== modified file 'src/compiler/expression/flwor_expr.h'
--- src/compiler/expression/flwor_expr.h	2012-09-19 18:18:02 +
+++ src/compiler/expression/flwor_expr.h	2012-09-26 20:24:22 +
@@ -89,6 +89,8 @@
   }
 
 public:
+  virtual ~flwor_clause();
+
   const QueryLoc& get_loc() const { return theLocation; }
 
   ClauseKind get_kind() const { return theKind; }

-- 
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