misc: fix typos (#1162)

Minor IDE auto white space strip

Author: jbampton

src/Lucene.Net.Facet/Taxonomy/CachedOrdinalsReader.cs

@@ -39,25 +39,25 @@ namespace Lucene.Net.Facet.Taxonomy
 
     /// <summary>
     /// A per-segment cache of documents' facet ordinals. Every
-    /// <see cref="CachedOrds"/> holds the ordinals in a raw <see cref="T:int[]"/>, 
+    /// <see cref="CachedOrds"/> holds the ordinals in a raw <see cref="T:int[]"/>,
     /// and therefore consumes as much RAM as the total
     /// number of ordinals found in the segment, but saves the
     /// CPU cost of decoding ordinals during facet counting.
-    /// 
+    ///
     /// <para>
     /// <b>NOTE:</b> every <see cref="CachedOrds"/> is limited to 2.1B
     /// total ordinals. If that is a limitation for you then
     /// consider limiting the segment size to fewer documents, or
     /// use an alternative cache which pages through the category
     /// ordinals.
-    /// 
+    ///
     /// </para>
     /// <para>
     /// <b>NOTE:</b> when using this cache, it is advised to use
     /// a <see cref="DocValuesFormat"/> that does not cache the data in
     /// memory, at least for the category lists fields, or
     /// otherwise you'll be doing double-caching.
-    /// 
+    ///
     /// </para>
     /// <para>
     /// <b>NOTE:</b> create one instance of this and re-use it
@@ -225,7 +225,7 @@ public virtual long RamBytesUsed()
                 foreach (var pair in ordsCache)
                     cachedOrdsList.Add(pair.Value);
 #else
-                // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a CondtionalWeakTable enumerator,
+                // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a ConditionalWeakTable enumerator,
                 // we use a weak event to retrieve the CachedOrds instances. We look each of these up here to avoid the need
                 // to attach events to the CachedOrds instances themselves (thus using the existing IndexReader.Dispose()
                 // method to detach the events rather than using a finalizer in CachedOrds to ensure they are cleaned up).
@@ -250,4 +250,4 @@ public virtual long RamBytesUsed()
             return total;
         }
     }
-}
+}

src/Lucene.Net.Tests.Analysis.Common/Analysis/Util/TestCharArraySet.cs

@@ -273,7 +273,7 @@ public virtual void TestUnmodifiableSet()
             //    CharArraySet.UnmodifiableSet(null);
             //    fail("can not make null unmodifiable");
             //}
-            //catch (ArgumentNullException) // NOTE: In .NET we throw an ArgumentExcpetion, not a NullReferenceExeption
+            //catch (ArgumentNullException) // NOTE: In .NET we throw an ArgumentException, not a NullReferenceException
             //{
             //    // expected
             //}
@@ -341,7 +341,7 @@ public virtual void TestSingleHighSurrogate()
         }
 
         /// @deprecated (3.1) remove this test when lucene 3.0 "broken unicode 4" support is
-        ///             no longer needed. 
+        ///             no longer needed.
         [Test]
         [Obsolete("(3.1) remove this test when lucene 3.0 'broken unicode 4' support is no longer needed.")]
         public virtual void TestSupplementaryCharsBWCompat()
@@ -375,7 +375,7 @@ public virtual void TestSupplementaryCharsBWCompat()
         }
 
         /// @deprecated (3.1) remove this test when lucene 3.0 "broken unicode 4" support is
-        ///             no longer needed. 
+        ///             no longer needed.
         [Test]
         [Obsolete("(3.1) remove this test when lucene 3.0 'broken unicode 4' support is no longer needed.")]
         public virtual void TestSingleHighSurrogateBWComapt()
@@ -600,15 +600,15 @@ public virtual void TestContainsWithNull()
                 set.Contains((char[])null, 0, 10);
                 fail("null value must raise NPE");
             }
-            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentExcpetion, not a NullReferenceExeption
+            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentException, not a NullReferenceException
             {
             }
             try
             {
                 set.Contains((ICharSequence)null);
                 fail("null value must raise NPE");
             }
-            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentExcpetion, not a NullReferenceExeption
+            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentException, not a NullReferenceException
             {
             }
             // LUCENENET Specific test for string (since it does not implement ICharSequence)
@@ -617,15 +617,15 @@ public virtual void TestContainsWithNull()
                 set.Contains((string)null);
                 fail("null value must raise NPE");
             }
-            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentExcpetion, not a NullReferenceExeption
+            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentException, not a NullReferenceException
             {
             }
             try
             {
                 set.Contains((object)null);
                 fail("null value must raise NPE");
             }
-            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentExcpetion, not a NullReferenceExeption
+            catch (ArgumentException) // NOTE: In .NET we throw an ArgumentException, not a NullReferenceException
             {
             }
         }
