summaryrefslogtreecommitdiffstats
path: root/src/com/android
diff options
context:
space:
mode:
authorAndreas Gampe <agampe@google.com>2018-01-22 21:24:48 -0800
committerLuK1337 <priv.luk@gmail.com>2019-10-21 21:40:56 +0200
commit8986bc6480e25376e58f8c65be87d5f2cfb422ae (patch)
tree7030f4242ef637d450f7c755ffdde0cb082285d2 /src/com/android
parentc8812868800de388e4464c1d01c9ecac2e5de5bb (diff)
downloadandroid_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.java2
-rw-r--r--src/com/android/gallery3d/filtershow/imageshow/EclipseControl.java6
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;