PR feedback: don't rethrow exception if null; range checks

Author: paulirwin

src/Lucene.Net/Store/MMapDirectory.cs

@@ -633,7 +633,14 @@ private static void DisposeResourcesWhileHandlingException(Exception? priorExcep
                     .Concat([mmf, fs]);
 
                 // DisposeWhileHandlingException tolerates null disposables
-                IOUtils.DisposeWhileHandlingException(priorException, disposables);
+                if (priorException is null)
+                {
+                    IOUtils.DisposeWhileHandlingException(disposables);
+                }
+                else
+                {
+                    IOUtils.DisposeWhileHandlingException(priorException, disposables);
+                }
             }
         }
 

src/Lucene.Net/Support/Store/UnsafeChunkIndexInput.cs

@@ -187,7 +187,7 @@ protected void EnsureOpen()
 
         public override void Seek(long pos)
         {
-            if ((uint)pos > (uint)length)
+            if ((uint)pos >= (uint)length)
             {
                 throw new ArgumentOutOfRangeException(nameof(pos), $"Seek position is out of bounds: {pos}");
             }
@@ -436,7 +436,7 @@ public override void SkipBytes(long numBytes)
                 throw new ArgumentOutOfRangeException(nameof(numBytes), "numBytes must not be negative");
             }
             long newPos = position + numBytes;
-            if ((ulong)newPos > (ulong)length)
+            if ((ulong)newPos >= (ulong)length)
             {
                 throw EOFException.Create("skip past EOF: " + this);
             }