Revert "Caching the parsed NugetVersion and VersionRange objects."

This reverts commit a240062.

src/NuGet.Core/NuGet.ProjectModel/JsonPackageSpecReader.Utf8JsonStreamReader.cs

@@ -319,7 +319,7 @@ private static LibraryDependency ReadLibraryDependency(ref Utf8JsonStreamReader
             {
                 try
                 {
-                    dependencyVersionRange = JsonUtility.ParseVersionRange(dependencyVersionValue);
+                    dependencyVersionRange = VersionRange.Parse(dependencyVersionValue);
                 }
                 catch (Exception ex)
                 {
@@ -489,7 +489,7 @@ private static void ReadDependencies(
                                         var versionPropValue = jsonReader.GetString();
                                         try
                                         {
-                                            versionOverride = JsonUtility.ParseVersionRange(versionPropValue);
+                                            versionOverride = VersionRange.Parse(versionPropValue);
                                         }
                                         catch (Exception ex)
                                         {
@@ -518,7 +518,7 @@ private static void ReadDependencies(
                         {
                             try
                             {
-                                dependencyVersionRange = JsonUtility.ParseVersionRange(dependencyVersionValue);
+                                dependencyVersionRange = VersionRange.Parse(dependencyVersionValue);
                             }
                             catch (Exception ex)
                             {
@@ -628,7 +628,7 @@ private static void ReadCentralPackageVersions(
                         throw FileFormatException.Create("The version cannot be null or empty.", filePath);
                     }
 
-                    centralPackageVersions[propertyName] = new CentralPackageVersion(propertyName, JsonUtility.ParseVersionRange(version));
+                    centralPackageVersions[propertyName] = new CentralPackageVersion(propertyName, VersionRange.Parse(version));
                 }
             }
         }
@@ -735,7 +735,7 @@ private static void ReadDownloadDependencies(
 
                         try
                         {
-                            VersionRange version = JsonUtility.ParseVersionRange(singleVersionValue);
+                            VersionRange version = VersionRange.Parse(singleVersionValue);
 
                             downloadDependencies.Add(new DownloadDependency(name, version));
                         }
@@ -1494,7 +1494,7 @@ private static RuntimeDependencySet ReadRuntimeDependencySet(ref Utf8JsonStreamR
                     var propertyName = jsonReader.GetString();
                     dependencies ??= [];
 
-                    var dependency = new RuntimePackageDependency(propertyName, JsonUtility.ParseVersionRange(jsonReader.ReadNextTokenAsString()));
+                    var dependency = new RuntimePackageDependency(propertyName, VersionRange.Parse(jsonReader.ReadNextTokenAsString()));
 
                     dependencies.Add(dependency);
                 }

src/NuGet.Core/NuGet.ProjectModel/JsonPackageSpecReader.cs

@@ -306,7 +306,7 @@ private static void ReadCentralPackageVersions(
                         filePath);
                 }
 
-                centralPackageVersions[propertyName] = new CentralPackageVersion(propertyName, JsonUtility.ParseVersionRange(version));
+                centralPackageVersions[propertyName] = new CentralPackageVersion(propertyName, VersionRange.Parse(version));
             });
         }
 
@@ -449,7 +449,7 @@ private static void ReadDependencies(
                                     {
                                         try
                                         {
-                                            versionOverride = JsonUtility.ParseVersionRange((string)jsonReader.Value);
+                                            versionOverride = VersionRange.Parse((string)jsonReader.Value);
                                         }
                                         catch (Exception ex)
                                         {
@@ -478,7 +478,7 @@ private static void ReadDependencies(
                     {
                         try
                         {
-                            dependencyVersionRange = JsonUtility.ParseVersionRange(dependencyVersionValue);
+                            dependencyVersionRange = VersionRange.Parse(dependencyVersionValue);
                         }
                         catch (Exception ex)
                         {
@@ -620,7 +620,7 @@ internal static void ReadCentralTransitiveDependencyGroup(
                     {
                         try
                         {
-                            dependencyVersionRange = JsonUtility.ParseVersionRange(dependencyVersionValue);
+                            dependencyVersionRange = VersionRange.Parse(dependencyVersionValue);
                         }
                         catch (Exception ex)
                         {
@@ -737,7 +737,7 @@ private static void ReadDownloadDependencies(
                     {
                         try
                         {
-                            VersionRange version = JsonUtility.ParseVersionRange(singleVersionValue);
+                            VersionRange version = VersionRange.Parse(singleVersionValue);
 
                             downloadDependencies.Add(new DownloadDependency(name, version));
                         }
@@ -1502,7 +1502,7 @@ static RuntimeDependencySet ReadRuntimeDependencySet(JsonTextReader jsonReader,
             {
                 dependencies ??= new List<RuntimePackageDependency>();
 
-                var dependency = new RuntimePackageDependency(propertyName, JsonUtility.ParseVersionRange(jsonReader.ReadNextTokenAsString()));
+                var dependency = new RuntimePackageDependency(propertyName, VersionRange.Parse(jsonReader.ReadNextTokenAsString()));
 
                 dependencies.Add(dependency);
             });

src/NuGet.Core/NuGet.ProjectModel/JsonUtility.cs

@@ -14,9 +14,6 @@ namespace NuGet.ProjectModel
 {
     internal static class JsonUtility
     {
-        private static readonly Dictionary<string, NuGetVersion> NuGetVersionCache = new();
-        private static readonly Dictionary<string, VersionRange> VersionRangeCache = new();
-
         internal const string NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING = nameof(NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING);
         internal static bool? UseNewtonsoftJson = null;
         internal static readonly char[] PathSplitChars = new[] { LockFile.DirectorySeparatorChar };
@@ -60,7 +57,7 @@ internal static PackageDependency ReadPackageDependency(string property, JToken
             var versionStr = json.Value<string>();
             return new PackageDependency(
                 property,
-                versionStr == null ? null : JsonUtility.ParseVersionRange(versionStr));
+                versionStr == null ? null : VersionRange.Parse(versionStr));
         }
 
         internal static bool UseNewstonSoftJsonForParsing(IEnvironmentVariableReader environmentVariableReader, bool bypassCache)
@@ -170,36 +167,5 @@ internal static JToken WriteString(string item)
         {
             return item != null ? new JValue(item) : JValue.CreateNull();
         }
-
-        internal static NuGetVersion ParseNugetVersion(string value)
-        {
-            if (!NuGetVersionCache.ContainsKey(value))
-            {
-                var result = NuGetVersion.Parse(value);
-                NuGetVersionCache[value] = result;
-            }
-            return NuGetVersionCache[value];
-        }
-
-        internal static bool TryParseNugetVersion(string value, out NuGetVersion version)
-        {
-            if (!NuGetVersionCache.ContainsKey(value))
-            {
-                _ = NuGetVersion.TryParse(value, out version);
-                NuGetVersionCache[value] = version;
-            }
-            version = NuGetVersionCache[value];
-            return version is not null;
-        }
-
-        internal static VersionRange ParseVersionRange(string value)
-        {
-            if (!VersionRangeCache.ContainsKey(value))
-            {
-                var result = VersionRange.Parse(value);
-                VersionRangeCache[value] = result;
-            }
-            return VersionRangeCache[value];
-        }
     }
 }

src/NuGet.Core/NuGet.ProjectModel/LockFile/LockFileFormat.cs

@@ -93,7 +93,7 @@ public LockFile Read(Stream stream, ILogger log, string path)
 
         internal LockFile Read(Stream stream, ILogger log, string path, IEnvironmentVariableReader environmentVariableReader, bool bypassCache = false)
         {
-            if (!JsonUtility.UseNewstonSoftJsonForParsing(environmentVariableReader, bypassCache))
+            if (JsonUtility.UseNewstonSoftJsonForParsing(environmentVariableReader, bypassCache))
             {
                 return Utf8JsonRead(stream, log, path);
             }
@@ -284,7 +284,7 @@ private static LockFileLibrary ReadLibrary(string property, JToken json)
             library.Name = parts[0];
             if (parts.Length == 2)
             {
-                library.Version = JsonUtility.ParseNugetVersion(parts[1]);
+                library.Version = NuGetVersion.Parse(parts[1]);
             }
 
             library.Type = ReadString(json[TypeProperty]);
@@ -560,7 +560,7 @@ private static LockFileTargetLibrary ReadTargetLibrary(string property, JToken j
             else
             {
                 library.Name = property.Substring(0, slashIndex);
-                library.Version = JsonUtility.ParseNugetVersion(property.Substring(slashIndex + 1));
+                library.Version = NuGetVersion.Parse(property.Substring(slashIndex + 1));
             }
 
             var jObject = json as JObject;

src/NuGet.Core/NuGet.ProjectModel/LockFile/Utf8JsonStreamLockFileLibraryConverter.cs

@@ -35,7 +35,7 @@ public LockFileLibrary Read(ref Utf8JsonStreamReader reader)
             lockFileLibrary.Name = name;
             if (!string.IsNullOrWhiteSpace(version))
             {
-                lockFileLibrary.Version = JsonUtility.ParseNugetVersion(version);
+                lockFileLibrary.Version = NuGetVersion.Parse(version);
             }
 
             reader.Read();

src/NuGet.Core/NuGet.ProjectModel/LockFile/Utf8JsonStreamLockFileTargetLibraryConverter.cs

@@ -43,7 +43,7 @@ public LockFileTargetLibrary Read(ref Utf8JsonStreamReader reader)
             var propertyName = reader.GetString();
             var (targetLibraryName, version) = propertyName.SplitInTwo('/');
             lockFileTargetLibrary.Name = targetLibraryName;
-            lockFileTargetLibrary.Version = version is null ? null : JsonUtility.ParseNugetVersion(version);
+            lockFileTargetLibrary.Version = version is null ? null : NuGetVersion.Parse(version);
 
             reader.Read();
             if (reader.TokenType != JsonTokenType.StartObject)
@@ -149,7 +149,7 @@ private IList<PackageDependency> ReadPackageDependencyList(ref Utf8JsonStreamRea
                 string versionString = reader.ReadNextTokenAsString();
                 packageDependencies.Add(new PackageDependency(
                     propertyName,
-                    versionString == null ? null : JsonUtility.ParseVersionRange(versionString)));
+                    versionString == null ? null : VersionRange.Parse(versionString)));
             }
             return packageDependencies;
         }

src/NuGet.Core/NuGet.ProjectModel/PackageSpecUtility.cs

@@ -27,7 +27,7 @@ public static NuGetVersion SpecifySnapshot(string version, string snapshotValue)
                 }
             }
 
-            return JsonUtility.ParseNugetVersion(version);
+            return NuGetVersion.Parse(version);
         }
 
         /// <summary>
@@ -44,7 +44,7 @@ public static bool IsSnapshotVersion(string version)
             {
                 // Verify the version is valid
                 NuGetVersion parsed = null;
-                return JsonUtility.TryParseNugetVersion(version.Substring(0, version.Length - 2), out parsed);
+                return NuGetVersion.TryParse(version.Substring(0, version.Length - 2), out parsed);
             }
 
             return false;

test/NuGet.Core.Tests/NuGet.ProjectModel.Test/LockFileFormatTests.cs

@@ -2384,7 +2384,7 @@ public void LockFileFormat_WritesCentralTransitiveDependencyGroups()
             Assert.Equal(expected.ToString(), output.ToString());
         }
 
-        public static LockFile Read(string filePath, IEnvironmentVariableReader environmentVariableReader)
+        private LockFile Read(string filePath, IEnvironmentVariableReader environmentVariableReader)
         {
             var reader = new LockFileFormat();
             using (var stream = File.OpenRead(filePath))

test/NuGet.Core.Tests/NuGet.ProjectModel.Test/LockFileParsingEnvironmentVariable.cs

@@ -2,25 +2,12 @@
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System.Collections.Generic;
-using NuGet.Common;
 using Test.Utility;
 
-namespace NuGet.ProjectModel.Test
+namespace NuGet.ProjectModel
 {
     public class LockFileParsingEnvironmentVariable
     {
-        public static readonly IEnvironmentVariableReader UseNjForProcessingEnvironmentVariable = new TestEnvironmentVariableReader(
-            new Dictionary<string, string>()
-            {
-                ["NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING"] = bool.TrueString
-            }, "NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING: true");
-
-        public static readonly IEnvironmentVariableReader UseStjForProcessingEnvironmentVariable = new TestEnvironmentVariableReader(
-            new Dictionary<string, string>()
-            {
-                ["NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING"] = bool.FalseString
-            }, "NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING: false");
-
         public static IEnumerable<object[]> TestEnvironmentVariableReader()
         {
             return GetTestEnvironmentVariableReader();
@@ -43,8 +30,20 @@ public static IEnumerable<object[]> TestEnvironmentVariableReader(object value1,
 
         private static IEnumerable<object[]> GetTestEnvironmentVariableReader(params object[] objects)
         {
-            var UseNjForFileTrue = new List<object> { UseNjForProcessingEnvironmentVariable };
-            var UseNjForFileFalse = new List<object> { UseStjForProcessingEnvironmentVariable };
+            var UseNjForFileTrue = new List<object> {
+                new TestEnvironmentVariableReader(
+                    new Dictionary<string, string>()
+                    {
+                        ["NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING"] = bool.TrueString
+                    }, "NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING: true")
+                };
+            var UseNjForFileFalse = new List<object> {
+                new TestEnvironmentVariableReader(
+                    new Dictionary<string, string>()
+                    {
+                        ["NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING"] = bool.FalseString
+                    }, "NUGET_EXPERIMENTAL_USE_NJ_FOR_FILE_PARSING: false")
+                };
 
             if (objects != null)
             {