--- jsr166/src/test/tck/RecursiveTaskTest.java	2009/08/01 21:56:02	1.3
+++ jsr166/src/test/tck/RecursiveTaskTest.java	2010/09/11 07:31:52	1.12
@@ -11,18 +11,17 @@ import java.util.*;
 public class RecursiveTaskTest extends JSR166TestCase {
 
     public static void main(String[] args) {
-	junit.textui.TestRunner.run (suite());
+        junit.textui.TestRunner.run(suite());
     }
     public static Test suite() {
-	return new TestSuite(RecursiveTaskTest.class);
+        return new TestSuite(RecursiveTaskTest.class);
     }
 
     static final ForkJoinPool mainPool = new ForkJoinPool();
     static final ForkJoinPool singletonPool = new ForkJoinPool(1);
-    static final ForkJoinPool asyncSingletonPool = new ForkJoinPool(1);
-    static {
-        asyncSingletonPool.setAsyncMode(true);
-    }
+    static final ForkJoinPool asyncSingletonPool =
+        new ForkJoinPool(1, ForkJoinPool.defaultForkJoinWorkerThreadFactory,
+                         null, true);
 
     static final class FJException extends RuntimeException {
         FJException() { super(); }
@@ -65,21 +64,20 @@ public class RecursiveTaskTest extends J
      * isCompletedAbnormally and isCancelled return false for normally
      * completed tasks. getRawResult of a completed non-null task
      * returns value;
-     *
      */
     public void testInvoke() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    Integer r = f.invoke();
-                    threadAssertTrue(r == 21);
-                    threadAssertTrue(f.isDone());
-                    threadAssertFalse(f.isCancelled());
-                    threadAssertFalse(f.isCompletedAbnormally());
-                    threadAssertTrue(f.getRawResult() == 21);
-                    return r;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                Integer r = f.invoke();
+                threadAssertTrue(r == 21);
+                threadAssertTrue(f.isDone());
+                threadAssertFalse(f.isCancelled());
+                threadAssertFalse(f.isCompletedAbnormally());
+                threadAssertTrue(f.getRawResult() == 21);
+                return r;
+            }
+        };
         assertTrue(mainPool.invoke(a) == 21);
     }
 
@@ -90,17 +88,17 @@ public class RecursiveTaskTest extends J
      */
     public void testQuietlyInvoke() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    f.quietlyInvoke();
-                    Integer r = f.getRawResult();
-                    threadAssertTrue(r == 21);
-                    threadAssertTrue(f.isDone());
-                    threadAssertFalse(f.isCancelled());
-                    threadAssertFalse(f.isCompletedAbnormally());
-                    return r;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                f.quietlyInvoke();
+                Integer r = f.getRawResult();
+                threadAssertTrue(r == 21);
+                threadAssertTrue(f.isDone());
+                threadAssertFalse(f.isCancelled());
+                threadAssertFalse(f.isCompletedAbnormally());
+                return r;
+            }
+        };
         assertTrue(mainPool.invoke(a) == 21);
     }
 
@@ -109,15 +107,15 @@ public class RecursiveTaskTest extends J
      */
     public void testForkJoin() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    Integer r = f.join();
-                    threadAssertTrue(r == 21);
-                    threadAssertTrue(f.isDone());
-                    return r;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                f.fork();
+                Integer r = f.join();
+                threadAssertTrue(r == 21);
+                threadAssertTrue(f.isDone());
+                return r;
+            }
+        };
         assertTrue(mainPool.invoke(a) == 21);
     }
 
@@ -126,96 +124,60 @@ public class RecursiveTaskTest extends J
      */
     public void testForkGet() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.fork();
