James-, Do you really care to catch the SQL exceptions in the finally{} block? Since doTag(...) permits any exceptions to be trown, just let the them go up the stack frames. I noticed this coding pattern in other tags too.
Also, someone else repsonded to these fixes on commons-dev, check it out. Thanks, Hristo __________________________________________________ Do you Yahoo!? New DSL Internet Access from SBC & Yahoo! http://sbc.yahoo.com -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>