Misc cleanup.

Author: sharpjs

PSql.Engine/SqlConnection.cs

@@ -169,7 +169,7 @@ protected SqlCommand SetUpCommand(
     /// </summary>
     protected void AutoOpen(CancellationToken cancellation)
     {
-        if (Connection.State == ConnectionState.Closed)
+        if (Connection.State is ConnectionState.Closed)
             Connection.OpenAsync(cancellation).GetAwaiter().GetResult();
     }
 

PSql/Data/SqlConnection.cs

@@ -26,7 +26,9 @@ internal SqlConnection(string connectionString, NetworkCredential? credential, I
     /// </summary>
     internal E.SqlConnection InnerConnection => _connection;
 
-    /// <inheritdoc cref="E.SqlConnection.Dispose"/>
+    /// <summary>
+    ///   Closes the connection and frees resources owned by it.
+    /// </summary>
     public void Dispose()
     {
         _connection.Dispose();

PSql/Data/SqlContext.cs

@@ -16,7 +16,7 @@ namespace PSql;
 public class SqlContext : ICloneable
 {
     // Connection string defaults are here:
-    // https://github.com/dotnet/SqlClient/blob/v2.1.1/src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/Common/DbConnectionStringCommon.cs#L690-L731
+    // https://github.com/dotnet/SqlClient/blob/v6.1.3/src/Microsoft.Data.SqlClient/src/Microsoft/Data/Common/ConnectionString/DbConnectionStringDefaults.cs
 
     protected const string
         LocalServerName    = ".",

PSql/Utilities/CmdletSqlMessageLogger.cs

@@ -1,9 +1,10 @@
 // Copyright Subatomix Research Inc.
 // SPDX-License-Identifier: MIT
 
+using PSql.Commands;
+
 namespace PSql;
 
-using PSql.Commands;
 using static E.SqlMessageConstants;
 
 internal class CmdletSqlMessageLogger : E.ISqlMessageLogger

PSql/Utilities/PSCredentialExtensions.cs

@@ -1,8 +1,6 @@
 // Copyright Subatomix Research Inc.
 // SPDX-License-Identifier: MIT
 
-using System.Diagnostics.CodeAnalysis;
-
 namespace PSql;
 
 internal static class PSCredentialExtensions

PSql/Utilities/SqlConnectionStringBuilder.cs

@@ -12,7 +12,7 @@ namespace PSql;
 ///   A simple, append-only connection string builder that supportes multiple
 ///   SqlClient versions.
 /// </summary>
-public readonly ref struct SqlConnectionStringBuilder
+internal readonly ref struct SqlConnectionStringBuilder
 {
     private readonly StringBuilder _builder;