[ 
https://issues.apache.org/jira/browse/LANG-753?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13155697#comment-13155697
 ] 

Oliver Siegmar commented on LANG-753:
-------------------------------------

Consider that there are currently many more 2.6 users than 3.0 users. So 
breaking 3.0 compatibility is not as bad as breaking 2.6 compatibility if the 
change would be made *now*. This breaking changed should be documented very 
well, of course. Probably it could also be a 3.1 verison.

>From my point of view, a NullPointerException is the wrong type of Exception 
>anyway. A NullPointerException should be thrown by the JVM only if an 
>operation is executed on a null reference accidently. If I want to ensure that 
>a method argument is required (must not be null) I'd like to throw an 
>IllegalArgumentException because that describes what really happend - an 
>illegal argument (null in this case) was passed to the method.

                
> Validate.notNull() should throw IllegalArgumentException but throws 
> NullPointerException
> ----------------------------------------------------------------------------------------
>
>                 Key: LANG-753
>                 URL: https://issues.apache.org/jira/browse/LANG-753
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.*
>    Affects Versions: 3.0.1
>            Reporter: Oliver Siegmar
>
> Validate.notNull() throws an IllegalArgumentException in commons-lang 2.6, 
> but in 3.0.1 it throws a NullPointerException. I can't find any note about 
> this in the release notes. I think this is a regression, because it could 
> break others code.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to