-                        Integer r = f.get();
-                        threadAssertTrue(r == 21);
-                        threadAssertTrue(f.isDone());
-                        return r;
-                    } catch (Exception ex) {
-                        unexpectedException();
-                    }
-                    return NoResult;
-                }
-            };
-        assertTrue(mainPool.invoke(a) == 21);
-    }
-
-    /**
-     * timed get of a forked task returns when task completes
-     */
-    public void testForkTimedGet() {
-        RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.fork();
-                        Integer r = f.get(5L, TimeUnit.SECONDS);
-                        threadAssertTrue(r == 21);
-                        threadAssertTrue(f.isDone());
-                        return r;
-                    } catch (Exception ex) {
-                        unexpectedException();
-                    }
-                    return NoResult;
-                }
-            };
-        assertTrue(mainPool.invoke(a) == 21);
-    }
-
-    /**
-     * helpJoin of a forked task returns when task completes
-     */
-    public void testForkHelpJoin() {
-        RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
+            public Integer compute() {
+                try {
                     FibTask f = new FibTask(8);
                     f.fork();
-                    Integer r = f.helpJoin();
+                    Integer r = f.get();
                     threadAssertTrue(r == 21);
                     threadAssertTrue(f.isDone());
                     return r;
+                } catch (Exception ex) {
+                    unexpectedException(ex);
                 }
-            };
+                return NoResult;
+            }
+        };
         assertTrue(mainPool.invoke(a) == 21);
     }
 
     /**
-     * quietlyJoin of a forked task returns when task completes
+     * timed get of a forked task returns when task completes
      */
-    public void testForkQuietlyJoin() {
+    public void testForkTimedGet() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
+            public Integer compute() {
+                try {
                     FibTask f = new FibTask(8);
                     f.fork();
-                    f.quietlyJoin();
-                    Integer r = f.getRawResult();
+                    Integer r = f.get(5L, TimeUnit.SECONDS);
                     threadAssertTrue(r == 21);
                     threadAssertTrue(f.isDone());
                     return r;
+                } catch (Exception ex) {
+                    unexpectedException(ex);
                 }
-            };
+                return NoResult;
+            }
+        };
         assertTrue(mainPool.invoke(a) == 21);
     }
 
-
     /**
-     * quietlyHelpJoin of a forked task returns when task completes
+     * quietlyJoin of a forked task returns when task completes
      */
-    public void testForkQuietlyHelpJoin() {
+    public void testForkQuietlyJoin() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    f.quietlyHelpJoin();
-                    Integer r = f.getRawResult();
-                    threadAssertTrue(r == 21);
-                    threadAssertTrue(f.isDone());
-                    return r;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                f.fork();
+                f.quietlyJoin();
+                Integer r = f.getRawResult();
+                threadAssertTrue(r == 21);
+                threadAssertTrue(f.isDone());
+                return r;
+            }
+        };
         assertTrue(mainPool.invoke(a) == 21);
     }
 
@@ -226,17 +188,17 @@ public class RecursiveTaskTest extends J
      */
     public void testForkHelpQuiesce() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    f.helpQuiesce();
-                    Integer r = f.getRawResult();
-                    threadAssertTrue(r == 21);
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(getQueuedTaskCount() == 0);
-                    return r;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                f.fork();
+                f.helpQuiesce();
+                Integer r = f.getRawResult();
+                threadAssertTrue(r == 21);
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(getQueuedTaskCount() == 0);
+                return r;
+            }
+        };
         assertTrue(mainPool.invoke(a) == 21);
     }
 
