[
https://issues.apache.org/jira/browse/LANG-1410?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16713438#comment-16713438
]
Rostislav Krasny commented on LANG-1410:
----------------------------------------
Hi [[email protected]],
I made a PR with similar approach of eliminating unnecessary local variable
from isBlank() and from isWhitespace(), but in my case of the "i" variable:
[https://github.com/apache/commons-lang/pull/392]
This way I don't call the cs.length() method too many times.
I've also added more tests and fixed formatting of two unchanged methods that
use isBlank().
> StringsUtil.isBlank check null first
> ------------------------------------
>
> Key: LANG-1410
> URL: https://issues.apache.org/jira/browse/LANG-1410
> Project: Commons Lang
> Issue Type: Improvement
> Components: lang.*
> Reporter: Yifan Guo
> Priority: Minor
>
> StringsUtil.isBlank should check null for cs first instead of declaring int
> strLen first.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
