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 17:45:25 UTC

[maven] branch MNG-6571 updated: [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


The following commit(s) were added to refs/heads/MNG-6571 by this push:
     new 106dc11  [MNG-6571] add synchronization since require to be threadsafe
106dc11 is described below

commit 106dc1173fb1be0b869c18a1b62dddadf29a4696
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..ea33587 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;