Author: sandymac Date: Fri Dec 15 13:18:47 2006 New Revision: 487667 URL: http://svn.apache.org/viewvc?view=rev&rev=487667 Log: Make StackObjectPool.borrowObject to throw a NoSuchElementException when the PoolableObjectFactory.makeObject returns null. addresses POOL-91
Modified: jakarta/commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/StackObjectPool.java Modified: jakarta/commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/StackObjectPool.java URL: http://svn.apache.org/viewvc/jakarta/commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/StackObjectPool.java?view=diff&rev=487667&r1=487666&r2=487667 ============================================================================== --- jakarta/commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/StackObjectPool.java (original) +++ jakarta/commons/proper/pool/trunk/src/java/org/apache/commons/pool/impl/StackObjectPool.java Fri Dec 15 13:18:47 2006 @@ -148,6 +148,9 @@ } else { newlyCreated = true; obj = _factory.makeObject(); + if (obj == null) { + throw new NoSuchElementException("PoolableObjectFactory.makeObject() returned null."); + } } } if (!newlyCreated && null != _factory && null != obj) { --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]