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/24 07:11:10 UTC

[maven] branch MNG-6571 updated: [MNG-6571] add cache also to createFromVersion(String) factory method

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 47dad72  [MNG-6571] add cache also to createFromVersion(String) factory method
47dad72 is described below

commit 47dad72e8d4479ba759e061e1b6ce48e97732fd4
Author: Hervé Boutemy <hb...@apache.org>
AuthorDate: Thu Jan 24 08:11:08 2019 +0100

    [MNG-6571] add cache also to createFromVersion(String) factory method
---
 .../java/org/apache/maven/artifact/versioning/VersionRange.java  | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

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 68d8b30..2b5de99 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
@@ -236,8 +236,13 @@ public class VersionRange
 
     public static VersionRange createFromVersion( String version )
     {
-        List<Restriction> restrictions = Collections.emptyList();
-        return new VersionRange( new DefaultArtifactVersion( version ), restrictions );
+        VersionRange cached = CACHE.get( version );
+        if ( cached == null )
+        {
+            List<Restriction> restrictions = Collections.emptyList();
+            cached = new VersionRange( new DefaultArtifactVersion( version ), restrictions );
+        }
+        return cached;
     }
 
     /**