@@ -246,32 +208,32 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalInvoke() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FailingFibTask f = new FailingFibTask(8);
-                        f.invoke();
-                        shouldThrow();
-                        return NoResult;
-                    } catch (FJException success) {
-                    }
+            public Integer compute() {
+                try {
+                    FailingFibTask f = new FailingFibTask(8);
+                    f.invoke();
+                    shouldThrow();
                     return NoResult;
+                } catch (FJException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
     /**
-     * quietelyInvoke task returns when task completes abnormally
+     * quietlyInvoke task returns when task completes abnormally
      */
     public void testAbnormalQuietlyInvoke() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FailingFibTask f = new FailingFibTask(8);
-                    f.quietlyInvoke();
-                    threadAssertTrue(f.isDone());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FailingFibTask f = new FailingFibTask(8);
+                f.quietlyInvoke();
+                threadAssertTrue(f.isDone());
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -280,18 +242,18 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalForkJoin() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FailingFibTask f = new FailingFibTask(8);
-                        f.fork();
-                        Integer r = f.join();
-                        shouldThrow();
-                        return r;
-                    } catch (FJException success) {
-                    }
-                    return NoResult;
+            public Integer compute() {
+                try {
+                    FailingFibTask f = new FailingFibTask(8);
+                    f.fork();
+                    Integer r = f.join();
+                    shouldThrow();
+                    return r;
+                } catch (FJException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -300,18 +262,20 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalForkGet() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FailingFibTask f = new FailingFibTask(8);
-                        f.fork();
-                        Integer r = f.get();
-                        shouldThrow();
-                        return r;
-                    } catch (Exception success) {
-                    }
-                    return NoResult;
+            public Integer compute() {
+                try {
+                    FailingFibTask f = new FailingFibTask(8);
+                    f.fork();
+                    Integer r = f.get();
+                    shouldThrow();
+                    return r;
+                } catch (ExecutionException success) {
+                } catch (Exception ex) {
+                    unexpectedException(ex);
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -320,60 +284,20 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalForkTimedGet() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FailingFibTask f = new FailingFibTask(8);
-                        f.fork();
-                        Integer r = f.get(5L, TimeUnit.SECONDS);
-                        shouldThrow();
-                        return r;
-                    } catch (Exception success) {
-                    }
-                    return NoResult;
-                }
-            };
-        mainPool.invoke(a);
-    }
-
-    /**
-     * join of a forked task throws exception when task completes abnormally
-     */
-    public void testAbnormalForkHelpJoin() {
-        RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FailingFibTask f = new FailingFibTask(8);
-                        f.fork();
-                        Integer r = f.helpJoin();
-                        shouldThrow();
-                        return r;
-                    } catch (FJException success) {
-                    }
-                    return NoResult;
-                }
-            };
-        mainPool.invoke(a);
-    }
-
-    /**
-     * quietlyHelpJoin of a forked task returns when task completes abnormally.
-     * getException of failed task returns its exception.
-     * isCompletedAbnormally of a failed task returns true.
-     * isCancelled of a failed uncancelled task returns false
-     */
-    public void testAbnormalForkQuietlyHelpJoin() {
-        RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
+            public Integer compute() {
+                try {
                     FailingFibTask f = new FailingFibTask(8);
                     f.fork();
-                    f.quietlyHelpJoin();
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.isCompletedAbnormally());
-                    threadAssertFalse(f.isCancelled());
-                    threadAssertTrue(f.getException() instanceof FJException);
-                    return NoResult;
+                    Integer r = f.get(5L, TimeUnit.SECONDS);
+                    shouldThrow();
+                    return r;
+                } catch (ExecutionException success) {
+                } catch (Exception ex) {
+                    unexpectedException(ex);
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -382,16 +306,16 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalForkQuietlyJoin() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FailingFibTask f = new FailingFibTask(8);
-                    f.fork();
-                    f.quietlyJoin();
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.isCompletedAbnormally());
-                    threadAssertTrue(f.getException() instanceof FJException);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FailingFibTask f = new FailingFibTask(8);
+                f.fork();
+                f.quietlyJoin();
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(f.isCompletedAbnormally());
+                threadAssertTrue(f.getException() instanceof FJException);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -400,18 +324,18 @@ public class RecursiveTaskTest extends J
      */
     public void testCancelledInvoke() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.cancel(true);
-                        Integer r = f.invoke();
-                        shouldThrow();
-                        return r;
-                    } catch (CancellationException success) {
-                    }
-                    return NoResult;
+            public Integer compute() {
+                try {
+                    FibTask f = new FibTask(8);
+                    f.cancel(true);
+                    Integer r = f.invoke();
+                    shouldThrow();
+                    return r;
+                } catch (CancellationException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -420,19 +344,19 @@ public class RecursiveTaskTest extends J
      */
     public void testCancelledForkJoin() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.cancel(true);
-                        f.fork();
-                        Integer r = f.join();
-                        shouldThrow();
-                        return r;
-                    } catch (CancellationException success) {
-                    }
-                    return NoResult;
+            public Integer compute() {
+                try {
+                    FibTask f = new FibTask(8);
+                    f.cancel(true);
+                    f.fork();
+                    Integer r = f.join();
+                    shouldThrow();
+                    return r;
+                } catch (CancellationException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -441,19 +365,21 @@ public class RecursiveTaskTest extends J
      */
     public void testCancelledForkGet() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.cancel(true);
-                        f.fork();
-                        Integer r = f.get();
-                        shouldThrow();
-                        return r;
-                    } catch (Exception success) {
-                    }
-                    return NoResult;
+            public Integer compute() {
+                try {
+                    FibTask f = new FibTask(8);
+                    f.cancel(true);
+                    f.fork();
+                    Integer r = f.get();
+                    shouldThrow();
+                    return r;
+                } catch (CancellationException success) {
+                } catch (Exception ex) {
+                    unexpectedException(ex);
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -462,63 +388,21 @@ public class RecursiveTaskTest extends J
      */
     public void testCancelledForkTimedGet() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.cancel(true);
-                        f.fork();
-                        Integer r = f.get(5L, TimeUnit.SECONDS);
-                        shouldThrow();
-                        return r;
-                    } catch (Exception success) {
-                    }
-                    return NoResult;
-                }
-            };
-        mainPool.invoke(a);
-    }
-
-    /**
-     * join of a forked task throws exception when task cancelled
-     */
-    public void testCancelledForkHelpJoin() {
-        RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.cancel(true);
-                        f.fork();
-                        Integer r = f.helpJoin();
-                        shouldThrow();
-                        return r;
-                    } catch (CancellationException success) {
-                    }
-                    return NoResult;
-                }
-            };
-        mainPool.invoke(a);
-    }
-
-    /**
-     * quietlyHelpJoin of a forked task returns when task cancelled.
-     * getException of cancelled task returns its exception
-     * isCompletedAbnormally of a cancelled task returns true.
-     * isCancelled of a cancelled task returns true
-     */
-    public void testCancelledForkQuietlyHelpJoin() {
-        RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
+            public Integer compute() {
+                try {
                     FibTask f = new FibTask(8);
                     f.cancel(true);
                     f.fork();
-                    f.quietlyHelpJoin();
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.isCompletedAbnormally());
-                    threadAssertTrue(f.isCancelled());
-                    threadAssertTrue(f.getException() instanceof CancellationException);
-                    return NoResult;
+                    Integer r = f.get(5L, TimeUnit.SECONDS);
+                    shouldThrow();
+                    return r;
+                } catch (CancellationException success) {
+                } catch (Exception ex) {
+                    unexpectedException(ex);
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -527,17 +411,17 @@ public class RecursiveTaskTest extends J
      */
     public void testCancelledForkQuietlyJoin() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    f.cancel(true);
-                    f.fork();
-                    f.quietlyJoin();
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.isCompletedAbnormally());
-                    threadAssertTrue(f.getException() instanceof CancellationException);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                f.cancel(true);
+                f.fork();
+                f.quietlyJoin();
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(f.isCompletedAbnormally());
+                threadAssertTrue(f.getException() instanceof CancellationException);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -546,11 +430,11 @@ public class RecursiveTaskTest extends J
      */
     public void testGetPool() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    threadAssertTrue(getPool() == mainPool);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                threadAssertTrue(getPool() == mainPool);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -572,11 +456,11 @@ public class RecursiveTaskTest extends J
      */
     public void testInForkJoinPool() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    threadAssertTrue(inForkJoinPool());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                threadAssertTrue(inForkJoinPool());
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -585,11 +469,11 @@ public class RecursiveTaskTest extends J
      */
     public void testInForkJoinPool2() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    threadAssertTrue(!inForkJoinPool());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                threadAssertTrue(!inForkJoinPool());
+                return NoResult;
+            }
+        };
         a.invoke();
     }
 
