[GitHub] commons-lang issue #342: LANG-1411: Add empty checks to ObjectUtils

2018-08-17 Thread kinow
Github user kinow commented on the issue:

https://github.com/apache/commons-lang/pull/342
  
Hi @nictas , thanks for your contribution. The code looks really good, well 
commented and formatted with unit tests. :clap: :clap: 

And you seem to have a valid use case for it. I think I could even get used 
to calling this method in some of my projects.

Leaving my +1 here, but leaving it for others to review. Would be 
especially good someone with more time to confirm we don't have anything 
similar, or that could be chained, etc, I think. 

Thank you again
Bruno


---


[GitHub] commons-lang issue #342: LANG-1411: Add empty checks to ObjectUtils

2018-08-16 Thread coveralls
Github user coveralls commented on the issue:

https://github.com/apache/commons-lang/pull/342
  

[![Coverage 
Status](https://coveralls.io/builds/18531577/badge)](https://coveralls.io/builds/18531577)

Coverage increased (+0.01%) to 95.26% when pulling 
**2a11642511a3c69bac5aa1abd83d3219871395e3 on nictas:master** into 
**c3de2d69ce9ad778a0bc22971cb8ff36dd0ee062 on apache:master**.



---