commit 9c4b677fa582919a645f072045f508dea41210f3
author Tobias Thierer <tobiast@google.com> Thu Apr 21 14:59:13 2016 +0100
committer Tobias Thierer <tobiast@google.com> Thu Apr 21 21:16:38 2016 +0100
tree 8940c6fa9536b15f4082237deae5a9fb5845dcda
parent d5a2e6ec5cad49c864f085950c9d8da74736ec21

frameworks/ml: Avoid allocations that are immediately unboxed.

Outside specifically of Longs, I haven't checked carefully
whether there are other similar occurences in frameworks/ml.

Bug: 28289401
Change-Id: I891f9c69b80fa86fb0bb2863312f5cca836a2ede

bordeaux/service/src/android/bordeaux/services/ClusterManager.java

diff --git a/bordeaux/service/src/android/bordeaux/services/ClusterManager.java b/bordeaux/service/src/android/bordeaux/services/ClusterManager.java
index 4f0dd68..71e77a3 100644
--- a/bordeaux/service/src/android/bordeaux/services/ClusterManager.java
+++ b/bordeaux/service/src/android/bordeaux/services/ClusterManager.java
@@ -262,9 +262,9 @@
             mSemanticClusters.clear();
             for (Map<String, String> map : allData) {
                 String semanticId = map.get(SEMANTIC_ID);
-                double longitude = Double.valueOf(map.get(SEMANTIC_LONGITUDE));
-                double latitude = Double.valueOf(map.get(SEMANTIC_LATITUDE));
-                long duration = Long.valueOf(map.get(SEMANTIC_DURATION));
+                double longitude = Double.parseDouble(map.get(SEMANTIC_LONGITUDE));
+                double latitude = Double.parseDouble(map.get(SEMANTIC_LATITUDE));
+                long duration = Long.parseLong(map.get(SEMANTIC_DURATION));
                 BaseCluster cluster =
                     new BaseCluster(semanticId, longitude, latitude, duration);
 

bordeaux/service/src/android/bordeaux/services/Predictor.java

diff --git a/bordeaux/service/src/android/bordeaux/services/Predictor.java b/bordeaux/service/src/android/bordeaux/services/Predictor.java
index b3c0d45..9d9047a 100644
--- a/bordeaux/service/src/android/bordeaux/services/Predictor.java
+++ b/bordeaux/service/src/android/bordeaux/services/Predictor.java
@@ -147,7 +147,7 @@
             }
         } else if (key.equals(USE_HISTORY)) {
             mUseHistory = true;
-            mHistorySpan = Long.valueOf(value);
+            mHistorySpan = Long.parseLong(value);
         } else {
             Log.e(TAG,"Setting parameter " + key + " with " + value + " is not valid");
         }