@@ -598,11 +482,11 @@ public class RecursiveTaskTest extends J
      */
     public void testSetRawResult() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    setRawResult(NoResult);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                setRawResult(NoResult);
+                return NoResult;
+            }
+        };
         assertEquals(a.invoke(), NoResult);
     }
 
@@ -611,19 +495,19 @@ public class RecursiveTaskTest extends J
      */
     public void testReinitialize() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    Integer r = f.invoke();
-                    threadAssertTrue(r == 21);
-                    threadAssertTrue(f.isDone());
-                    threadAssertFalse(f.isCancelled());
-                    threadAssertFalse(f.isCompletedAbnormally());
-                    f.reinitialize();
-                    r = f.invoke();
-                    threadAssertTrue(r == 21);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                Integer r = f.invoke();
+                threadAssertTrue(r == 21);
+                threadAssertTrue(f.isDone());
+                threadAssertFalse(f.isCancelled());
+                threadAssertFalse(f.isCompletedAbnormally());
+                f.reinitialize();
+                r = f.invoke();
+                threadAssertTrue(r == 21);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -632,18 +516,18 @@ public class RecursiveTaskTest extends J
      */
     public void testCompleteExceptionally() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        f.completeExceptionally(new FJException());
-                        Integer r = f.invoke();
-                        shouldThrow();
-                        return r;
-                    } catch (FJException success) {
-                    }
-                    return NoResult;
+            public Integer compute() {
+                try {
+                    FibTask f = new FibTask(8);
+                    f.completeExceptionally(new FJException());
+                    Integer r = f.invoke();
+                    shouldThrow();
+                    return r;
+                } catch (FJException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -652,15 +536,15 @@ public class RecursiveTaskTest extends J
      */
     public void testComplete() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    f.complete(NoResult);
-                    Integer r = f.invoke();
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(r == NoResult);
-                    return r;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                f.complete(NoResult);
+                Integer r = f.invoke();
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(r == NoResult);
+                return r;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -669,17 +553,17 @@ public class RecursiveTaskTest extends J
      */
     public void testInvokeAll2() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    FibTask g = new FibTask(9);
-                    invokeAll(f, g);
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.join() == 21);
-                    threadAssertTrue(g.isDone());
-                    threadAssertTrue(g.join() == 34);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                FibTask g = new FibTask(9);
+                invokeAll(f, g);
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(f.join() == 21);
+                threadAssertTrue(g.isDone());
+                threadAssertTrue(g.join() == 34);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -688,14 +572,14 @@ public class RecursiveTaskTest extends J
      */
     public void testInvokeAll1() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    invokeAll(f);
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.join() == 21);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                invokeAll(f);
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(f.join() == 21);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -704,20 +588,20 @@ public class RecursiveTaskTest extends J
      */
     public void testInvokeAll3() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    FibTask g = new FibTask(9);
