--- a/src/java.management/share/classes/javax/management/ImmutableDescriptor.java	Fri Dec 07 17:11:17 2018 +0100
+++ b/src/java.management/share/classes/javax/management/ImmutableDescriptor.java	Fri Dec 07 11:51:17 2018 -0500
@@ -110,7 +110,7 @@
                 new TreeMap<String, Object>(String.CASE_INSENSITIVE_ORDER);
         for (Map.Entry<String, ?> entry : fields.entrySet()) {
             String name = entry.getKey();
-            if (name == null || name.equals(""))
+            if (name == null || name.isEmpty())
                 throw new IllegalArgumentException("Empty or null field name");
             if (map.containsKey(name))
                 throw new IllegalArgumentException("Duplicate name: " + name);
@@ -166,7 +166,7 @@
                 new TreeMap<String, Object>(String.CASE_INSENSITIVE_ORDER);
         for (int i = 0; i < fieldNames.length; i++) {
             String name = fieldNames[i];
-            if (name == null || name.equals(""))
+            if (name == null || name.isEmpty())
                 throw new IllegalArgumentException("Empty or null field name");
             Object old = map.put(name, fieldValues[i]);
             if (old != null) {
@@ -333,7 +333,7 @@
         Object[] result = new Object[fieldNames.length];
         for (int i = 0; i < fieldNames.length; i++) {
             String name = fieldNames[i];
-            if (name != null && !name.equals(""))
+            if (name != null && !name.isEmpty())
                 result[i] = getFieldValue(name);
         }
         return result;
@@ -543,7 +543,7 @@
     }
 
     private static void checkIllegalFieldName(String name) {
-        if (name == null || name.equals(""))
+        if (name == null || name.isEmpty())
             illegal("Null or empty field name");
     }