Skip to content

Remove sorting requirement for interface lists - #60541

Merged
jkotas merged 2 commits into
dotnet:mainfrom
MichalStrehovsky:fix60454
Nov 8, 2021
Merged

Remove sorting requirement for interface lists#60541
jkotas merged 2 commits into
dotnet:mainfrom
MichalStrehovsky:fix60454

Conversation

@MichalStrehovsky

Copy link
Copy Markdown
Member

Fixes#60454.

@ghost

Copy link
Copy Markdown

Tagging subscribers to this area: @buyaa-n
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #60454.

Author:MichalStrehovsky
Assignees:-
Labels:

area-System.Reflection.Metadata

Milestone:-

@MichalStrehovsky

Copy link
Copy Markdown
MemberAuthor

@davidwrighton could you have a look at the ECMA spec change proposal?

@runfoapprunfoappBot mentioned this pull request Oct 19, 2021

@davidwrightondavidwrighton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I've checked a bunch of sources to try and find the rationale for this rule, and have come up empty. I think we're good to just change the spec.

tmat
tmat approved these changes Nov 8, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

ValidateInterfaceImplTable enforces bogus ordering rules

5 participants

@MichalStrehovsky@tmat@davidwrighton@jkotas@buyaa-n