BREAKING: Lucene.Net.Support.CRC32, Lucene.Net.Support.IChecksum, Luc…

…ene.Net.Store.BufferedChecksum: Marked internal now to prevent a future breaking change, since the plan is to use HashAlgorithm as a replacement for IChecksum (see LUCENENET-637)

src/Lucene.Net/Store/BufferedChecksum.cs

@@ -24,7 +24,12 @@ namespace Lucene.Net.Store
     /// Wraps another <see cref="IChecksum"/> with an internal buffer
     /// to speed up checksum calculations.
     /// </summary>
-    public class BufferedChecksum : IChecksum
+    // LUCENENET TODO: This class was public in Lucene. Marking internal, since
+    // a better approach would be to map this to the HashAlgorithm abstract class in .NET
+    // instead of using IChecksum from Java. See LUCENENET-637.
+    // After this conversion is done, this can be made public again. However, it is
+    // now internal so the conversion doesn't introduce a breaking public API change.
+    internal class BufferedChecksum : IChecksum
     {
         private readonly IChecksum @in;
         private readonly byte[] buffer;

src/Lucene.Net/Support/CRC32.cs

@@ -23,7 +23,7 @@
 
 namespace Lucene.Net.Support
 {
-    public class CRC32 : IChecksum
+    internal class CRC32 : IChecksum
     {
         private static readonly UInt32[] crcTable = InitializeCRCTable();
 

src/Lucene.Net/Support/IChecksum.cs

@@ -20,7 +20,7 @@ namespace Lucene.Net.Support
     /// <summary>
     /// Contains conversion support elements such as classes, interfaces and static methods.
     /// </summary>
-    public interface IChecksum
+    internal interface IChecksum
     {
         void Reset();