directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel Lecharny <elecha...@gmail.com>
Subject Re: svn commit: r951314 - /directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/util/StringTools.java
Date Fri, 04 Jun 2010 08:09:27 GMT
On 6/4/10 9:50 AM, felixk@apache.org wrote:
> Author: felixk
> Date: Fri Jun  4 07:50:56 2010
> New Revision: 951314
>
> URL: http://svn.apache.org/viewvc?rev=951314&view=rev
> Log:
> Don't catch RuntimeExceptions accidentally
>    

Good 'catch' :-)

  btw, there are many places where we throw NullPointerExceptions 
(mainly when checking the methods arguments, which is wrong : we should 
throw a IllegalArgumentException instead).


-- 
Regards,
Cordialement,
Emmanuel L├ęcharny
www.nextury.com



Mime
View raw message