@@ -1743,4 +1743,4 @@ public virtual void TestCopyToCharSequence()
 
         #endregion
     }
-}
+}

src/Lucene.Net.Tests/Index/TestReaderClosed.cs

@@ -101,7 +101,7 @@ public virtual void TestReaderChaining()
             catch (Exception ace) when (ace.IsAlreadyClosedException())
             {
                 //Assert.AreEqual("this IndexReader cannot be used anymore as one of its child readers was closed", ace.Message);
-                // LUCENENET specific - ObjectDisposedExeption appends the type of object to the end of the message,
+                // LUCENENET specific - ObjectDisposedException appends the type of object to the end of the message,
                 // so we need to check the start of the message only.
                 assertTrue(ace.Message.StartsWith("this IndexReader cannot be used anymore as one of its child readers was disposed.", StringComparison.Ordinal));
             }

src/Lucene.Net/Index/IndexReader.cs

@@ -411,7 +411,7 @@ public static DirectoryReader Open(Directory directory)
         /// <param name="directory"> the index directory </param>
         /// <param name="termInfosIndexDivisor"> Subsamples which indexed
         ///  terms are loaded into RAM. this has the same effect as
-        ///  <see cref="LiveIndexWriterConfig.TermIndexInterval"/> 
+        ///  <see cref="LiveIndexWriterConfig.TermIndexInterval"/>
         ///  (which can be set on <see cref="IndexWriterConfig"/>) except that setting
         ///  must be done at indexing time while this setting can be
         ///  set per reader.  When set to <c>N</c>, then one in every
@@ -452,7 +452,7 @@ public static DirectoryReader Open(IndexWriter writer, bool applyAllDeletes)
 
         /// <summary>
         /// Expert: returns an <see cref="IndexReader"/> reading the index in the given
-        /// <see cref="IndexCommit"/>. 
+        /// <see cref="IndexCommit"/>.
         /// </summary>
         /// <param name="commit"> the commit point to open </param>
         /// <exception cref="IOException"> if there is a low-level IO error </exception>
@@ -468,7 +468,7 @@ public static DirectoryReader Open(IndexCommit commit)
         /// <param name="commit"> the commit point to open </param>
         /// <param name="termInfosIndexDivisor"> Subsamples which indexed
         ///  terms are loaded into RAM. this has the same effect as
-        ///  <see cref="LiveIndexWriterConfig.TermIndexInterval"/> 
+        ///  <see cref="LiveIndexWriterConfig.TermIndexInterval"/>
         ///  (which can be set in <see cref="IndexWriterConfig"/>) except that setting
         ///  must be done at indexing time while this setting can be
         ///  set per reader.  When set to <c>N</c>, then one in every
@@ -540,7 +540,7 @@ public Terms GetTermVector(int docID, string field)
         /// <b>NOTE:</b> for performance reasons, this method does not check if the
         /// requested document is deleted, and therefore asking for a deleted document
         /// may yield unspecified results. Usually this is not required, however you
-        /// can test if the doc is deleted by checking the 
+        /// can test if the doc is deleted by checking the
         /// <see cref="Util.IBits"/> returned from <see cref="MultiFields.GetLiveDocs"/>.
         /// <para/>
         /// <b>NOTE:</b> only the content of a field is returned,
@@ -591,13 +591,13 @@ public void Dispose()
 
         /// <summary>
         /// Closes files associated with this index.
-        /// This method implements the disposable pattern. 
+        /// This method implements the disposable pattern.
         /// It may be overridden to dispose any managed or unmanaged resources,
         /// but be sure to call <c>base.Dispose(disposing)</c> to close files associated with the
         /// underlying <see cref="IndexReader"/>.
         /// </summary>
-        /// <param name="disposing"><c>true</c> indicates to dispose all managed 
-        /// and unmanaged resources, <c>false</c> indicates dispose unmanaged 
+        /// <param name="disposing"><c>true</c> indicates to dispose all managed
+        /// and unmanaged resources, <c>false</c> indicates dispose unmanaged
         /// resources only</param>
         protected virtual void Dispose(bool disposing)
         {
@@ -619,7 +619,7 @@ protected virtual void Dispose(bool disposing)
             }
 
 #if !FEATURE_CONDITIONALWEAKTABLE_ENUMERATOR
-            // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a CondtionalWeakTable enumerator,
+            // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a ConditionalWeakTable enumerator,
             // we use a weak event to retrieve the ConditionalWeakTable items
             foreach (var getParentReadersEvent in getParentReadersEvents)
                 getParentReadersEvent.Unsubscribe(OnGetParentReaders);
@@ -636,7 +636,7 @@ protected virtual void Dispose(bool disposing)
         protected internal abstract void DoClose();
 
 #if !FEATURE_CONDITIONALWEAKTABLE_ENUMERATOR
