RFR: 8368775: Remove outdated comment in OutlineTextRenderer

Matthias Baesken mbaesken at openjdk.org
Thu Oct 2 07:18:02 UTC 2025


On Fri, 26 Sep 2025 18:54:03 GMT, Phil Race <prr at openjdk.org> wrote:

> > You can use str.isEmpty() here.
> 
> I was actually going for consistency with all of the other optimizations of this type, which all use a length check. I can change it to `isEmpty` if you feel strongly about it, though.
> 

Hi, you replaced this strange `''".equals`  check ; is this because it looks not nice compared to length or isEmpty or for other reasons ? I ask because there are quite a few of those checks in the codebase (below are only some of them) .



src/java.base/share/classes/java/lang/runtime/ObjectMethods.java:416:                List<String> nameList = "".equals(names) ? List.of() : List.of(names.split(";"));
src/java.base/share/classes/java/net/NetworkInterface.java:230:        return "".equals(displayName) ? null : displayName;
src/java.base/share/classes/java/net/SocketPermission.java:885:        if (this.wildcard && "".equals(this.cname))
src/java.base/share/classes/java/security/CodeSource.java:466:            if (("".equals(thisHost) || "localhost".equals(thisHost)) &&
src/java.base/share/classes/java/security/CodeSource.java:467:                ("".equals(thatHost) || "localhost".equals(thatHost))) {
src/java.base/share/classes/java/text/CompactNumberFormat.java:2546:            !"".equals(compactPatterns[index])) { // ignore empty pattern
src/java.base/share/classes/sun/security/tools/keytool/Main.java:931:            if ("".equals(dest)) {
src/java.base/share/classes/sun/security/tools/keytool/Main.java:939:            if ("".equals(alias)) {
src/java.base/share/classes/sun/security/tools/keytool/Main.java:2510:                    if ("".equals(newAlias)) {
src/java.base/share/classes/sun/security/util/SecurityProperties.java:155:        if ("".equals(rawPropVal)) {
src/java.base/share/classes/sun/util/locale/provider/LocaleResources.java:604:        } else if ("".equals(pattern)) {
src/java.desktop/macosx/classes/com/apple/laf/AquaTextFieldSearch.java:246:        if (!text.hasFocus() && "".equals(text.getText())) {
src/java.desktop/macosx/classes/com/apple/laf/AquaTextFieldSearch.java:290:        button.setVisible(!"".equals(text.getText()));
src/java.desktop/share/classes/com/sun/media/sound/JDK13Services.java:175:        if ("".equals(value)) {
src/java.desktop/share/classes/java/awt/FileDialog.java:153:                    fileDialog.file = ("".equals(file)) ? null : file;
src/java.desktop/share/classes/java/awt/FileDialog.java:156:                    fileDialog.dir = ("".equals(directory)) ? null : directory;
src/java.desktop/share/classes/java/beans/PropertyDescriptor.java:101:        if ("".equals(readMethodName) || "".equals(writeMethodName)) {
src/java.desktop/share/classes/javax/swing/JTable.java:5567:                if ("".equals(s)) {
src/java.desktop/share/classes/javax/swing/plaf/basic/BasicComboBoxUI.java:1450:                            (!(value instanceof String) || !"".equals(value))) {
src/java.desktop/share/classes/javax/swing/plaf/synth/SynthComboBoxUI.java:551:                if ("".equals(text)) {
src/java.desktop/share/classes/javax/swing/plaf/synth/SynthMenuItemLayoutHelper.java:276:        if (useCheckAndArrow() && (!"".equals(getAccText()))) {
src/java.desktop/share/classes/javax/swing/text/html/CSS.java:1064:        if (borderValue == HTML.NULL_ATTRIBUTE_VALUE || "".equals(borderValue)) {
src/java.desktop/share/classes/javax/swing/undo/AbstractUndoableEdit.java:230:        if (!"".equals(name)) {
src/java.desktop/share/classes/javax/swing/undo/AbstractUndoableEdit.java:257:        if (!"".equals(name)) {
src/java.desktop/share/classes/sun/font/TrueTypeFont.java:696:        if ("".equals(encoding)) {
src/java.desktop/share/classes/sun/font/TrueTypeFont.java:1493:        while (!"".equals(key)) {
src/java.desktop/share/classes/sun/java2d/pipe/OutlineTextRenderer.java:76:        if ("".equals(str)) {
src/java.naming/share/classes/com/sun/jndi/url/ldap/ldapURLContext.java:76:        if (!"".equals(dn)) {
src/java.naming/share/classes/com/sun/jndi/url/ldap/ldapURLContextFactory.java:65:        if (!"".equals(dn)) {
src/java.net.http/share/classes/jdk/internal/net/http/HeaderParser.java:232:            if (val != null && "".equals (val)) {
src/java.net.http/share/classes/jdk/internal/net/http/ResponseBodyHandlers.java:215:            if ("".equals(src))

-------------

PR Comment: https://git.openjdk.org/jdk/pull/27523#issuecomment-3359526026


More information about the client-libs-dev mailing list