Changeset 2245

Timestamp:

12/11/17 12:13:10 (7 years ago)

Author:

pharms

Message:

• solved some findbugs issues

File:

• trunk/autoquest-htmlmonitor/src/main/java/de/ugoe/cs/autoquest/htmlmonitor/HtmlMonitorLogManager.java (modified) (2 diffs)

trunk/autoquest-htmlmonitor/src/main/java/de/ugoe/cs/autoquest/htmlmonitor/HtmlMonitorLogManager.java

@@ -115 +115 @@
      */
     @Override
-    public void handleMessage(HtmlClientInfos clientInfos,
-                              HtmlGUIElement  guiStructure,
-                              HtmlEvent[]     events)
+    public synchronized void handleMessage(HtmlClientInfos clientInfos,
+                                           HtmlGUIElement  guiStructure,
+                                           HtmlEvent[]     events)
     {
         HtmlMonitorOutputWriter writer =
@@ -124 +124 @@
         try {
             if (writer == null) {
-                synchronized (this) {
-                    writer = writers.get(clientInfos.getClientId() + clientInfos.getWebAppId());
-                    if (writer == null) {
-                        writer = new HtmlMonitorOutputWriter
-                            (logFileBaseDir, clientInfos.getWebAppId(),
-                             clientInfos.getClientId(), clientInfos.getUserAgent());
+                writer = new HtmlMonitorOutputWriter
+                    (logFileBaseDir, clientInfos.getWebAppId(),
+                     clientInfos.getClientId(), clientInfos.getUserAgent());
 
-                        writer.init();
-                        writer.start();
-                        writers.put(clientInfos.getClientId() + clientInfos.getWebAppId(), writer);
-                    }
-                }
+                writer.init();
+                writer.start();
+                writers.put(clientInfos.getClientId() + clientInfos.getWebAppId(), writer);
             }