diff --git a/src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessible.java b/src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessible.java index 72bfa8d541e47..3a66e456956c1 100644 --- a/src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessible.java +++ b/src/java.desktop/macosx/classes/sun/lwawt/macosx/CAccessible.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2011, 2019, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2011, 2021, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -115,19 +115,19 @@ public void propertyChange(PropertyChangeEvent e) { if ( ptr != 0 ) { Object newValue = e.getNewValue(); Object oldValue = e.getOldValue(); - if (name.compareTo(ACCESSIBLE_CARET_PROPERTY) == 0) { + if (name.equals(ACCESSIBLE_CARET_PROPERTY)) { selectedTextChanged(ptr); - } else if (name.compareTo(ACCESSIBLE_TEXT_PROPERTY) == 0) { + } else if (name.equals(ACCESSIBLE_TEXT_PROPERTY)) { valueChanged(ptr); - } else if (name.compareTo(ACCESSIBLE_SELECTION_PROPERTY) == 0) { + } else if (name.equals(ACCESSIBLE_SELECTION_PROPERTY)) { selectionChanged(ptr); - } else if (name.compareTo(ACCESSIBLE_TABLE_MODEL_CHANGED) == 0) { + } else if (name.equals(ACCESSIBLE_TABLE_MODEL_CHANGED)) { valueChanged(ptr); - } else if (name.compareTo(ACCESSIBLE_ACTIVE_DESCENDANT_PROPERTY) == 0 ) { + } else if (name.equals(ACCESSIBLE_ACTIVE_DESCENDANT_PROPERTY)) { if (newValue instanceof AccessibleContext) { activeDescendant = (AccessibleContext)newValue; } - } else if (name.compareTo(ACCESSIBLE_STATE_PROPERTY) == 0) { + } else if (name.equals(ACCESSIBLE_STATE_PROPERTY)) { AccessibleContext thisAC = accessible.getAccessibleContext(); AccessibleRole thisRole = thisAC.getAccessibleRole(); Accessible parentAccessible = thisAC.getAccessibleParent(); @@ -167,12 +167,12 @@ public void propertyChange(PropertyChangeEvent e) { if (thisRole == AccessibleRole.CHECK_BOX) { valueChanged(ptr); } - } else if (name.compareTo(ACCESSIBLE_NAME_PROPERTY) == 0) { + } else if (name.equals(ACCESSIBLE_NAME_PROPERTY)) { //for now trigger only for JTabbedPane. if (e.getSource() instanceof JTabbedPane) { titleChanged(ptr); } - } else if (name.compareTo(ACCESSIBLE_VALUE_PROPERTY) == 0) { + } else if (name.equals(ACCESSIBLE_VALUE_PROPERTY)) { AccessibleRole thisRole = accessible.getAccessibleContext() .getAccessibleRole(); if (thisRole == AccessibleRole.SLIDER || diff --git a/src/java.desktop/share/classes/javax/swing/JList.java b/src/java.desktop/share/classes/javax/swing/JList.java index 83b6cbed5530f..7932209974bcc 100644 --- a/src/java.desktop/share/classes/javax/swing/JList.java +++ b/src/java.desktop/share/classes/javax/swing/JList.java @@ -2947,7 +2947,7 @@ public void propertyChange(PropertyChangeEvent e) { Object newValue = e.getNewValue(); // re-set listData listeners - if (name.compareTo("model") == 0) { + if (name.equals("model")) { if (oldValue != null && oldValue instanceof ListModel) { ((ListModel) oldValue).removeListDataListener(this); @@ -2957,7 +2957,7 @@ public void propertyChange(PropertyChangeEvent e) { } // re-set listSelectionModel listeners - } else if (name.compareTo("selectionModel") == 0) { + } else if (name.equals("selectionModel")) { if (oldValue != null && oldValue instanceof ListSelectionModel) { ((ListSelectionModel) oldValue).removeListSelectionListener(this); diff --git a/src/java.desktop/share/classes/javax/swing/JTable.java b/src/java.desktop/share/classes/javax/swing/JTable.java index b4d3037e5726a..4b3e02f860f0c 100644 --- a/src/java.desktop/share/classes/javax/swing/JTable.java +++ b/src/java.desktop/share/classes/javax/swing/JTable.java @@ -6756,7 +6756,7 @@ public void propertyChange(PropertyChangeEvent e) { Object newValue = e.getNewValue(); // re-set tableModel listeners - if (name.compareTo("model") == 0) { + if (name.equals("model")) { if (oldValue != null && oldValue instanceof TableModel) { ((TableModel) oldValue).removeTableModelListener(this); @@ -6766,7 +6766,7 @@ public void propertyChange(PropertyChangeEvent e) { } // re-set selectionModel listeners - } else if (name.compareTo("selectionModel") == 0) { + } else if (name.equals("selectionModel")) { Object source = e.getSource(); if (source == JTable.this) { // row selection model @@ -6797,7 +6797,7 @@ public void propertyChange(PropertyChangeEvent e) { // re-set columnModel listeners // and column's selection property listener as well - } else if (name.compareTo("columnModel") == 0) { + } else if (name.equals("columnModel")) { if (oldValue != null && oldValue instanceof TableColumnModel) { TableColumnModel tcm = (TableColumnModel) oldValue; @@ -6811,7 +6811,7 @@ public void propertyChange(PropertyChangeEvent e) { } // re-se cellEditor listeners - } else if (name.compareTo("tableCellEditor") == 0) { + } else if (name.equals("tableCellEditor")) { if (oldValue != null && oldValue instanceof TableCellEditor) { ((TableCellEditor) oldValue).removeCellEditorListener(this);