Log on debug status only. Use BuildConfig.DEBUG to open it.

Author: MikeAfc

gradle.properties

@@ -45,7 +45,7 @@ org.gradle.jvmargs=-Xmx2048m
 GROUP=com.alibaba.android
 ARTIFACT=virtualview
 VERSION=1
-VERSION_NAME=1.4.4
+VERSION_NAME=1.4.5
 PACKAGING_TYPE=aar
 systemProp.compileSdkVersion=26
 systemProp.targetSdkVersion=26

virtualview/src/main/java/com/tmall/wireless/vaf/virtualview/ViewFactory.java

@@ -35,6 +35,7 @@
 import android.util.DisplayMetrics;
 import android.util.Log;
 import android.util.SparseArray;
+
 import com.libra.TextUtils;
 import com.libra.virtualview.common.Common;
 import com.tmall.wireless.vaf.framework.VafContext;
@@ -76,6 +77,7 @@
 import com.tmall.wireless.vaf.virtualview.view.text.NativeText;
 import com.tmall.wireless.vaf.virtualview.view.text.VirtualText;
 import com.tmall.wireless.vaf.virtualview.view.vh.VH;
+import com.tmall.wireless.virtualview.BuildConfig;
 
 /**
  * Created by gujicheng on 16/8/16.
@@ -99,6 +101,7 @@ public class ViewFactory {
         mLoader.setUiCodeManager(mUiCodeLoader);
         mLoader.setExprCodeManager(mExprCodeLoader);
     }
+
     private SparseArray<ViewBase.IBuilder> mBuilders = new SparseArray<>();
 
     private VafContext mAppContext;
@@ -199,7 +202,9 @@ public void loadBinBufferAsync(String type, byte[] buf) {
     public void loadBinBufferAsync(String type, byte[] buf, boolean override) {
         if (!TextUtils.isEmpty(type) && buf != null) {
             if (mTmplWorker.isRunning()) {
-                Log.d(TAG, "load " + type + " start ");
+                if (BuildConfig.DEBUG) {
+                    Log.d(TAG, "load " + type + " start ");
+                }
                 mTmplWorker.offerTask(new TmplTask(type, buf, override));
             }
         }
@@ -298,7 +303,7 @@ public ViewBase newView(String type, SparseArray<ViewBase> uuidContainers) {
                             if (null != view) {
                                 Layout.Params p;
                                 if (null != curView) {
-                                    p = ((Layout)curView).generateParams();
+                                    p = ((Layout) curView).generateParams();
                                     mComArr.push(curView);
                                 } else {
                                     p = new Layout.Params();
@@ -417,7 +422,7 @@ public ViewBase newView(String type, SparseArray<ViewBase> uuidContainers) {
                     ret = curView;
                     cr.seek(Common.TAG.length() + 4);
                     int version = cr.readShort();
-                    ret.setVersion(version );
+                    ret.setVersion(version);
                 }
             } else {
                 Log.e(TAG, "can not find component type:" + type);

virtualview/src/main/java/com/tmall/wireless/vaf/virtualview/loader/BinaryLoader.java

@@ -29,6 +29,7 @@
 
 import com.libra.virtualview.common.Common;
 import com.tmall.wireless.vaf.framework.VafContext;
+import com.tmall.wireless.virtualview.BuildConfig;
 
 import java.io.FileInputStream;
 import java.io.FileNotFoundException;
@@ -158,7 +159,9 @@ public int loadFromBuffer(byte[] buf, boolean override) {
                                     Log.e(TAG, "mStringManager is null");
                                 }
                             } else {
-                                Log.e(TAG, "string pos error:" + strStartPos + "  read pos:" + reader.getPos());
+                                if (BuildConfig.DEBUG) {
+                                    Log.e(TAG, "string pos error:" + strStartPos + "  read pos:" + reader.getPos());
+                                }
                             }
 
                             // parse expr
@@ -169,13 +172,17 @@ public int loadFromBuffer(byte[] buf, boolean override) {
                                     Log.e(TAG, "mExprCodeStore is null");
                                 }
                             } else {
-                                Log.e(TAG, "expr pos error:" + exprCodeStartPos + "  read pos:" + reader.getPos());
+                                if (BuildConfig.DEBUG) {
+                                    Log.e(TAG, "expr pos error:" + exprCodeStartPos + "  read pos:" + reader.getPos());
+                                }
                             }
 
                             // load extra data
                             if (reader.getPos() == extraStartPos) {
                             } else {
-                                Log.e(TAG, "extra pos error:" + extraStartPos + "  read pos:" + reader.getPos());
+                                if (BuildConfig.DEBUG) {
+                                    Log.e(TAG, "extra pos error:" + extraStartPos + "  read pos:" + reader.getPos());
+                                }
                             }
 
                             if (result) {

virtualview/src/main/java/com/tmall/wireless/vaf/virtualview/loader/StringLoader.java

@@ -28,9 +28,11 @@
 import java.util.concurrent.ConcurrentHashMap;
 
 import android.util.Log;
+
 import com.libra.TextUtils;
 import com.libra.expr.common.StringSupport;
 import com.libra.virtualview.common.StringBase;
+import com.tmall.wireless.virtualview.BuildConfig;
 
 /**
  * Created by gujicheng on 16/8/3.
@@ -108,7 +110,9 @@ public String getString(int id) {
         if (mIndex2String.containsKey(id)) {
             return mIndex2String.get(id);
         }
-        Log.e(TAG, "getString null:" + id);
+        if (BuildConfig.DEBUG) {
+            Log.e(TAG, "getString null:" + id);
+        }
         return null;
     }