-                    FibTask h = new FibTask(7);
-                    invokeAll(f, g, h);
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.join() == 21);
-                    threadAssertTrue(g.isDone());
-                    threadAssertTrue(g.join() == 34);
-                    threadAssertTrue(h.isDone());
-                    threadAssertTrue(h.join() == 13);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                FibTask g = new FibTask(9);
+                FibTask h = new FibTask(7);
+                invokeAll(f, g, h);
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(f.join() == 21);
+                threadAssertTrue(g.isDone());
+                threadAssertTrue(g.join() == 34);
+                threadAssertTrue(h.isDone());
+                threadAssertTrue(h.join() == 13);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -726,24 +610,24 @@ public class RecursiveTaskTest extends J
      */
     public void testInvokeAllCollection() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask f = new FibTask(8);
-                    FibTask g = new FibTask(9);
-                    FibTask h = new FibTask(7);
-                    HashSet set = new HashSet();
-                    set.add(f);
-                    set.add(g);
-                    set.add(h);
-                    invokeAll(set);
-                    threadAssertTrue(f.isDone());
-                    threadAssertTrue(f.join() == 21);
-                    threadAssertTrue(g.isDone());
-                    threadAssertTrue(g.join() == 34);
-                    threadAssertTrue(h.isDone());
-                    threadAssertTrue(h.join() == 13);
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask f = new FibTask(8);
+                FibTask g = new FibTask(9);
+                FibTask h = new FibTask(7);
+                HashSet set = new HashSet();
+                set.add(f);
+                set.add(g);
+                set.add(h);
+                invokeAll(set);
+                threadAssertTrue(f.isDone());
+                threadAssertTrue(f.join() == 21);
+                threadAssertTrue(g.isDone());
+                threadAssertTrue(g.join() == 34);
+                threadAssertTrue(h.isDone());
+                threadAssertTrue(h.join() == 13);
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -753,18 +637,18 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalInvokeAll2() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        FailingFibTask g = new FailingFibTask(9);
-                        invokeAll(f, g);
-                        shouldThrow();
-                        return NoResult;
-                    } catch (FJException success) {
-                    }
+            public Integer compute() {
+                try {
+                    FibTask f = new FibTask(8);
+                    FailingFibTask g = new FailingFibTask(9);
+                    invokeAll(f, g);
+                    shouldThrow();
                     return NoResult;
+                } catch (FJException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -773,17 +657,17 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalInvokeAll1() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FailingFibTask g = new FailingFibTask(9);
-                        invokeAll(g);
-                        shouldThrow();
-                        return NoResult;
-                    } catch (FJException success) {
-                    }
+            public Integer compute() {
+                try {
+                    FailingFibTask g = new FailingFibTask(9);
+                    invokeAll(g);
+                    shouldThrow();
                     return NoResult;
+                } catch (FJException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -792,19 +676,19 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalInvokeAll3() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FibTask f = new FibTask(8);
-                        FailingFibTask g = new FailingFibTask(9);
-                        FibTask h = new FibTask(7);
-                        invokeAll(f, g, h);
-                        shouldThrow();
-                        return NoResult;
-                    } catch (FJException success) {
-                    }
+            public Integer compute() {
+                try {
+                    FibTask f = new FibTask(8);
+                    FailingFibTask g = new FailingFibTask(9);
+                    FibTask h = new FibTask(7);
+                    invokeAll(f, g, h);
+                    shouldThrow();
                     return NoResult;
+                } catch (FJException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -813,23 +697,23 @@ public class RecursiveTaskTest extends J
      */
     public void testAbnormalInvokeAllCollection() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    try {
-                        FailingFibTask f = new FailingFibTask(8);
-                        FibTask g = new FibTask(9);
-                        FibTask h = new FibTask(7);
-                        HashSet set = new HashSet();
-                        set.add(f);
-                        set.add(g);
-                        set.add(h);
-                        invokeAll(set);
-                        shouldThrow();
-                        return NoResult;
-                    } catch (FJException success) {
-                    }
+            public Integer compute() {
+                try {
+                    FailingFibTask f = new FailingFibTask(8);
+                    FibTask g = new FibTask(9);
+                    FibTask h = new FibTask(7);
+                    HashSet set = new HashSet();
+                    set.add(f);
+                    set.add(g);
+                    set.add(h);
+                    invokeAll(set);
+                    shouldThrow();
                     return NoResult;
+                } catch (FJException success) {
                 }
-            };
+                return NoResult;
+            }
+        };
         mainPool.invoke(a);
     }
 
@@ -839,18 +723,18 @@ public class RecursiveTaskTest extends J
      */
     public void testTryUnfork() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(f.tryUnfork());
-                    helpQuiesce();
-                    threadAssertFalse(f.isDone());
-                    threadAssertTrue(g.isDone());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(f.tryUnfork());
+                helpQuiesce();
+                threadAssertFalse(f.isDone());
+                threadAssertTrue(g.isDone());
+                return NoResult;
+            }
+        };
         singletonPool.invoke(a);
     }
 
