--- old/src/java.management/share/classes/javax/management/MBeanPermission.java 2018-12-06 17:26:00.481383224 -0500 +++ new/src/java.management/share/classes/javax/management/MBeanPermission.java 2018-12-06 17:26:00.273382193 -0500 @@ -256,7 +256,7 @@ if (actions == null) throw new IllegalArgumentException("MBeanPermission: " + "actions can't be null"); - if (actions.equals("")) + if (actions.isEmpty()) throw new IllegalArgumentException("MBeanPermission: " + "actions can't be empty"); @@ -279,7 +279,7 @@ throw new IllegalArgumentException("MBeanPermission name " + "cannot be null"); - if (name.equals("")) + if (name.isEmpty()) throw new IllegalArgumentException("MBeanPermission name " + "cannot be empty"); @@ -310,7 +310,7 @@ // String on = name.substring(openingBracket + 1, name.length() - 1); - if (on.equals("")) + if (on.isEmpty()) objectName = ObjectName.WILDCARD; else if (on.equals("-")) objectName = null; @@ -359,7 +359,7 @@ if (className == null || className.equals("-")) { classNamePrefix = null; classNameExactMatch = false; - } else if (className.equals("") || className.equals("*")) { + } else if (className.isEmpty() || className.equals("*")) { classNamePrefix = ""; classNameExactMatch = false; } else if (className.endsWith(".*")) { @@ -375,7 +375,7 @@ private void setMember(String member) { if (member == null || member.equals("-")) this.member = null; - else if (member.equals("")) + else if (member.isEmpty()) this.member = "*"; else this.member = member;