--- jsr166/src/test/tck/AtomicLongTest.java	2011/06/10 20:01:21	1.23
+++ jsr166/src/test/tck/AtomicLongTest.java	2017/08/04 03:30:21	1.33
@@ -6,12 +6,14 @@
  * Pat Fisher, Mike Judd.
  */
 
-import junit.framework.*;
 import java.util.concurrent.atomic.AtomicLong;
 
+import junit.framework.Test;
+import junit.framework.TestSuite;
+
 public class AtomicLongTest extends JSR166TestCase {
     public static void main(String[] args) {
-        junit.textui.TestRunner.run(suite());
+        main(suite(), args);
     }
     public static Test suite() {
         return new TestSuite(AtomicLongTest.class);
@@ -102,10 +104,10 @@ public class AtomicLongTest extends JSR1
      */
     public void testWeakCompareAndSet() {
         AtomicLong ai = new AtomicLong(1);
-        while (!ai.weakCompareAndSet(1, 2));
-        while (!ai.weakCompareAndSet(2, -4));
+        do {} while (!ai.weakCompareAndSet(1, 2));
+        do {} while (!ai.weakCompareAndSet(2, -4));
         assertEquals(-4, ai.get());
-        while (!ai.weakCompareAndSet(-4, 7));
+        do {} while (!ai.weakCompareAndSet(-4, 7));
         assertEquals(7, ai.get());
     }
 
@@ -191,14 +193,15 @@ public class AtomicLongTest extends JSR1
     }
 
     /**
-     * a deserialized serialized atomic holds same value
+     * a deserialized/reserialized atomic holds same value
      */
     public void testSerialization() throws Exception {
         AtomicLong x = new AtomicLong();
         AtomicLong y = serialClone(x);
-        assertTrue(x != y);
+        assertNotSame(x, y);
         x.set(-22);
         AtomicLong z = serialClone(x);
+        assertNotSame(y, z);
         assertEquals(-22, x.get());
         assertEquals(0, y.get());
         assertEquals(-22, z.get());
@@ -210,9 +213,9 @@ public class AtomicLongTest extends JSR1
     public void testToString() {
         AtomicLong ai = new AtomicLong();
         assertEquals("0", ai.toString());
-        for (long i : VALUES) {
-            ai.set(i);
-            assertEquals(ai.toString(), Long.toString(i));
+        for (long x : VALUES) {
+            ai.set(x);
+            assertEquals(Long.toString(x), ai.toString());
         }
     }