Skip to content

Interop Changes Part 1 - Cleanup Interop source#665

Merged
cheenamalhotra merged 5 commits into
dotnet:masterfrom
cheenamalhotra:nativeshims
Aug 27, 2020
Merged

Interop Changes Part 1 - Cleanup Interop source#665
cheenamalhotra merged 5 commits into
dotnet:masterfrom
cheenamalhotra:nativeshims

Conversation

@cheenamalhotra

@cheenamalhotra cheenamalhotra commented Jul 23, 2020

Copy link
Copy Markdown
Member

Cleanups only.

@cheenamalhotra cheenamalhotra marked this pull request as draft July 23, 2020 19:51
@Wraith2

Wraith2 commented Jul 24, 2020

Copy link
Copy Markdown
Contributor

That is a lot of dead code. Well done getting through it all.

@cheenamalhotra cheenamalhotra marked this pull request as ready for review July 24, 2020 16:41
# Conflicts:
#	src/Microsoft.Data.SqlClient/netcore/src/Common/src/CoreLib/Interop/Unix/Interop.IOErrors.cs
#	src/Microsoft.Data.SqlClient/netcore/src/Common/src/Interop/Unix/System.Net.Security.Native/Interop.GssBuffer.cs
#	src/Microsoft.Data.SqlClient/netcore/src/Common/src/Interop/Unix/System.Security.Cryptography.Native/Interop.EcDsa.ImportExport.cs
#	src/Microsoft.Data.SqlClient/netcore/src/Common/src/Interop/Unix/System.Security.Cryptography.Native/Interop.OpenSsl.cs
#	src/Microsoft.Data.SqlClient/netcore/src/Common/src/Interop/Unix/System.Security.Cryptography.Native/Interop.SslCtx.cs
#	src/Microsoft.Data.SqlClient/netcore/src/Common/src/System/Net/Security/NegotiateStreamPal.Unix.cs
#	src/Microsoft.Data.SqlClient/tests/ManualTests/DataCommon/AssemblyResourceManager.cs
@cheenamalhotra cheenamalhotra added this to the 2.1.0-preview2 milestone Aug 27, 2020
@cheenamalhotra cheenamalhotra merged commit fab0fba into dotnet:master Aug 27, 2020
TrayanZapryanov pushed a commit to TrayanZapryanov/SqlClient that referenced this pull request Aug 31, 2020
TrayanZapryanov added a commit to TrayanZapryanov/SqlClient that referenced this pull request Aug 31, 2020
@cheenamalhotra cheenamalhotra deleted the nativeshims branch April 1, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants