Simplify previous fix due to root cause fixing

The fix for #1680 introduced in ad93745d42f9aaf5bc4925075987ef64c13b2f71
had to account for the possibility of angles being outside the [0, 360[
range.

Commit 0034e358668d5911639628468be7409b2696d6c5 fixed #1685 and now
ensures that the returned orientation is within the right domain range
even after the phone rotation has been added to the sensor angle.

It is now possible to simplify the computation because we now that the
difference of two angle values will always be in the ]-360; 720[ range,
and can this fixed by adding 360 and taking the module with 360 to enter
back the [0, 360[ range.

1 files changed, 2 insertions, 8 deletions

diff --git a/main/src/cgeo/geocaching/ui/CompassView.java b/main/src/cgeo/geocaching/ui/CompassView.java
index 5401bfd..6f5b8b9 100644
--- a/main/src/cgeo/geocaching/ui/CompassView.java
+++ b/main/src/cgeo/geocaching/ui/CompassView.java
@@ -128,10 +128,7 @@ public class CompassView extends View {
      * @return the new value
      */
     static protected double smoothUpdate(double goal, double actual) {
-        double diff = (goal - actual) % 360;
-        if (diff < 0.0) {
-            diff += 360.0;
-        }
+        final double diff = (goal - actual + 360) % 360;
 
         double offset = 0.0;
 
@@ -179,10 +176,7 @@ public class CompassView extends View {
         }
 
         double azimuthTemp = azimuthDrawn;
-        double azimuthRelative = (azimuthTemp - headingDrawn) % 360;
-        if (azimuthRelative < 0) {
-            azimuthRelative += 360;
-        }
+        final double azimuthRelative = (azimuthTemp - headingDrawn + 360) % 360;
 
         // compass margins
         int canvasCenterX = (compassRoseWidth / 2) + ((getWidth() - compassRoseWidth) / 2);