diff options
author | Andreas Gampe <agampe@google.com> | 2018-01-22 21:24:48 -0800 |
---|---|---|
committer | LuK1337 <priv.luk@gmail.com> | 2019-10-21 21:40:56 +0200 |
commit | 8986bc6480e25376e58f8c65be87d5f2cfb422ae (patch) | |
tree | 7030f4242ef637d450f7c755ffdde0cb082285d2 /src/com/android | |
parent | c8812868800de388e4464c1d01c9ecac2e5de5bb (diff) | |
download | android_packages_apps_Gallery2-8986bc6480e25376e58f8c65be87d5f2cfb422ae.tar.gz android_packages_apps_Gallery2-8986bc6480e25376e58f8c65be87d5f2cfb422ae.tar.bz2 android_packages_apps_Gallery2-8986bc6480e25376e58f8c65be87d5f2cfb422ae.zip |
Gallery2: Fix NaN comparisons
Use Float.isNan for correct evaluation. Found by errorprone.
Test: m
Change-Id: I3d47a2a64059d9c89528c82a1ca766248db4bdf9
Diffstat (limited to 'src/com/android')
-rw-r--r-- | src/com/android/gallery3d/filtershow/filters/FilterVignetteRepresentation.java | 2 | ||||
-rw-r--r-- | src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/com/android/gallery3d/filtershow/filters/FilterVignetteRepresentation.java b/src/com/android/gallery3d/filtershow/filters/FilterVignetteRepresentation.java index cff923132..e90e12fb4 100644 --- a/src/com/android/gallery3d/filtershow/filters/FilterVignetteRepresentation.java +++ b/src/com/android/gallery3d/filtershow/filters/FilterVignetteRepresentation.java @@ -151,7 +151,7 @@ public class FilterVignetteRepresentation extends FilterRepresentation implement } public boolean isCenterSet() { - return mCenterX != Float.NaN; + return !Float.isNaN(mCenterX); } @Override diff --git a/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java b/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java index 3e6d0e71f..b67da4633 100644 --- a/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java +++ b/src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java @@ -189,7 +189,7 @@ public class EclipseControl { } public void paintGrayPoint(Canvas canvas, float x, float y) { - if (x == Float.NaN) { + if (Float.isNaN(x)) { return; } @@ -206,7 +206,7 @@ public class EclipseControl { } public void paintPoint(Canvas canvas, float x, float y) { - if (x == Float.NaN) { + if (Float.isNaN(x)) { return; } @@ -223,7 +223,7 @@ public class EclipseControl { } void paintRadius(Canvas canvas, float cx, float cy, float rx, float ry) { - if (cx == Float.NaN) { + if (Float.isNaN(cx)) { return; } int mSliderColor = 0xFF33B5E5; |