On Mar 16, 2013, at 12:45, Thomas Neidhart <[email protected]> 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: [email protected]
> For additional commands, e-mail: [email protected]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to