Merge lp:~zorba-coders/zorba/fix-leak_flwor_clause into lp:zorba

Proposed by Matthias Brantner
Status: Merged
Approved by: Matthias Brantner
Approved revision: 11066
Merged at revision: 11066
Proposed branch: lp:~zorba-coders/zorba/fix-leak_flwor_clause
Merge into: lp:zorba
Diff against target: 32 lines (+9/-0)
2 files modified
src/compiler/expression/flwor_expr.cpp (+7/-0)
src/compiler/expression/flwor_expr.h (+2/-0)
To merge this branch: bzr merge lp:~zorba-coders/zorba/fix-leak_flwor_clause
Reviewer Review Type Date Requested Status
Markos Zaharioudakis Approve
Matthias Brantner Approve
Review via email: mp+126539@code.launchpad.net

Commit message

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

To post a comment you must log in.
Revision history for this message
Matthias Brantner (matthias-brantner) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :
Revision history for this message
Markos Zaharioudakis (markos-za) :
review: Approve
Revision history for this message
Zorba Build Bot (zorba-buildbot) wrote :

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

All tests succeeded!

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/compiler/expression/flwor_expr.cpp'
2--- src/compiler/expression/flwor_expr.cpp 2012-09-19 18:18:02 +0000
3+++ src/compiler/expression/flwor_expr.cpp 2012-09-26 20:24:22 +0000
4@@ -38,8 +38,15 @@
5 namespace zorba
6 {
7
8+/*******************************************************************************
9+
10+********************************************************************************/
11 DEF_EXPR_ACCEPT (flwor_expr)
12
13+flwor_clause::~flwor_clause()
14+{
15+}
16+
17
18 /*******************************************************************************
19
20
21=== modified file 'src/compiler/expression/flwor_expr.h'
22--- src/compiler/expression/flwor_expr.h 2012-09-19 18:18:02 +0000
23+++ src/compiler/expression/flwor_expr.h 2012-09-26 20:24:22 +0000
24@@ -89,6 +89,8 @@
25 }
26
27 public:
28+ virtual ~flwor_clause();
29+
30 const QueryLoc& get_loc() const { return theLocation; }
31
32 ClauseKind get_kind() const { return theKind; }

Subscribers

People subscribed via source and target branches