From 46d90b978587ac1ad9572adea13ca9e5bc520a6f Mon Sep 17 00:00:00 2001 From: "marcins78@gmail.com" Date: Mon, 30 Jul 2012 20:19:13 +0000 Subject: [PATCH] Fixing multi-threaded issues. --- .../com/gempukku/lotro/game/LotroServer.java | 16 +++------------- 1 file changed, 3 insertions(+), 13 deletions(-) diff --git a/gemp-lotr/gemp-lotr-server/src/main/java/com/gempukku/lotro/game/LotroServer.java b/gemp-lotr/gemp-lotr-server/src/main/java/com/gempukku/lotro/game/LotroServer.java index 2ab6f7e4c..8a8503958 100644 --- a/gemp-lotr/gemp-lotr-server/src/main/java/com/gempukku/lotro/game/LotroServer.java +++ b/gemp-lotr/gemp-lotr-server/src/main/java/com/gempukku/lotro/game/LotroServer.java @@ -131,24 +131,14 @@ public class LotroServer extends AbstractServer { new GameResultListener() { @Override public void gameFinished(String winnerPlayerId, String winReason, Map loserPlayerIdsWithReasons) { - _lock.writeLock().lock(); - try { - final Map.Entry loserEntry = loserPlayerIdsWithReasons.entrySet().iterator().next(); + final Map.Entry loserEntry = loserPlayerIdsWithReasons.entrySet().iterator().next(); - gameRecordingInProgress.finishRecording(winnerPlayerId, winReason, loserEntry.getKey(), loserEntry.getValue()); - } finally { - _lock.writeLock().unlock(); - } + gameRecordingInProgress.finishRecording(winnerPlayerId, winReason, loserEntry.getKey(), loserEntry.getValue()); } @Override public void gameCancelled() { - _lock.writeLock().lock(); - try { - gameRecordingInProgress.finishRecording(participants[0].getPlayerId(), "Game cancelled due to error", participants[1].getPlayerId(), "Game cancelled due to error"); - } finally { - _lock.writeLock().unlock(); - } + gameRecordingInProgress.finishRecording(participants[0].getPlayerId(), "Game cancelled due to error", participants[1].getPlayerId(), "Game cancelled due to error"); } } );