@@ -860,18 +744,18 @@ public class RecursiveTaskTest extends J
      */
     public void testGetSurplusQueuedTaskCount() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask h = new FibTask(7);
-                    h.fork();
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(getSurplusQueuedTaskCount() > 0);
-                    helpQuiesce();
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask h = new FibTask(7);
+                h.fork();
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(getSurplusQueuedTaskCount() > 0);
+                helpQuiesce();
+                return NoResult;
+            }
+        };
         singletonPool.invoke(a);
     }
 
@@ -880,18 +764,18 @@ public class RecursiveTaskTest extends J
      */
     public void testPeekNextLocalTask() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(peekNextLocalTask() == f);
-                    f.join();
-                    threadAssertTrue(f.isDone());
-                    helpQuiesce();
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(peekNextLocalTask() == f);
+                f.join();
+                threadAssertTrue(f.isDone());
+                helpQuiesce();
+                return NoResult;
+            }
+        };
         singletonPool.invoke(a);
     }
 
@@ -901,17 +785,17 @@ public class RecursiveTaskTest extends J
      */
     public void testPollNextLocalTask() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(pollNextLocalTask() == f);
-                    helpQuiesce();
-                    threadAssertFalse(f.isDone());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(pollNextLocalTask() == f);
+                helpQuiesce();
+                threadAssertFalse(f.isDone());
+                return NoResult;
+            }
+        };
         singletonPool.invoke(a);
     }
 
