commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Work logged] (LANG-1410) StringsUtil.isBlank check null first
Date Thu, 20 Dec 2018 03:56:00 GMT


ASF GitHub Bot logged work on LANG-1410:

                Author: ASF GitHub Bot
            Created on: 20/Dec/18 03:55
            Start Date: 20/Dec/18 03:55
    Worklog Time Spent: 10m 
      Work Description: MarkDacek commented on a change in pull request #392: LANG-1410: eliminate
one unnecessary local int variable and add more tests

 File path: src/main/java/org/apache/commons/lang3/
 @@ -338,15 +338,17 @@ public static boolean isAllEmpty(final CharSequence... css) {
      * @since 3.0 Changed signature from isBlank(String) to isBlank(CharSequence)
     public static boolean isBlank(final CharSequence cs) {
-        int strLen;
 Review comment:
   Hm, I've given another look through and it would appear that other uses of `sz` are final
as well. 
   I don't object to this, but I would question the need for it as the memory gain isn't noticeable.

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:

Issue Time Tracking

    Worklog Id:     (was: 177324)
    Time Spent: 1h 50m  (was: 1h 40m)

> StringsUtil.isBlank check null first
> ------------------------------------
>                 Key: LANG-1410
>                 URL:
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.*
>            Reporter: Yifan Guo
>            Priority: Minor
>          Time Spent: 1h 50m
>  Remaining Estimate: 0h
> StringsUtil.isBlank should check null for cs first instead of declaring int strLen first.

This message was sent by Atlassian JIRA

View raw message