User Story 34145: Tidy up Bulk Copy unmatched column name work

- Addressed review comments.

Author: paulmedynski

src/Microsoft.Data.SqlClient/netcore/src/Microsoft/Data/SqlClient/SqlBulkCopy.cs

@@ -578,8 +578,8 @@ private string AnalyzeTargetAndCreateUpdateBulkCommand(BulkCopySimpleResultSet i
 
                     // Are we missing a mapping between the result column and
                     // this local column (by ordinal or name)?
-                    if (! (localColumn._destinationColumnOrdinal == metadata.ordinal
-                           || UnquotedName(localColumn._destinationColumnName) == metadata.column))
+                    if (localColumn._destinationColumnOrdinal != metadata.ordinal
+                        && UnquotedName(localColumn._destinationColumnName) != metadata.column)
                     {
                         // Yes, so move on to the next local column.
                         continue;
@@ -596,7 +596,7 @@ private string AnalyzeTargetAndCreateUpdateBulkCommand(BulkCopySimpleResultSet i
                     //
                     // We will not process timestamp or identity columns.
                     //
-                    if ((metadata.type == SqlDbType.Timestamp)
+                    if (metadata.type == SqlDbType.Timestamp
                         || (metadata.IsIdentity && !IsCopyOption(SqlBulkCopyOptions.KeepIdentity)))
                     {
                         rejected = true;

src/Microsoft.Data.SqlClient/netfx/src/Microsoft/Data/SqlClient/SqlBulkCopy.cs

@@ -613,8 +613,8 @@ private string AnalyzeTargetAndCreateUpdateBulkCommand(BulkCopySimpleResultSet i
 
                     // Are we missing a mapping between the result column and
                     // this local column (by ordinal or name)?
-                    if (! (localColumn._destinationColumnOrdinal == metadata.ordinal
-                           || UnquotedName(localColumn._destinationColumnName) == metadata.column))
+                    if (localColumn._destinationColumnOrdinal != metadata.ordinal
+                        && UnquotedName(localColumn._destinationColumnName) != metadata.column)
                     {
                         // Yes, so move on to the next local column.
                         continue;
@@ -631,7 +631,7 @@ private string AnalyzeTargetAndCreateUpdateBulkCommand(BulkCopySimpleResultSet i
                     //
                     // We will not process timestamp or identity columns.
                     //
-                    if ((metadata.type == SqlDbType.Timestamp)
+                    if (metadata.type == SqlDbType.Timestamp
                         || (metadata.IsIdentity && !IsCopyOption(SqlBulkCopyOptions.KeepIdentity)))
                     {
                         rejected = true;