Skip to content

Commit

Permalink
Revert "Expose Marshall.LoadLibrary API" (dotnet/corefx#34322)
Browse files Browse the repository at this point in the history
In API review: https://github.com/dotnet/corefx/issues/32015,

The LoadLibrary APIs were originally Approved to live in
System.Runtime.InteropServices.Marshal class.
https://github.com/dotnet/corefx/issues/32015#issuecomment-428775858

However, recently the decision was changed such that the APIs are in a new NativeLibrary class.
https://github.com/dotnet/corefx/issues/32015#issuecomment-448324606

Therefore, undoing the changes to Marshal API.
I'll submit another PR for System.Runtime.InteropServices.NativeLibrary
once the corresponding change in CoreCLR is checked in.

This reverts commit dotnet/corefx@859351c.

Commit migrated from dotnet/corefx@6d59e6a
  • Loading branch information
swaroop-sridhar authored and jkotas committed Jan 3, 2019
1 parent 0964672 commit 9082aff
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 21 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@
</PropertyGroup>
<ItemGroup>
<Compile Include="System.Runtime.InteropServices.cs" />
<Compile Condition="'$(TargetsNetCoreApp)' == 'true'" Include="System.Runtime.InteropServices.netcoreapp.cs" />
</ItemGroup>
<ItemGroup>
<ProjectReference Include="..\..\System.Reflection\ref\System.Reflection.csproj" />
Expand Down

This file was deleted.

0 comments on commit 9082aff

Please sign in to comment.