Skip to content

Remove dead code in OdbcCommandBuilder.UnquoteIdentifier#63363

Merged
stephentoub merged 1 commit intodotnet:mainfrom
stephentoub:odbcdead
Jan 5, 2022
Merged

Remove dead code in OdbcCommandBuilder.UnquoteIdentifier#63363
stephentoub merged 1 commit intodotnet:mainfrom
stephentoub:odbcdead

Conversation

@stephentoub
Copy link
Member

Exact same pattern was previously changed in OleDbCommandBuilder.

Exact same pattern was previously changed in OleDbCommandBuilder.
@ghost ghost added the area-System.Data label Jan 4, 2022
@ghost ghost assigned stephentoub Jan 4, 2022
@ghost
Copy link

ghost commented Jan 4, 2022

Tagging subscribers to this area: @roji, @ajcvickers
See info in area-owners.md if you want to be subscribed.

Issue Details

Exact same pattern was previously changed in OleDbCommandBuilder.

Author: stephentoub
Assignees: -
Labels:

area-System.Data

Milestone: -

Copy link
Member

@roji roji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@stephentoub I'm guessing the original condition was supposed to be && instead of ||, so as to not unquote if quotePrefix is " "... Doesn't really matter at this point.

@stephentoub stephentoub merged commit a8e6f69 into dotnet:main Jan 5, 2022
@stephentoub stephentoub deleted the odbcdead branch January 5, 2022 11:38
@ghost ghost locked as resolved and limited conversation to collaborators Feb 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants