You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jcs-dev@jakarta.apache.org by as...@apache.org on 2007/03/18 23:12:06 UTC

svn commit: r519709 - in /jakarta/jcs/trunk/src: java/org/apache/jcs/utils/threadpool/ThreadPoolManager.java test/org/apache/jcs/utils/threadpool/ThreadPoolManagerUnitTest.java

Author: asmuts
Date: Sun Mar 18 15:12:05 2007
New Revision: 519709

URL: http://svn.apache.org/viewvc?view=rev&rev=519709
Log:
fixing a boolean setting problem

Modified:
    jakarta/jcs/trunk/src/java/org/apache/jcs/utils/threadpool/ThreadPoolManager.java
    jakarta/jcs/trunk/src/test/org/apache/jcs/utils/threadpool/ThreadPoolManagerUnitTest.java

Modified: jakarta/jcs/trunk/src/java/org/apache/jcs/utils/threadpool/ThreadPoolManager.java
URL: http://svn.apache.org/viewvc/jakarta/jcs/trunk/src/java/org/apache/jcs/utils/threadpool/ThreadPoolManager.java?view=diff&rev=519709&r1=519708&r2=519709
==============================================================================
--- jakarta/jcs/trunk/src/java/org/apache/jcs/utils/threadpool/ThreadPoolManager.java (original)
+++ jakarta/jcs/trunk/src/java/org/apache/jcs/utils/threadpool/ThreadPoolManager.java Sun Mar 18 15:12:05 2007
@@ -355,7 +355,7 @@
         {
             try
             {
-                config.setUseBoundary( Boolean.valueOf( (String) props.get( root + ".useBoundary" ) ) );
+                config.setUseBoundary( Boolean.valueOf( (String) props.get( root + ".useBoundary" ) ).booleanValue() );
             }
             catch ( NumberFormatException nfe )
             {

Modified: jakarta/jcs/trunk/src/test/org/apache/jcs/utils/threadpool/ThreadPoolManagerUnitTest.java
URL: http://svn.apache.org/viewvc/jakarta/jcs/trunk/src/test/org/apache/jcs/utils/threadpool/ThreadPoolManagerUnitTest.java?view=diff&rev=519709&r1=519708&r2=519709
==============================================================================
--- jakarta/jcs/trunk/src/test/org/apache/jcs/utils/threadpool/ThreadPoolManagerUnitTest.java (original)
+++ jakarta/jcs/trunk/src/test/org/apache/jcs/utils/threadpool/ThreadPoolManagerUnitTest.java Sun Mar 18 15:12:05 2007
@@ -168,15 +168,18 @@
      */
     public void testWithBoundary()
     {
+        // SETUP
         ThreadPoolManager.setPropsFileName( "thread_pool.properties" );
         ThreadPoolManager mgr = ThreadPoolManager.getInstance();
         // force config from new props file
         mgr.configure();
         assertNotNull( mgr );
 
+        // DO WORK
         ThreadPool pool = mgr.getPool( "withbound" );
+        
+        // VERIFY
         assertNotNull( "Should have gotten back a pool.", pool );
-
         assertTrue( "Should have a BoundedBuffer and not a linked queue.", pool.getQueue() instanceof BoundedBuffer );
     }    
 }



---------------------------------------------------------------------
To unsubscribe, e-mail: jcs-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jcs-dev-help@jakarta.apache.org