RFR 8214971 : Replace use of string.equals("") with isEmpty()

Lance Andersen lance.andersen at oracle.com
Thu Dec 6 23:19:12 UTC 2018


+1
> On Dec 6, 2018, at 6:03 PM, Roger Riggs <roger.riggs at oracle.com> wrote:
> 
> I missed reverting other Apache source files that should be unchanged.
> 
> Webrev:
> http://cr.openjdk.java.net/~rriggs/webrev-string-isempty-8214971-3/
> 
> Thanks, Roger
> 
> 
> On 12/6/18 5:37 PM, Roger Riggs wrote:
>> Hi Claes,
>> 
>> Thanks for the reminder about xalan sources.
>> 
>> Updated to revert changes to xalan.
>> 
>> Webrev:
>> http://cr.openjdk.java.net/~rriggs/webrev-string-isempty-8214971-2/
>> 
>> Thanks, Roger
>> 
>> On 12/6/18 4:48 PM, Claes Redestad wrote:
>>> Should changes to various com.sun.org.apache.xalan classes in java.xml be deferred to upstream?
>>> 
>>> Otherwise looks good to me!
>>> 
>>> /Claes
>>> 
>>> On 2018-12-06 21:04, Roger Riggs wroe te:
>>>> Please review changing string.equals("") to string.isEmpty().
>>>> isEmpty is preferred, performs better and is easier to optimize.
>>>> 
>>>> The change is a literal substitution in all files and only in these modules:
>>>> 
>>>>    java.base
>>>>    java.logging
>>>>    java.management
>>>>    java.management.rmi
>>>>    java.naming
>>>>    java.net.http
>>>>    java.prefs
>>>>    java.rmi
>>>>    java.scripting
>>>>    java.sql
>>>>    java.sql.rowset
>>>>    java.xml
>>>>    jdk.jartool
>>>>    jdk.javadoc
>>>>    jdk.jcmd
>>>>    jdk.jconsole
>>>>    jdk.management.agent
>>>>    jdk.naming.dns
>>>>    jdk.rmic
>>>> 
>>>> 
>>>> Webrev:
>>>> http://cr.openjdk.java.net/~rriggs/webrev-string-isempty-8214971-1/
>>>> 
>>>> Issue:
>>>>   https://bugs.openjdk.java.net/browse/JDK-8214971
>>>> 
>>>> Thanks, Roger
>>> 
>> 
> 

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>





More information about the core-libs-dev mailing list