Make synchronized variables final
diff --git a/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java b/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java
index 74f9702..360820c 100644
--- a/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java
+++ b/src/main/java/org/apache/log4j/chainsaw/LogPanelPreferenceModel.java
@@ -51,7 +51,7 @@
Properties properties = SettingsManager.getInstance().getDefaultSettings();
for (Map.Entry<Object, Object> objectObjectEntry : properties.entrySet()) {
- Map.Entry entry = (Map.Entry) objectObjectEntry;
+ Map.Entry entry = objectObjectEntry;
if (entry.getKey().toString().startsWith("DateFormat")) {
list.add(entry.getValue());
diff --git a/src/main/java/org/apache/log4j/chainsaw/LogUI.java b/src/main/java/org/apache/log4j/chainsaw/LogUI.java
index 7454f89..cbaee9a 100644
--- a/src/main/java/org/apache/log4j/chainsaw/LogUI.java
+++ b/src/main/java/org/apache/log4j/chainsaw/LogUI.java
@@ -178,7 +178,7 @@
* until notified.
*/
private boolean isGUIFullyInitialized = false;
- private Object initializationLock = new Object();
+ private final Object initializationLock = new Object();
/**
* The shutdownAction is called when the user requests to exit Chainsaw, and
diff --git a/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java b/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java
index 87c5945..a1801ff 100644
--- a/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java
+++ b/src/main/java/org/apache/log4j/chainsaw/layout/EventDetailLayout.java
@@ -38,7 +38,7 @@
* @author Paul Smith <psmith@apache.org>
*/
public class EventDetailLayout extends Layout {
- private EnhancedPatternLayout patternLayout = new EnhancedPatternLayout();
+ private final EnhancedPatternLayout patternLayout = new EnhancedPatternLayout();
public EventDetailLayout() {
}
diff --git a/src/main/java/org/apache/log4j/chainsaw/prefs/LoadSettingsEvent.java b/src/main/java/org/apache/log4j/chainsaw/prefs/LoadSettingsEvent.java
index 19df785..d0b8f0b 100644
--- a/src/main/java/org/apache/log4j/chainsaw/prefs/LoadSettingsEvent.java
+++ b/src/main/java/org/apache/log4j/chainsaw/prefs/LoadSettingsEvent.java
@@ -66,7 +66,7 @@
Collection c = new ArrayList();
for (Map.Entry<Object, Object> objectObjectEntry : properties.entrySet()) {
- Map.Entry entry = (Map.Entry) objectObjectEntry;
+ Map.Entry entry = objectObjectEntry;
if (entry.getKey().toString().startsWith(string)) {
c.add(entry.getValue());
}
diff --git a/src/main/java/org/apache/log4j/chainsaw/vfs/VFSLogFilePatternReceiver.java b/src/main/java/org/apache/log4j/chainsaw/vfs/VFSLogFilePatternReceiver.java
index 8b4470d..4a733ef 100644
--- a/src/main/java/org/apache/log4j/chainsaw/vfs/VFSLogFilePatternReceiver.java
+++ b/src/main/java/org/apache/log4j/chainsaw/vfs/VFSLogFilePatternReceiver.java
@@ -167,7 +167,7 @@
private boolean promptForUserInfo = false;
private Container container;
- private Object waitForContainerLock = new Object();
+ private final Object waitForContainerLock = new Object();
private boolean autoReconnect;
private VFSReader vfsReader;
diff --git a/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java b/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java
index 89c16a5..93b69f9 100644
--- a/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java
+++ b/src/main/java/org/apache/log4j/chainsaw/zeroconf/ZeroConfPlugin.java
@@ -88,7 +88,7 @@
private ZeroConfPreferenceModel preferenceModel;
- private Map serviceInfoToReceiveMap = new HashMap();
+ private final Map serviceInfoToReceiveMap = new HashMap();
private JMenu connectToMenu = new JMenu("Connect to");
private JMenuItem helpItem = new JMenuItem(new AbstractAction("Learn more about ZeroConf...",
diff --git a/src/main/java/org/apache/log4j/net/MulticastReceiver.java b/src/main/java/org/apache/log4j/net/MulticastReceiver.java
index 5e4f283..f0a542f 100644
--- a/src/main/java/org/apache/log4j/net/MulticastReceiver.java
+++ b/src/main/java/org/apache/log4j/net/MulticastReceiver.java
@@ -174,7 +174,7 @@
}
class MulticastHandlerThread extends Thread {
- private List list = new ArrayList();
+ private final List list = new ArrayList();
public MulticastHandlerThread() {
setDaemon(true);
diff --git a/src/main/java/org/apache/log4j/net/SocketHubReceiver.java b/src/main/java/org/apache/log4j/net/SocketHubReceiver.java
index 73f88e5..5bef1e1 100644
--- a/src/main/java/org/apache/log4j/net/SocketHubReceiver.java
+++ b/src/main/java/org/apache/log4j/net/SocketHubReceiver.java
@@ -85,7 +85,7 @@
/**
* Listener list.
*/
- private List listenerList = Collections.synchronizedList(new ArrayList());
+ private final List listenerList = Collections.synchronizedList(new ArrayList());
private boolean advertiseViaMulticastDNS;
private ZeroConfSupport zeroConf;
diff --git a/src/main/java/org/apache/log4j/net/SocketNode13.java b/src/main/java/org/apache/log4j/net/SocketNode13.java
index fb7a585..1d5d77c 100644
--- a/src/main/java/org/apache/log4j/net/SocketNode13.java
+++ b/src/main/java/org/apache/log4j/net/SocketNode13.java
@@ -74,7 +74,7 @@
/**
* List of listeners.
*/
- private List listenerList = Collections.synchronizedList(new ArrayList());
+ private final List listenerList = Collections.synchronizedList(new ArrayList());
diff --git a/src/main/java/org/apache/log4j/net/SocketReceiver.java b/src/main/java/org/apache/log4j/net/SocketReceiver.java
index 6c482f0..821935f 100644
--- a/src/main/java/org/apache/log4j/net/SocketReceiver.java
+++ b/src/main/java/org/apache/log4j/net/SocketReceiver.java
@@ -87,7 +87,7 @@
/**
* Listeners.
*/
- private List listenerList = Collections.synchronizedList(new ArrayList());
+ private final List listenerList = Collections.synchronizedList(new ArrayList());
private boolean advertiseViaMulticastDNS;
private ZeroConfSupport zeroConf;
diff --git a/src/main/java/org/apache/log4j/net/UDPReceiver.java b/src/main/java/org/apache/log4j/net/UDPReceiver.java
index bc927ab..562d495 100644
--- a/src/main/java/org/apache/log4j/net/UDPReceiver.java
+++ b/src/main/java/org/apache/log4j/net/UDPReceiver.java
@@ -171,7 +171,7 @@
}
class UDPHandlerThread extends Thread {
- private List list = new ArrayList();
+ private final List list = new ArrayList();
public UDPHandlerThread() {
setDaemon(true);