Hi Greg,

A patch attached to a bugzilla issue would be nice.
In this case it is a trivial change but every contribution/patch is appreciated. A bugzilla issue to document the change is also needed.


Changing to using an iterator would increase performance slightly (unneeded synchronization in enumeration). But this is not really an issue for the clear method.
But it is a simple change so go for it!


Cheers
Dirk


Greg Billock wrote:
I recently compiled commons-pool under JDK1.5 without a problem
except for two files:

/src/java/org/apache/commons/pool/impl/StackKeyedObjectPool.java
/src/java/org/apache/commons/pool/impl/StackObjectPool.java

These two files use the new 1.5 keyword 'enum' in an enumeration.
A shallow fix is to simply rename the variable to 'enm' or something.
A deeper fix might be to use iterator() method. I'm happy to contribute
a patch which enables clean 1.5 compilation, but assuming the
shallow fix is desired, it is pretty straightforward. Please email me
directly if you (Dirk?) would like the diffs.

Thanks for the pooling classes!



--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]



Reply via email to