Merge pull request #163 from RoadTrain/uniqueid-performance

Calculate UniqueID via increment

SevenZip/SevenZipBase.cs

@@ -15,7 +15,7 @@ public abstract class SevenZipBase : MarshalByRefObject
     {
         private readonly bool _reportErrors;
         private readonly int _uniqueID;
-        private static readonly List<int> Identifiers = new List<int>();
+        private static int _incrementingUniqueId = int.MinValue;
 
         /// <summary>
         /// True if the instance of the class needs to be recreated in new thread context; otherwise, false.
@@ -103,22 +103,8 @@ internal virtual void ReleaseContext()
 
         private static int GetUniqueID()
         {
-            lock (Identifiers)
-            {
-                int id;
-
-                var rnd = new Random(DateTime.Now.Millisecond);
-
-                do
-                {
-                    id = rnd.Next(int.MaxValue);
-                }
-                while (Identifiers.Contains(id));
-
-                Identifiers.Add(id);
-
-                return id;
-            }
+            var newUniqueId = Interlocked.Increment(ref _incrementingUniqueId);
+            return newUniqueId;
         }
 
         /// <summary>
@@ -132,18 +118,6 @@ protected SevenZipBase(string password = "")
             _uniqueID = GetUniqueID();
         }
 
-        /// <summary>
-        /// Removes the UniqueID from the list.
-        /// </summary>
-        ~SevenZipBase()
-        {
-            // This lock probably isn't necessary but just in case...
-            lock (Identifiers)
-            {
-                Identifiers.Remove(_uniqueID);
-            }
-        }
-
         /// <summary>
         /// Gets or sets the archive password
         /// </summary>