You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@maven.apache.org by hb...@apache.org on 2019/01/23 19:50:41 UTC

[maven] 01/01: [MNG-6571] add synchronization since require to be threadsafe

This is an automated email from the ASF dual-hosted git repository.

hboutemy pushed a commit to branch MNG-6571
in repository https://gitbox.apache.org/repos/asf/maven.git

commit b43bdd41eff12d87c61ed1fe7cffdd31ec6905e9
Author: Hervé Boutemy <hb...@apache.org>
AuthorDate: Wed Jan 23 18:45:23 2019 +0100

    [MNG-6571] add synchronization since require to be threadsafe
---
 .../main/java/org/apache/maven/artifact/versioning/VersionRange.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java b/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java
index fb6c0ad..68d8b30 100644
--- a/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java
+++ b/maven-artifact/src/main/java/org/apache/maven/artifact/versioning/VersionRange.java
@@ -23,6 +23,7 @@ import java.util.ArrayList;
 import java.util.Collections;
 import java.util.Iterator;
 import java.util.List;
+import java.util.Map;
 import java.util.WeakHashMap;
 
 import org.apache.maven.artifact.Artifact;
@@ -34,7 +35,8 @@ import org.apache.maven.artifact.Artifact;
  */
 public class VersionRange
 {
-    private static final WeakHashMap<String, VersionRange> CACHE = new WeakHashMap<>();
+    private static final Map<String, VersionRange> CACHE =
+        Collections.<String, VersionRange>synchronizedMap( new WeakHashMap<String, VersionRange>() );
 
     private final ArtifactVersion recommendedVersion;