Merge pull request #293 from AppDevNext/lbtMaster

Fix poolable object leak context issue

Author: hannesa2

MPChartLib/src/main/java/com/github/mikephil/charting/jobs/AnimatedMoveViewJob.java

@@ -36,6 +36,15 @@ public static AnimatedMoveViewJob getInstance(ViewPortHandler viewPortHandler, f
     }
 
     public static void recycleInstance(AnimatedMoveViewJob instance){
+        // Clear reference avoid memory leak
+        instance.mViewPortHandler = null;
+        instance.xValue = 0f;
+        instance.yValue = 0f;
+        instance.mTrans = null;
+        instance.view = null;
+        instance.xOrigin = 0f;
+        instance.yOrigin = 0f;
+        instance.animator.setDuration(0);
         pool.recycle(instance);
     }
 

MPChartLib/src/main/java/com/github/mikephil/charting/jobs/MoveViewJob.java

@@ -31,6 +31,12 @@ public static MoveViewJob getInstance(ViewPortHandler viewPortHandler, float xVa
 
     public static void recycleInstance(MoveViewJob instance){
         instance.recycle();
+        // Clear reference avoid memory leak
+        instance.mViewPortHandler = null;
+        instance.xValue = 0f;
+        instance.yValue = 0f;
+        instance.mTrans = null;
+        instance.view = null;
         pool.recycle(instance);
     }
 

MPChartLib/src/main/java/com/github/mikephil/charting/jobs/ZoomJob.java

@@ -37,6 +37,15 @@ public static ZoomJob getInstance(ViewPortHandler viewPortHandler, float scaleX,
     }
 
     public static void recycleInstance(ZoomJob instance) {
+        // Clear reference avoid memory leak
+        instance.xValue = 0f;
+        instance.yValue = 0f;
+        instance.scaleX = 0f;
+        instance.scaleY = 0f;
+        instance.mViewPortHandler = null;
+        instance.mTrans = null;
+        instance.axisDependency = null;
+        instance.view = null;
         pool.recycle(instance);
     }
 

MPChartLib/src/main/java/com/github/mikephil/charting/utils/ObjectPool.java

@@ -117,6 +117,7 @@ public synchronized T get(){
         }
 
         T result = (T)objects[this.objectsPointer];
+        objects[this.objectsPointer] = null;
         result.currentOwnerId = Poolable.NO_OWNER;
         this.objectsPointer--;