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

Use beforefieldinit field for Comparer/EqualityComparer.Default#4340

Merged
jkotas merged 1 commit intodotnet:masterfrom
jkotas:equality-comparer
Apr 15, 2016
Merged

Use beforefieldinit field for Comparer/EqualityComparer.Default#4340
jkotas merged 1 commit intodotnet:masterfrom
jkotas:equality-comparer

Conversation

@jkotas
Copy link
Member

@jkotas jkotas commented Apr 14, 2016

No description provided.

@jkotas
Copy link
Member Author

jkotas commented Apr 14, 2016

Fixed #1157

@jkotas
Copy link
Member Author

jkotas commented Apr 14, 2016

@stephentoub PTLA

@VSadov
Copy link
Member

VSadov commented Apr 14, 2016

LGTM

@stephentoub
Copy link
Member

@jkotas jkotas force-pushed the equality-comparer branch from 1185df1 to 3395109 Compare April 15, 2016 00:51
@jkotas
Copy link
Member Author

jkotas commented Apr 15, 2016

Fixed Comparer.Default as well.

@jkotas
Copy link
Member Author

jkotas commented Apr 15, 2016

@dotnet-bot test Ubuntu x64 Checked Build and Test please

@jkotas jkotas changed the title Use beforefieldinit field for EqualityComparer.Default Use beforefieldinit field for Comparer/EqualityComparer.Default Apr 15, 2016
@jkotas jkotas merged commit d5abe2f into dotnet:master Apr 15, 2016
@dotnet-bot dotnet-bot changed the title Use beforefieldinit field for Comparer/EqualityComparer.Default Use beforefieldinit field for Comparer/EqualityComparer.Default Apr 15, 2016
stephentoub added a commit to dotnet/corefx that referenced this pull request Apr 15, 2016
Add comment about dotnet/coreclr#4340 port to CoreRT in future
@jkotas jkotas deleted the equality-comparer branch April 16, 2016 00:26
picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
Use beforefieldinit field for Comparer/EqualityComparer.Default 

Commit migrated from dotnet/coreclr@d5abe2f
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.

6 participants