Fixed multiple typos (#5742)

Umar Hayat · web-flow · commit c0888f503baf · 2024-06-05T18:17:43.000Z
diff --git a/src/Npgsql.GeoJSON/CrsMap.WellKnown.cs b/src/Npgsql.GeoJSON/CrsMap.WellKnown.cs
@@ -5,7 +5,7 @@ public partial class CrsMap
     /// <summary>
     /// These entries came from spatial_res_sys. They are used to elide memory allocations
     /// if they are identical to the entries for the current connection. Otherwise,
-    /// memory allocated for overrided entries only (added, removed, or modified).
+    /// memory allocated for overridden entries only (added, removed, or modified).
     /// </summary>
     internal static readonly CrsMapEntry[] WellKnown =
     {
diff --git a/src/Npgsql.GeoJSON/CrsMap.cs b/src/Npgsql.GeoJSON/CrsMap.cs
@@ -6,13 +6,13 @@ namespace Npgsql.GeoJSON;
 /// </summary>
 public partial class CrsMap
 {
-    readonly CrsMapEntry[]? _overriden;
+    readonly CrsMapEntry[]? _overridden;
 
-    internal CrsMap(CrsMapEntry[]? overriden)
-        => _overriden = overriden;
+    internal CrsMap(CrsMapEntry[]? overridden)
+        => _overridden = overridden;
 
     internal string? GetAuthority(int srid)
-        => GetAuthority(_overriden, srid) ?? GetAuthority(WellKnown, srid);
+        => GetAuthority(_overridden, srid) ?? GetAuthority(WellKnown, srid);
 
     static string? GetAuthority(CrsMapEntry[]? entries, int srid)
     {
diff --git a/src/Npgsql.GeoJSON/Internal/CrsMapBuilder.cs b/src/Npgsql.GeoJSON/Internal/CrsMapBuilder.cs
@@ -5,7 +5,7 @@ namespace Npgsql.GeoJSON.Internal;
 struct CrsMapBuilder
 {
     CrsMapEntry[] _overrides;
-    int _overridenIndex;
+    int _overriddenIndex;
     int _wellKnownIndex;
 
     internal void Add(in CrsMapEntry entry)
@@ -33,21 +33,21 @@ internal void Add(in CrsMapEntry entry)
 
     void AddCore(in CrsMapEntry entry)
     {
-        var index = _overridenIndex + 1;
+        var index = _overriddenIndex + 1;
         if (_overrides == null)
             _overrides = new CrsMapEntry[4];
         else
         if (_overrides.Length == index)
             Array.Resize(ref _overrides, _overrides.Length << 1);
 
-        _overrides[_overridenIndex] = entry;
-        _overridenIndex = index;
+        _overrides[_overriddenIndex] = entry;
+        _overriddenIndex = index;
     }
 
     internal CrsMap Build()
     {
-        if (_overrides != null && _overrides.Length < _overridenIndex)
-            Array.Resize(ref _overrides, _overridenIndex);
+        if (_overrides != null && _overrides.Length < _overriddenIndex)
+            Array.Resize(ref _overrides, _overriddenIndex);
 
         return new CrsMap(_overrides);
     }
diff --git a/src/Npgsql.GeoJSON/Internal/GeoJSONConverter.cs b/src/Npgsql.GeoJSON/Internal/GeoJSONConverter.cs
@@ -323,7 +323,7 @@ static Size GetSize(LineString value)
     {
         var coordinates = value.Coordinates;
         if (NotValid(coordinates, out var hasZ))
-            throw AllOrNoneCoordiantesMustHaveZ(nameof(LineString));
+            throw AllOrNoneCoordinatesMustHaveZ(nameof(LineString));
 
         var length = Size.Create(SizeOfHeaderWithLength + coordinates.Count * SizeOfPoint(hasZ));
         if (GetSrid(value.CRS) != 0)
@@ -344,12 +344,12 @@ static Size GetSize(Polygon value)
         {
             var coordinates = lines[i].Coordinates;
             if (NotValid(coordinates, out var lineHasZ))
-                throw AllOrNoneCoordiantesMustHaveZ(nameof(Polygon));
+                throw AllOrNoneCoordinatesMustHaveZ(nameof(Polygon));
 
             if (hasZ != lineHasZ)
             {
                 if (i == 0) hasZ = lineHasZ;
-                else throw AllOrNoneCoordiantesMustHaveZ(nameof(LineString));
+                else throw AllOrNoneCoordinatesMustHaveZ(nameof(LineString));
             }
 
             length = length.Combine(coordinates.Count * SizeOfPoint(hasZ));
@@ -678,7 +678,7 @@ static int SizeOfPoint(EwkbGeometryType type)
     static Exception UnknownPostGisType()
         => throw new InvalidOperationException("Invalid PostGIS type");
 
-    static Exception AllOrNoneCoordiantesMustHaveZ(string typeName)
+    static Exception AllOrNoneCoordinatesMustHaveZ(string typeName)
         => new ArgumentException($"The Z coordinate must be specified for all or none elements of {typeName}");
 
     static int GetSrid(ICRSObject crs)
diff --git a/src/Npgsql.Json.NET/Internal/JsonNetJsonConverter.cs b/src/Npgsql.Json.NET/Internal/JsonNetJsonConverter.cs
@@ -38,7 +38,7 @@ public override ValueTask WriteAsync(PgWriter writer, T? value, CancellationToke
         => JsonNetJsonConverter.Write(_jsonb, async: true, writer, cancellationToken);
 }
 
-// Split out to avoid unneccesary code duplication.
+// Split out to avoid unnecessary code duplication.
 static class JsonNetJsonConverter
 {
     public const byte JsonbProtocolVersion = 1;
diff --git a/src/Npgsql/MultiplexingDataSource.cs b/src/Npgsql/MultiplexingDataSource.cs
@@ -270,7 +270,7 @@ bool WriteCommand(NpgsqlConnector connector, NpgsqlCommand command, ref Multiple
                     }
 
                     // There's almost certainly more buffered outgoing data for the command, after the flush
-                    // occured. Complete the write, which will flush again (and update statistics).
+                    // occurred. Complete the write, which will flush again (and update statistics).
                     try
                     {
                         Flush(conn, ref clonedStats);
diff --git a/src/Npgsql/NpgsqlTypes/NpgsqlTsQuery.cs b/src/Npgsql/NpgsqlTypes/NpgsqlTsQuery.cs
@@ -721,7 +721,7 @@ public override int GetHashCode()
 }
 
 /// <summary>
-/// Represents an empty tsquery. Shold only be used as top node.
+/// Represents an empty tsquery. Should only be used as top node.
 /// </summary>
 public sealed class NpgsqlTsQueryEmpty : NpgsqlTsQuery
 {
diff --git a/src/Npgsql/Util/ResettableCancellationTokenSource.cs b/src/Npgsql/Util/ResettableCancellationTokenSource.cs
@@ -23,7 +23,7 @@ sealed class ResettableCancellationTokenSource : IDisposable
     CancellationTokenRegistration? _registration;
 
     /// <summary>
-    /// Used, so we wouldn't concurently use the cts for the cancellation, while it's being disposed
+    /// Used, so we wouldn't concurrently use the cts for the cancellation, while it's being disposed
     /// </summary>
     readonly object lockObject = new();
 
diff --git a/test/Npgsql.Tests/PrepareTests.cs b/test/Npgsql.Tests/PrepareTests.cs
@@ -462,7 +462,7 @@ public void Overloaded_sql()
 
         // SQL overloading is a pretty rare/exotic scenario. Handling it properly would involve keying
         // prepared statements not just by SQL but also by the parameter types, which would pointlessly
-        // increase allocations. Instead, the second execution simply reuns unprepared
+        // increase allocations. Instead, the second execution simply reruns unprepared
         AssertNumPreparedStatements(conn, 1);
         conn.UnprepareAll();
     }
@@ -659,7 +659,7 @@ public void Same_sql_different_params()
         using (var conn = OpenConnectionAndUnprepare())
         using (var cmd = new NpgsqlCommand("SELECT @p", conn))
         {
-            throw new NotImplementedException("Problem: currentl setting NpgsqlParameter.Value clears/invalidates...");
+            throw new NotImplementedException("Problem: current setting NpgsqlParameter.Value clears/invalidates...");
             cmd.Parameters.Add(new NpgsqlParameter("p", NpgsqlDbType.Integer));
             cmd.Prepare(true);
 
diff --git a/test/Npgsql.Tests/SchemaTests.cs b/test/Npgsql.Tests/SchemaTests.cs
@@ -47,7 +47,7 @@ public async Task No_parameter()
         Assert.That(collections1, Is.EquivalentTo(collections2));
     }
 
-    [Test, Description("Calling GetSchema(collectionName [, restrictions]) case insensive collectionName can be used")]
+    [Test, Description("Calling GetSchema(collectionName [, restrictions]) case insensitive collectionName can be used")]
     public async Task Case_insensitive_collection_name()
     {
         await using var conn = await OpenConnectionAsync();
diff --git a/test/Npgsql.Tests/TestMetrics.cs b/test/Npgsql.Tests/TestMetrics.cs
@@ -46,7 +46,7 @@ public static TestMetrics Start(TimeSpan allowedTime, bool reportOnStop)
     }
 
     /// <summary>
-    /// Incremnent the Iterations value by one.
+    /// Increment the Iterations value by one.
     /// </summary>
     public void IncrementIterations()
     {
@@ -102,7 +102,7 @@ public double IterationsPer(TimeSpan timeSpan)
 
     /// <summary>
     /// Calculate the number of iterations accumulated per second.
-    /// Equivelent to calling IterationsPer(new TimeSpan(0, 0, 1)).
+    /// Equivalent to calling IterationsPer(new TimeSpan(0, 0, 1)).
     /// </summary>
     /// <returns>The number of iterations accumulated per second.</returns>
     public double IterationsPerSecond()
@@ -122,7 +122,7 @@ public double IterationsPerCPU(TimeSpan timeSpan)
 
     /// <summary>
     /// Calculate the number of iterations accumulated per CPU second.
-    /// Equivelent to calling IterationsPerCPU(new TimeSpan(0, 0, 1)).
+    /// Equivalent to calling IterationsPerCPU(new TimeSpan(0, 0, 1)).
     /// </summary>
     /// <param name="timeSpan"></param>
     /// <returns>The number of iterations accumulated per CPU second.</returns>
diff --git a/test/Npgsql.Tests/TransactionTests.cs b/test/Npgsql.Tests/TransactionTests.cs
@@ -14,7 +14,7 @@ namespace Npgsql.Tests;
 
 public class TransactionTests : MultiplexingTestBase
 {
-    [Test, Description("Basic insert within a commited transaction")]
+    [Test, Description("Basic insert within a committed transaction")]
     public async Task Commit([Values(PrepareOrNot.NotPrepared, PrepareOrNot.Prepared)] PrepareOrNot prepare)
     {
         if (prepare == PrepareOrNot.Prepared && IsMultiplexing)
@@ -43,7 +43,7 @@ public async Task Commit([Values(PrepareOrNot.NotPrepared, PrepareOrNot.Prepared
             Assert.That(() => tx.Connection, Throws.Exception.TypeOf<ObjectDisposedException>());
     }
 
-    [Test, Description("Basic insert within a commited transaction")]
+    [Test, Description("Basic insert within a committed transaction")]
     public async Task CommitAsync([Values(PrepareOrNot.NotPrepared, PrepareOrNot.Prepared)] PrepareOrNot prepare)
     {
         if (prepare == PrepareOrNot.Prepared && IsMultiplexing)
diff --git a/test/Npgsql.Tests/Types/RangeTests.cs b/test/Npgsql.Tests/Types/RangeTests.cs
@@ -97,7 +97,7 @@ public void Equality_infinite()
     {
         var r1 = new NpgsqlRange<int>(0, false, true, 1, false, false);
 
-        //different upper bound (lower bound shoulnd't matter since it is infinite)
+        //different upper bound (lower bound shouldn't matter since it is infinite)
         var r2 = new NpgsqlRange<int>(1, false, true, 2, false, false);
         Assert.IsFalse(r1 == r2);
 

Original file line number	Diff line number	Diff line change
`@@ -5,7 +5,7 @@ public partial class CrsMap`
`5`	`5`	`/// <summary>`
`6`	`6`	`/// These entries came from spatial_res_sys. They are used to elide memory allocations`
`7`	`7`	`/// if they are identical to the entries for the current connection. Otherwise,`
`8`		`- /// memory allocated for overrided entries only (added, removed, or modified).`
	`8`	`+ /// memory allocated for overridden entries only (added, removed, or modified).`
`9`	`9`	`/// </summary>`
`10`	`10`	`internal static readonly CrsMapEntry[] WellKnown =`
`11`	`11`	`{`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public override ValueTask WriteAsync(PgWriter writer, T? value, CancellationToke`
`38`	`38`	`=> JsonNetJsonConverter.Write(_jsonb, async: true, writer, cancellationToken);`
`39`	`39`	`}`
`40`	`40`
`41`		`-// Split out to avoid unneccesary code duplication.`
	`41`	`+// Split out to avoid unnecessary code duplication.`
`42`	`42`	`static class JsonNetJsonConverter`
`43`	`43`	`{`
`44`	`44`	`public const byte JsonbProtocolVersion = 1;`
Original file line number	Diff line number	Diff line change
`@@ -270,7 +270,7 @@ bool WriteCommand(NpgsqlConnector connector, NpgsqlCommand command, ref Multiple`
`270`	`270`	`}`
`271`	`271`
`272`	`272`	`// There's almost certainly more buffered outgoing data for the command, after the flush`
`273`		`- // occured. Complete the write, which will flush again (and update statistics).`
	`273`	`+ // occurred. Complete the write, which will flush again (and update statistics).`
`274`	`274`	`try`
`275`	`275`	`{`
`276`	`276`	`Flush(conn, ref clonedStats);`
Original file line number	Diff line number	Diff line change
`@@ -721,7 +721,7 @@ public override int GetHashCode()`
`721`	`721`	`}`
`722`	`722`
`723`	`723`	`/// <summary>`
`724`		`-/// Represents an empty tsquery. Shold only be used as top node.`
	`724`	`+/// Represents an empty tsquery. Should only be used as top node.`
`725`	`725`	`/// </summary>`
`726`	`726`	`public sealed class NpgsqlTsQueryEmpty : NpgsqlTsQuery`
`727`	`727`	`{`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ public async Task No_parameter()`
`47`	`47`	`Assert.That(collections1, Is.EquivalentTo(collections2));`
`48`	`48`	`}`
`49`	`49`
`50`		`- [Test, Description("Calling GetSchema(collectionName [, restrictions]) case insensive collectionName can be used")]`
	`50`	`+ [Test, Description("Calling GetSchema(collectionName [, restrictions]) case insensitive collectionName can be used")]`
`51`	`51`	`public async Task Case_insensitive_collection_name()`
`52`	`52`	`{`
`53`	`53`	`await using var conn = await OpenConnectionAsync();`