6. Optional refactoring that reduces #if directives in merged TdsParserStateObject also benefits merged SqlInternalTransaction.

panoskj · panoskj · commit 98f818a3afb8 · 2023-10-25T22:36:26.000+03:00
diff --git a/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlInternalTransaction.cs b/src/Microsoft.Data.SqlClient/src/Microsoft/Data/SqlClient/SqlInternalTransaction.cs
@@ -203,9 +203,7 @@ internal void CloseFromConnection()
             }
             finally
             {
-#if NETFRAMEWORK
                 TdsParser.ReliabilitySection.Assert("unreliable call to CloseFromConnection");  // you need to setup for a thread abort somewhere before you call this method
-#endif
                 if (processFinallyBlock)
                 {
                     // Always ensure we're zombied; 2005 will send an EnvChange that

Original file line number	Diff line number	Diff line change
`@@ -203,9 +203,7 @@ internal void CloseFromConnection()`
`203`	`203`	`}`
`204`	`204`	`finally`
`205`	`205`	`{`
`206`		`-#if NETFRAMEWORK`
`207`	`206`	`TdsParser.ReliabilitySection.Assert("unreliable call to CloseFromConnection"); // you need to setup for a thread abort somewhere before you call this method`
`208`		`-#endif`
`209`	`207`	`if (processFinallyBlock)`
`210`	`208`	`{`
`211`	`209`	`// Always ensure we're zombied; 2005 will send an EnvChange that`