SWEEP: Reviewed all applications of .IncrementAndGet(), .GetAndIncrem…

…ent(), .DecrementAndGet(), and .GetAndDecrement() from atomic numeric classes. Closes #917.

src/Lucene.Net.Replicator/LocalReplicator.cs

@@ -3,6 +3,7 @@
 using System;
 using System.Collections.Generic;
 using System.Diagnostics;
+using System.Globalization;
 using System.IO;
 using System.Linq;
 
@@ -189,7 +190,7 @@ public virtual SessionToken CheckForUpdate(string currentVersion)
                 // currentVersion is either null or older than latest published revision
                 currentRevision.IncRef();
 
-                string sessionID = sessionToken.IncrementAndGet().ToString();
+                string sessionID = sessionToken.IncrementAndGet().ToString(CultureInfo.InvariantCulture);
                 SessionToken token = new SessionToken(sessionID, currentRevision.Revision);
                 sessions[sessionID] = new ReplicationSession(token, currentRevision);
                 return token;
@@ -329,4 +330,4 @@ public virtual void Release(string sessionId)
             }
         }
     }
-}
+}

src/Lucene.Net.TestFramework/Codecs/RAMOnly/RAMOnlyPostingsFormat.cs

@@ -572,7 +572,7 @@ public override long GetCost()
         // Holds all indexes created, keyed by the ID assigned in fieldsConsumer
         private readonly IDictionary<int, RAMPostings> state = new Dictionary<int, RAMPostings>();
 
-        private readonly AtomicInt64 nextID = new AtomicInt64();
+        private readonly AtomicInt32 nextID = new AtomicInt32();
 
         private readonly string RAM_ONLY_NAME = "RAMOnly";
         private const int VERSION_START = 0;
@@ -582,7 +582,7 @@ public override long GetCost()
 
         public override FieldsConsumer FieldsConsumer(SegmentWriteState writeState)
         {
-            int id = (int)nextID.GetAndIncrement();
+            int id = nextID.GetAndIncrement();
 
             // TODO -- ok to do this up front instead of
             // on close....?  should be ok?
@@ -659,4 +659,4 @@ public override FieldsProducer FieldsProducer(SegmentReadState readState)
             }
         }
     }
-}
+}