On Mar 16, 2013, at 12:45, Thomas Neidhart <thomas.neidh...@gmail.com> wrote:

> 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).

Right! Good catch.

G
>
> Thomas
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>

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

Reply via email to