commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Thomas Neidhart <thomas.neidh...@gmail.com>
Subject Re: [VOTE] Release of Commons Logging 1.1.2 based on RC2
Date Sat, 16 Mar 2013 16:45:23 GMT
On 03/16/2013 03:21 PM, Gary Gregory wrote:
> Not a blocker but not pretty:
> 
> The variable testObjCls may be null at line 199 in LoadTestCase.java
> (/commons-logging/src/test/java/org/apache/commons/logging)
> 
> And... wow is that code some weird format!

no question about that ;-), but I don't think the variable can be null
at line 199. This should be covered by the two catches before, and the
Assert.fail (which eclipse can not detect I guess).

Thomas

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


Mime
View raw message