Merge pull request #380 from AppDevNext/qiduhe/master

Simplify View reference memory leak

Author: hannesa2

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

@@ -37,14 +37,12 @@ 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);
+        instance.recycle();
         pool.recycle(instance);
     }
 

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

@@ -32,11 +32,8 @@ 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

@@ -42,10 +42,8 @@ public static void recycleInstance(ZoomJob instance) {
         instance.yValue = 0f;
         instance.scaleX = 0f;
         instance.scaleY = 0f;
-        instance.mViewPortHandler = null;
-        instance.mTrans = null;
         instance.axisDependency = null;
-        instance.view = null;
+        instance.recycle();
         pool.recycle(instance);
     }