You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@directory.apache.org by Emmanuel Lecharny <el...@gmail.com> on 2010/06/04 10:09:27 UTC

Re: svn commit: r951314 - /directory/shared/trunk/ldap/src/main/java/org/apache/directory/shared/ldap/util/StringTools.java

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