Hi,
I've reviewed some more code in openejb1 cvs module and found out that most of the inner classes used do not rely on instance-level information in the enclosing class. They are mostly independent of any specific class instance, so i've transformed them to static (less coupling and resources to be managed by the JVM so we achieve better performance (i guess...)).
Same with lots of constants that weren't final (only static), it's ok to be static, but some could by mistake change that value and cause and application crash...
I've found some loop string concatenation and replaced it with a StringBuffer, since String objects are immutable it is required to create a new object every time we concatenate two strings, with StringBuffer it just appends the second string in the available buffer space, reducing the object creation overhead.
Finally i did some changes on openejb-test.bat file in order to make it run the integration tests.
Best regards, Paulo