-        // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a CondtionalWeakTable enumerator,
+        // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a ConditionalWeakTable enumerator,
         // we use a weak event to retrieve the ConditionalWeakTable items
         [ExcludeFromRamUsageEstimation]
         private readonly ISet<WeakEvents.GetParentReadersEvent> getParentReadersEvents = new JCG.HashSet<WeakEvents.GetParentReadersEvent>();
@@ -699,10 +699,10 @@ private void OnGetCacheKeys(WeakEvents.GetCacheKeysEventArgs e)
         public IList<AtomicReaderContext> Leaves => Context.Leaves;
 
         /// <summary>
-        /// Expert: Returns a key for this <see cref="IndexReader"/>, so 
+        /// Expert: Returns a key for this <see cref="IndexReader"/>, so
         /// <see cref="Search.FieldCache"/>/<see cref="Search.CachingWrapperFilter"/> can find
         /// it again.
-        /// This key must not have Equals()/GetHashCode() methods, 
+        /// This key must not have Equals()/GetHashCode() methods,
         /// so &quot;equals&quot; means &quot;identical&quot;.
         /// </summary>
         public virtual object CoreCacheKey =>
@@ -713,7 +713,7 @@ private void OnGetCacheKeys(WeakEvents.GetCacheKeysEventArgs e)
         /// <summary>
         /// Expert: Returns a key for this <see cref="IndexReader"/> that also includes deletions,
         /// so <see cref="Search.IFieldCache"/>/<see cref="Search.CachingWrapperFilter"/> can find it again.
-        /// This key must not have Equals()/GetHashCode() methods, 
+        /// This key must not have Equals()/GetHashCode() methods,
         /// so &quot;equals&quot; means &quot;identical&quot;.
         /// </summary>
         public virtual object CombinedCoreAndDeletesKey =>

src/Lucene.Net/Search/CachingWrapperFilter.cs

@@ -64,7 +64,7 @@ public CachingWrapperFilter(Filter filter)
 
         /// <summary>
         /// Provide the <see cref="DocIdSet"/> to be cached, using the <see cref="DocIdSet"/> provided
-        /// by the wrapped Filter. 
+        /// by the wrapped Filter.
         /// <para/>This implementation returns the given <see cref="DocIdSet"/>,
         /// if <see cref="DocIdSet.IsCacheable"/> returns <c>true</c>, else it calls
         /// <see cref="CacheImpl(DocIdSetIterator, AtomicReader)"/>
@@ -134,7 +134,7 @@ public override DocIdSet GetDocIdSet(AtomicReaderContext context, IBits acceptDo
                 try
                 {
                     cache.AddOrUpdate(key, docIdSet);
-                    // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a CondtionalWeakTable enumerator,
+                    // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a ConditionalWeakTable enumerator,
                     // we use a weak event to retrieve the DocIdSet instances
                     reader.SubscribeToGetCacheKeysEvent(eventAggregator.GetEvent<WeakEvents.GetCacheKeysEvent>());
                 }
@@ -196,7 +196,7 @@ public virtual long GetSizeInBytes()
                 foreach (var pair in cache)
                     docIdSets.Add(pair.Value);
 #else
-                // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a CondtionalWeakTable enumerator,
+                // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a ConditionalWeakTable enumerator,
                 // we use a weak event to retrieve the DocIdSet instances. We look each of these up here to avoid the need
                 // to attach events to the DocIdSet instances themselves (thus using the existing IndexReader.Dispose()
                 // method to detach the events rather than using a finalizer in DocIdSet to ensure they are cleaned up).
@@ -221,4 +221,4 @@ public virtual long GetSizeInBytes()
             return total;
         }
     }
-}
+}

src/Lucene.Net/Search/FieldCacheImpl.cs

@@ -186,7 +186,7 @@ private void AddCacheEntries<TKey, TValue>(IList<FieldCache.CacheEntry> result,
                     }
                 }
 #else
-                // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a CondtionalWeakTable enumerator,
+                // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a ConditionalWeakTable enumerator,
                 // we use a weak event to retrieve the readerKey instances and then lookup the values in the table one by one.
                 var e = new WeakEvents.GetCacheKeysEventArgs();
                 eventAggregator.GetEvent<WeakEvents.GetCacheKeysEvent>().Publish(e);
@@ -268,7 +268,7 @@ private void InitReader(AtomicReader reader)
                 }
             }
 #if !FEATURE_CONDITIONALWEAKTABLE_ENUMERATOR
-            // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a CondtionalWeakTable enumerator,
+            // LUCENENET specific - since .NET Standard 2.0 and .NET Framework don't have a ConditionalWeakTable enumerator,
             // we use a weak event to retrieve the readerKey instances
             reader.SubscribeToGetCacheKeysEvent(eventAggregator.GetEvent<WeakEvents.GetCacheKeysEvent>());
 #endif