OSDN Git Service

Merge "Float.NaN != ... always evaluates to true, use Float.isNaN."
authorIan Rogers <irogers@google.com>
Fri, 18 Mar 2016 17:45:14 +0000 (17:45 +0000)
committerGerrit Code Review <noreply-gerritcodereview@google.com>
Fri, 18 Mar 2016 17:45:14 +0000 (17:45 +0000)
src/com/android/gallery3d/filtershow/colorpicker/ColorBrightnessView.java
src/com/android/gallery3d/filtershow/colorpicker/ColorHueView.java
src/com/android/gallery3d/filtershow/colorpicker/ColorOpacityView.java
src/com/android/gallery3d/filtershow/colorpicker/ColorRectView.java
src/com/android/gallery3d/filtershow/colorpicker/ColorSVRectView.java
src/com/android/gallery3d/filtershow/colorpicker/ColorSaturationView.java
src/com/android/gallery3d/filtershow/colorpicker/ColorValueView.java

index 9b955ef..b2118e0 100644 (file)
@@ -171,7 +171,7 @@ public class ColorBrightnessView extends View implements ColorListener {
         canvas.drawRect(mBorder, mBorder, mWidth - mBorder, mHeight - mBorder, mBarPaint1);
         canvas.drawLine(mDotX, mDotY, mWidth - mBorder, mDotY, mLinePaint1);
         canvas.drawLine(mBorder, mDotY, mDotX, mDotY, mLinePaint2);
-        if (mDotX != Float.NaN) {
+        if (!Float.isNaN(mDotX)) {
             canvas.drawCircle(mDotX, mDotY, mDotRadius, mDotPaint);
         }
     }
index 498f5a4..61b3621 100644 (file)
@@ -179,7 +179,7 @@ public class ColorHueView extends View implements ColorListener {
 
         canvas.drawLine(mDotX, mDotY, mWidth - mBorder, mDotY, mLinePaint1);
         canvas.drawLine(mBorder, mDotY, mDotX, mDotY, mLinePaint2);
-        if (mDotX != Float.NaN) {
+        if (!Float.isNaN(mDotX)) {
             canvas.drawCircle(mDotX, mDotY, mDotRadius, mDotPaint);
         }
     }
index ec15b5b..970a2ff 100644 (file)
@@ -171,7 +171,7 @@ public class ColorOpacityView extends View implements ColorListener {
         canvas.drawRect(mBorder, 0, mWidth - mBorder, mHeight, mBarPaint1);
         canvas.drawLine(mDotX, mDotY, mWidth - mBorder, mDotY, mLinePaint1);
         canvas.drawLine(mBorder, mDotY, mDotX, mDotY, mLinePaint2);
-        if (mDotX != Float.NaN) {
+        if (!Float.isNaN(mDotX)) {
             canvas.drawCircle(mDotX, mDotY, mDotRadius, mDotPaint);
         }
     }
index 07d7c71..92f4ad8 100644 (file)
@@ -170,8 +170,7 @@ public class ColorRectView extends View implements ColorListener {
         canvas.drawRect(rect, mWheelPaint3);
         canvas.drawRect(rect, mWheelPaint2);
 
-        if (mDotX != Float.NaN) {
-
+        if (!Float.isNaN(mDotX)) {
             canvas.drawCircle(mDotX, mDotY, mDotRadus, mDotPaint);
         }
     }
index fb8b4fc..6ebb48b 100644 (file)
@@ -126,8 +126,7 @@ public class ColorSVRectView extends View implements ColorListener {
         mRect.right -= mBorder;
         canvas.drawBitmap(mBitmap, null, mRect, mPaint1);
 
-        if (mDotX != Float.NaN) {
-
+        if (!Float.isNaN(mDotX)) {
             canvas.drawCircle(mDotX, mDotY, mDotRadus, mDotPaint);
         }
     }
index af4dfda..a29eee9 100644 (file)
@@ -170,7 +170,7 @@ public class ColorSaturationView extends View implements ColorListener {
         canvas.drawRect(mBorder, mBorder, mWidth - mBorder, mHeight - mBorder, mBarPaint1);
         canvas.drawLine(mDotX, mDotY, mWidth - mBorder, mDotY, mLinePaint1);
         canvas.drawLine(mBorder, mDotY, mDotX, mDotY, mLinePaint2);
-        if (mDotX != Float.NaN) {
+        if (!Float.isNaN(mDotX)) {
             canvas.drawCircle(mDotX, mDotY, mDotRadius, mDotPaint);
         }
     }
index bbfd829..0ace5d0 100644 (file)
@@ -148,7 +148,7 @@ public class ColorValueView extends View implements ColorListener {
         canvas.drawRect(mBorder, mBorder, mWidth - mBorder, mHeight - mBorder, mBarPaint1);
         canvas.drawLine(mDotX, mDotY, mDotX, mHeight - mBorder, mLinePaint2);
         canvas.drawLine(mDotX, mBorder, mDotX, mDotY, mLinePaint1);
-        if (mDotX != Float.NaN) {
+        if (!Float.isNaN(mDotX)) {
             canvas.drawCircle(mDotX, mDotY, dotRadus, mDotPaint);
         }
     }