Small provider improvements

Author: alex-kulakov

Orm/Xtensive.Orm.Firebird/Sql.Drivers.Firebird/Connection.cs

@@ -13,8 +13,8 @@ namespace Xtensive.Sql.Drivers.Firebird
 {
   internal class Connection : SqlConnection
   {
-    private FbConnection underlyingConnection;
-    private FbTransaction activeTransaction;
+    protected FbConnection underlyingConnection;
+    protected FbTransaction activeTransaction;
 
     /// <inheritdoc/>
     public override DbConnection UnderlyingConnection => underlyingConnection;
@@ -36,26 +36,26 @@ public override void BeginTransaction(IsolationLevel isolationLevel)
 
       var transactionOptions = CreateTransactionOptions(isolationLevel);
       activeTransaction = underlyingConnection.BeginTransaction(transactionOptions);
-    }
 
-    private static FbTransactionOptions CreateTransactionOptions(IsolationLevel isolationLevel)
-    {
-      var transactionOptions = new FbTransactionOptions {WaitTimeout = TimeSpan.FromSeconds(10)};
-      switch (SqlHelper.ReduceIsolationLevel(isolationLevel)) {
-        case IsolationLevel.ReadCommitted:
-          transactionOptions.TransactionBehavior = FbTransactionBehavior.ReadCommitted
-            | FbTransactionBehavior.NoRecVersion
-            | FbTransactionBehavior.Write
-            | FbTransactionBehavior.NoWait;
-          break;
-        case IsolationLevel.Serializable:
-          transactionOptions.TransactionBehavior = FbTransactionBehavior.Concurrency
-            | FbTransactionBehavior.Write
-            | FbTransactionBehavior.Wait;
-          break;
-      }
+      static FbTransactionOptions CreateTransactionOptions(IsolationLevel isolationLevel)
+      {
+        var transactionOptions = new FbTransactionOptions { WaitTimeout = TimeSpan.FromSeconds(10) };
+        switch (SqlHelper.ReduceIsolationLevel(isolationLevel)) {
+          case IsolationLevel.ReadCommitted:
+            transactionOptions.TransactionBehavior = FbTransactionBehavior.ReadCommitted
+              | FbTransactionBehavior.NoRecVersion
+              | FbTransactionBehavior.Write
+              | FbTransactionBehavior.NoWait;
+            break;
+          case IsolationLevel.Serializable:
+            transactionOptions.TransactionBehavior = FbTransactionBehavior.Concurrency
+              | FbTransactionBehavior.Write
+              | FbTransactionBehavior.Wait;
+            break;
+        }
 
-      return transactionOptions;
+        return transactionOptions;
+      }
     }
 
     /// <inheritdoc/>
@@ -85,4 +85,4 @@ public Connection(SqlDriver driver)
       underlyingConnection = new FbConnection();
     }
   }
-} ;
+}

Orm/Xtensive.Orm.Firebird/Sql.Drivers.Firebird/v4_0/Driver.cs

@@ -12,30 +12,15 @@ namespace Xtensive.Sql.Drivers.Firebird.v4_0
 {
   internal class Driver : v2_5.Driver
   {
-    protected override Sql.TypeMapper CreateTypeMapper()
-    {
-      return new TypeMapper(this);
-    }
+    protected override Sql.TypeMapper CreateTypeMapper() => new TypeMapper(this);
 
-    protected override SqlCompiler CreateCompiler()
-    {
-      return new Compiler(this);
-    }
+    protected override SqlCompiler CreateCompiler() => new Compiler(this);
 
-    protected override SqlTranslator CreateTranslator()
-    {
-      return new Translator(this);
-    }
+    protected override SqlTranslator CreateTranslator() => new Translator(this);
 
-    protected override Model.Extractor CreateExtractor()
-    {
-      return new Extractor(this);
-    }
+    protected override Model.Extractor CreateExtractor() => new Extractor(this);
 
-    protected override Info.ServerInfoProvider CreateServerInfoProvider()
-    {
-      return new ServerInfoProvider(this);
-    }
+    protected override Info.ServerInfoProvider CreateServerInfoProvider() => new ServerInfoProvider(this);
 
     // Constructors