You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ms...@apache.org on 2018/03/17 15:55:03 UTC

svn commit: r1827092 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/Matrix.java

Author: msahyoun
Date: Sat Mar 17 15:55:02 2018
New Revision: 1827092

URL: http://svn.apache.org/viewvc?rev=1827092&view=rev
Log:
PDFBOX-4134: use Float.compare

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/Matrix.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/Matrix.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/Matrix.java?rev=1827092&r1=1827091&r2=1827092&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/Matrix.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/util/Matrix.java Sat Mar 17 15:55:02 2018
@@ -601,7 +601,7 @@ public final class Matrix implements Clo
          * sqrt(x2) =
          * abs(x)
          */
-        if( !(single[1]==0.0f && single[3]==0.0f) )
+        if( !(Float.compare(single[1], 0.0f) == 0 && Float.compare(single[3], 0.0f) ==0) )
         {
             xScale = (float)Math.sqrt(Math.pow(single[0], 2)+
                                       Math.pow(single[1], 2));
@@ -617,7 +617,7 @@ public final class Matrix implements Clo
     public float getScalingFactorY()
     {
         float yScale = single[4];
-        if( !(single[1]==0.0f && single[3]==0.0f) )
+        if( !(Float.compare(single[1], 0.0f) == 0 && Float.compare(single[3], 0.0f) == 0) )
         {
             yScale = (float)Math.sqrt(Math.pow(single[3], 2)+
                                       Math.pow(single[4], 2));