-
Bug
-
Resolution: Fixed
-
P5
-
16, 17
-
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.
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.