Unnecessary reentrant synchronized block in java.awt.Cursor

XMLWordPrintable

    • Type: Bug
    • Resolution: Fixed
    • Priority: P5
    • 18
    • Affects Version/s: 16, 17
    • Component/s: client-libs
    • None
    • b17

      There is lock on a field java.awt.Cursor.systemCustomCursors which is taken twice:
      1. In a method java.awt.Cursor#getSystemCustomCursor:
      ```
                  synchronized(systemCustomCursors) {
                      if (systemCustomCursorProperties == null)
                          loadSystemCustomCursorProperties();
                  }
      ```
      2. And then again in method 'loadSystemCustomCursorProperties' itself
      ```
      private static void loadSystemCustomCursorProperties() throws AWTException {
              synchronized(systemCustomCursors) {
                  systemCustomCursorProperties = new Properties();
      ```

      Such reentrancy is redundant and confusing.

            Assignee:
            Andrey Turbanov
            Reporter:
            Andrey Turbanov
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: