libbluray | branch: master | hpi1 <[email protected]> | Wed Aug 27 10:06:46
2014 +0300| [80342e5ea3a616330c80cc7df3621c43c3574884] | committer: hpi1
HScene: remove incorrect synchronized() {}
- Synchronizing linked list add/remove against linked list head is useless
(head element can change).
- List usage was not synchronized.
- HAVi does not require [add/remove]WindowListener to be thread safe.
> http://git.videolan.org/gitweb.cgi/libbluray.git/?a=commit;h=80342e5ea3a616330c80cc7df3621c43c3574884
---
src/libbluray/bdj/java/org/havi/ui/HScene.java | 8 ++------
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/src/libbluray/bdj/java/org/havi/ui/HScene.java
b/src/libbluray/bdj/java/org/havi/ui/HScene.java
index 372ad05..52888b6 100644
--- a/src/libbluray/bdj/java/org/havi/ui/HScene.java
+++ b/src/libbluray/bdj/java/org/havi/ui/HScene.java
@@ -181,15 +181,11 @@ public class HScene extends Container implements
HComponentOrdering {
}
public void addWindowListener(WindowListener listener) {
- synchronized (windowListener) {
- windowListener = HEventMulticaster.add(windowListener, listener);
- }
+ windowListener = HEventMulticaster.add(windowListener, listener);
}
public void removeWindowListener(WindowListener listener) {
- synchronized (windowListener) {
- windowListener = HEventMulticaster.remove(windowListener,
listener);
- }
+ windowListener = HEventMulticaster.remove(windowListener, listener);
}
protected void processWindowEvent(WindowEvent event) {
_______________________________________________
libbluray-devel mailing list
[email protected]
https://mailman.videolan.org/listinfo/libbluray-devel