@@ -921,18 +805,18 @@ public class RecursiveTaskTest extends J
      */
     public void testPollTask() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(pollTask() == f);
-                    helpQuiesce();
-                    threadAssertFalse(f.isDone());
-                    threadAssertTrue(g.isDone());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(pollTask() == f);
+                helpQuiesce();
+                threadAssertFalse(f.isDone());
+                threadAssertTrue(g.isDone());
+                return NoResult;
+            }
+        };
         singletonPool.invoke(a);
     }
 
@@ -941,18 +825,18 @@ public class RecursiveTaskTest extends J
      */
     public void testPeekNextLocalTaskAsync() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(peekNextLocalTask() == g);
-                    f.join();
-                    helpQuiesce();
-                    threadAssertTrue(f.isDone());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(peekNextLocalTask() == g);
+                f.join();
+                helpQuiesce();
+                threadAssertTrue(f.isDone());
+                return NoResult;
+            }
+        };
         asyncSingletonPool.invoke(a);
     }
 
@@ -962,18 +846,18 @@ public class RecursiveTaskTest extends J
      */
     public void testPollNextLocalTaskAsync() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(pollNextLocalTask() == g);
-                    helpQuiesce();
-                    threadAssertTrue(f.isDone());
-                    threadAssertFalse(g.isDone());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(pollNextLocalTask() == g);
+                helpQuiesce();
+                threadAssertTrue(f.isDone());
+                threadAssertFalse(g.isDone());
+                return NoResult;
+            }
+        };
         asyncSingletonPool.invoke(a);
     }
 
@@ -983,18 +867,18 @@ public class RecursiveTaskTest extends J
      */
     public void testPollTaskAsync() {
         RecursiveTask<Integer> a = new RecursiveTask<Integer>() {
-                public Integer compute() {
-                    FibTask g = new FibTask(9);
-                    g.fork();
-                    FibTask f = new FibTask(8);
-                    f.fork();
-                    threadAssertTrue(pollTask() == g);
-                    helpQuiesce();
-                    threadAssertTrue(f.isDone());
-                    threadAssertFalse(g.isDone());
-                    return NoResult;
-                }
-            };
+            public Integer compute() {
+                FibTask g = new FibTask(9);
+                g.fork();
+                FibTask f = new FibTask(8);
+                f.fork();
+                threadAssertTrue(pollTask() == g);
+                helpQuiesce();
+                threadAssertTrue(f.isDone());
+                threadAssertFalse(g.isDone());
+                return NoResult;
+            }
+        };
         asyncSingletonPool.invoke(a);
     }