[
https://issues.apache.org/jira/browse/LANG-781?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15240045#comment-15240045
]
ASF GitHub Bot commented on LANG-781:
-------------------------------------
Github user garydgregory commented on the pull request:
https://github.com/apache/commons-lang/pull/108#issuecomment-209650500
Yes:
- Thank you for the patch.
- The Javadoc has some bogus text, like starting with "<p></p>". You do not
need that.
- Do not use a for-each loop to traverse an array. This creates an iterator
when none is needed. Use a normal for loop, which will not create the extra
Iterator to GC. This matters when a libraries is used on the critical path
and/or environments trying to do low/no GC.
> Null-Check to variable Object parameters - ObjectUtils.isAnyNull
> ----------------------------------------------------------------
>
> Key: LANG-781
> URL: https://issues.apache.org/jira/browse/LANG-781
> Project: Commons Lang
> Issue Type: New Feature
> Components: lang.*
> Affects Versions: 3.1
> Reporter: Sandro Lehmann
> Priority: Minor
> Labels: patch
> Fix For: Discussion
>
> Attachments: ObjectUtils.java.patch, ObjectUtilsTest.java.patch
>
>
> Sometimes it would be very comfortable, to test a few objects to null.
> Instead of writing if(obj1==null || obj2==null || ...), using a method like
> isAnyNull(obj1, obj2, ...) would be much better.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
