Report a new location only after receiving a non-null update

Signed-off-by: mvglasow <michael -at- vonglasow.com>
mvglasow · Mar 17, 2016 · e8158cb · e8158cb
1 parent bcf1cff
commit e8158cb
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 5 deletions.
diff --git a/unifiednlp-base/src/main/java/org/microg/nlp/location/BackendFuser.java b/unifiednlp-base/src/main/java/org/microg/nlp/location/BackendFuser.java
@@ -78,12 +78,15 @@ public void bind() {
     }
 
     public void update() {
+        Boolean hasUpdates = false;
         fusing = true;
         for (BackendHelper handler : backendHelpers) {
-            handler.update();
+            if (handler.update() != null)
+                hasUpdates = true;
         }
         fusing = false;
-        updateLocation();
+        if (hasUpdates)
+            updateLocation();
     }
 
     void updateLocation() {

diff --git a/unifiednlp-base/src/main/java/org/microg/nlp/location/BackendHelper.java b/unifiednlp-base/src/main/java/org/microg/nlp/location/BackendHelper.java
@@ -54,20 +54,31 @@ public Location getLastLocation() {
         return lastLocation;
     }
 
-    public void update() {
+    /**
+     * @brief Requests a location update from the backend.
+     * 
+     * @return The location reported by the backend. This may be null if a backend cannot determine its
+     * location, or if it is going to return a location asynchronously.
+     */
+    public Location update() {
+        Location result = null;
         if (backend == null) {
             Log.d(TAG, "Not (yet) bound.");
             updateWaiting = true;
         } else {
             updateWaiting = false;
             try {
-                setLastLocation(backend.update());
-                backendFuser.reportLocation();
+                result = backend.update();
+                if (result != null) {
+                    setLastLocation(result);
+                    backendFuser.reportLocation();
+                }
             } catch (Exception e) {
                 Log.w(TAG, e);
                 unbind();
             }
         }
+        return result;
     }
 
     private void setLastLocation(Location location) {
@@ -139,6 +150,8 @@ public void onServiceDisconnected(ComponentName name) {
     private class Callback extends LocationCallback.Stub {
         @Override
         public void report(Location location) throws RemoteException {
+            if (location == null)
+                return;
             setLastLocation(location);
             backendFuser.reportLocation();
         }