Skip to content
This repository was archived by the owner on Jan 23, 2023. It is now read-only.

Remove unused methods#27370

Merged
jkotas merged 1 commit intodotnet:masterfrom
marek-safar:shared5
Oct 23, 2019
Merged

Remove unused methods#27370
jkotas merged 1 commit intodotnet:masterfrom
marek-safar:shared5

Conversation

@marek-safar
Copy link

No description provided.

/// </summary>
/// <param name="featureName">The directory name for the feature</param>
/// <returns>A path within the user's home directory for persisting data for the feature</returns>
internal static string GetUserFeatureDirectory(string featureName)
Copy link
Member

Choose a reason for hiding this comment

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

Doesn't System.Security.Cryptography.X509Certificates in CoreFX use this?

Copy link
Author

Choose a reason for hiding this comment

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

Could we move the corefx part to corefx as partial class of this type to make it more obvious?

Copy link
Member

@stephentoub stephentoub Oct 23, 2019

Choose a reason for hiding this comment

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

corefx uses everything in this file. It was originally written in corefx and only moved to the shared folder when some of the support in corefx that was using it moved to corelib.

Copy link
Author

Choose a reason for hiding this comment

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

undone the change

@jkotas jkotas merged commit e835c93 into dotnet:master Oct 23, 2019
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corefx that referenced this pull request Oct 23, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/corert that referenced this pull request Oct 23, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Dotnet-GitSync-Bot pushed a commit to Dotnet-GitSync-Bot/mono that referenced this pull request Oct 23, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
stephentoub pushed a commit to stephentoub/corefx that referenced this pull request Oct 23, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
jkotas pushed a commit to dotnet/corert that referenced this pull request Oct 23, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
vargaz pushed a commit to mono/mono that referenced this pull request Oct 24, 2019
* Apply SuppressGCAttribute to some SPCL functions. (dotnet/coreclr#27369)

Apply SuppressGCAttribute to some SPCL functions.

Only insert GC_POLL in first morph call.

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

* Remove unused methods (dotnet/coreclr#27370)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

* Move Matrix3x2/4x4, Plan, and Quaternion to the shared CoreLib partition (#42021)

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>

* Remove Socket.InnerSafeCloseSocket (dotnet/corefx#41888)

* Remove Socket.InnerSafeCloseSocket

* Move IsInvalid checks in SetHandle function

* PR feedback

* Remove unnecessary 'SocketPal.'

* Remove SafeSocketHandle.SetHandle

Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
stephentoub pushed a commit to Dotnet-GitSync-Bot/corefx that referenced this pull request Oct 24, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
stephentoub pushed a commit to dotnet/corefx that referenced this pull request Oct 24, 2019
Signed-off-by: dotnet-bot <dotnet-bot@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants