On Mon, Nov 30, 2009 at 11:30 AM, Peng Lee <[email protected]> wrote: > In my opinion,the line 176 [if (processorConstructor == null) ] > is redundant, because if processorConstructor is null, a > NullPointerException should be trowed previously. So, I think we > should remove the redundant code here. > Of course, maybe there are any other things that i haven't knew > about, and who knows?
The test is just a safety belt. It does not harm... We should probably refactor this code to be a bit moe accurate though ... -- Regards, Cordialement, Emmanuel Lécharny www.iktek.com
