< prev index next >

src/java.naming/share/classes/com/sun/jndi/toolkit/ctx/Continuation.java

Print this page
rev 52881 : 8214971: Replace use of string.equals("") with isEmpty()
Reviewed-by: jlaskey, prappo, lancea, dfuchs, redestad

@@ -201,11 +201,11 @@
      * @see #setErrorNNS(java.lang.Object, javax.naming.Name)
      */
     public void setErrorNNS(Object resObj, String remain) {
         CompositeName rname = new CompositeName();
         try {
-            if (remain != null && !remain.equals(""))
+            if (remain != null && !remain.isEmpty())
                 rname.add(remain);
 
             rname.add("");
         } catch (InvalidNameException e) {
             // ignore, can't happen for composite name

@@ -245,11 +245,11 @@
      *
      * @see #setError(java.lang.Object, javax.naming.Name)
      */
     public void setError(Object resObj, String remain) {
         CompositeName rname = new CompositeName();
-        if (remain != null && !remain.equals("")) {
+        if (remain != null && !remain.isEmpty()) {
             try {
                 rname.add(remain);
             } catch (InvalidNameException e) {
                 // ignore; can't happen for composite name
             }

@@ -373,18 +373,18 @@
      * @see #setContinue(java.lang.Object, java.lang.String, javax.naming.Context, java.lang.String)
      */
     public void setContinue(Object obj, String relResName,
         Context currCtx, String remain) {
         CompositeName relname = new CompositeName();
-        if (!relResName.equals("")) {
+        if (!relResName.isEmpty()) {
             try {
                 relname.add(relResName);
             } catch (NamingException e){}
         }
 
         CompositeName rname = new CompositeName();
-        if (!remain.equals("")) {
+        if (!remain.isEmpty()) {
             try {
                 rname.add(remain);
             } catch (NamingException e) {
             }
         }
< prev index next >