zorba-coders team mailing list archive

[Ghislain Fourny <ghislain.fourny@xxxxxxxxxx>]

Ghislain Fourny has proposed merging lp:~zorba-coders/zorba/memory-leak-array-append into lp:zorba.

Requested reviews:
  Till Westmann (tillw)
  Matthias Brantner (matthias-brantner)

For more details, see:
https://code.launchpad.net/~zorba-coders/zorba/memory-leak-array-append/+merge/118960

Fixing a memory leak in append update primitive.
-- 
https://code.launchpad.net/~zorba-coders/zorba/memory-leak-array-append/+merge/118960
Your team Zorba Coders is subscribed to branch lp:zorba.

=== modified file 'src/store/naive/simple_pul.cpp'
--- src/store/naive/simple_pul.cpp	2012-07-24 08:48:48 +0000
+++ src/store/naive/simple_pul.cpp	2012-08-09 13:44:20 +0000
@@ -1737,6 +1737,10 @@
       mergeTargetedUpdateLists(thisPul,
                                thisPul->theJSONArrayReplaceValueList,
                                otherPul->theJSONArrayReplaceValueList);
+
+      mergeTargetedUpdateLists(thisPul,
+                               thisPul->theJSONArrayAppendList,
+                               otherPul->theJSONArrayAppendList);
 #endif
 
       ++thisIte;
@@ -2633,6 +2637,7 @@
   cleanList(theJSONArrayInsertList);
   cleanList(theJSONArrayDeleteList);
   cleanList(theJSONArrayReplaceValueList);
+  cleanList(theJSONArrayAppendList);
 #endif
 
   cleanIndexDeltas();
@@ -2661,6 +2666,7 @@
   switchPulInPrimitivesList(theJSONArrayInsertList);
   switchPulInPrimitivesList(theJSONArrayDeleteList);
   switchPulInPrimitivesList(theJSONArrayReplaceValueList);
+  switchPulInPrimitivesList(theJSONArrayAppendList);
 #endif
 
   switchPulInPrimitivesList(theCreateCollectionList);