From fa81a67f3069f270535918a7853b105994440eb3 Mon Sep 17 00:00:00 2001 From: Eugene Zhuravlev Date: Mon, 3 Sep 2012 12:36:06 +0200 Subject: [PATCH] fix unsynchronized access to resulting data container --- .../intellij/compiler/server/DefaultMessageHandler.java | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/java/compiler/impl/src/com/intellij/compiler/server/DefaultMessageHandler.java b/java/compiler/impl/src/com/intellij/compiler/server/DefaultMessageHandler.java index 957f2898a160..72c24441bf2a 100644 --- a/java/compiler/impl/src/com/intellij/compiler/server/DefaultMessageHandler.java +++ b/java/compiler/impl/src/com/intellij/compiler/server/DefaultMessageHandler.java @@ -37,10 +37,7 @@ import org.jetbrains.annotations.Nullable; import org.jetbrains.jps.api.CmdlineProtoUtil; import org.jetbrains.jps.api.CmdlineRemoteProto; -import java.util.Collection; -import java.util.HashSet; -import java.util.Set; -import java.util.UUID; +import java.util.*; import java.util.concurrent.Executor; /** @@ -97,7 +94,7 @@ public abstract class DefaultMessageHandler implements BuilderMessageHandler { final boolean accessChanged = task.getIsAccessChanged(); final boolean isRemoved = task.getIsFieldRemoved(); final Ref isSuccess = Ref.create(Boolean.TRUE); - final Set affectedPaths = new HashSet(); + final Set affectedPaths = Collections.synchronizedSet(new HashSet()); // PsiSearchHelper runs multiple threads try { if (isDumbMode()) { // do not wait until dumb mode finishes