You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by gb...@apache.org on 2011/08/15 10:01:52 UTC

svn commit: r1157735 - /pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java

Author: gbailleul
Date: Mon Aug 15 08:01:52 2011
New Revision: 1157735

URL: http://svn.apache.org/viewvc?rev=1157735&view=rev
Log:
PDFBOX-1079 : fixed testHashCode instability and now compare hashCode (like in other TestCOS***)

Modified:
    pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java

Modified: pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java?rev=1157735&r1=1157734&r2=1157735&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java (original)
+++ pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java Mon Aug 15 08:01:52 2011
@@ -107,8 +107,9 @@ public class TestCOSFloat extends TestCO
             COSFloat test2 = new COSFloat(num);
             assertEquals(test1.hashCode(), test2.hashCode());
             
-            COSFloat test3 = new COSFloat(num + 0.01f);
-            assertFalse(test3.equals(test1));
+            float nf = Float.intBitsToFloat(Float.floatToIntBits(num)+1);
+            COSFloat test3 = new COSFloat(nf);
+            assertFalse(test3.hashCode()==test1.hashCode());
         }
     }