Fixed a small bug. In activate(), we need to make sure that an

instance of ThreadManager exists before we call thrCount(). This was
not the case and therefore we were getting a NullPointerException.

1 files changed, 4 insertions, 4 deletions

diff --git a/java/src/Task.java b/java/src/Task.java
index 0c1c41ec826..af6251ec783 100644
--- a/java/src/Task.java
+++ b/java/src/Task.java
@@ -102,14 +102,14 @@ public abstract class Task implements Runnable, EventHandler
    */
   public synchronized int activate (long flags, int nThreads, boolean forceActive)
     {
-       if (this.thrCount () > 0 && forceActive == false)
-	 return 1; // Already active.
-       this.flags_ = flags;
-
        // Create a Thread Manager if we do not already have one
        if (this.thrMgr_ == null)
 	 this.thrMgr_ = new ThreadManager ();
 
+       if (this.thrCount () > 0 && forceActive == false)
+	 return 1; // Already active.
+       this.flags_ = flags;
+
        if (ACE.BIT_ENABLED (flags, TaskFlags.THR_DAEMON))
 	 this.thrMgr_.spawnN (nThreads, this, true); // Spawn off all threads as daemon threads
        else        // Spawn off all threads as normal threads