Skip to content

Pack error codes. Disallow UnmanagedCallersOnly for abstract static methods.#54118

Merged
jaredpar merged 2 commits intodotnet:features/StaticAbstractMembersInInterfacesfrom
AlekseyTs:StaticAbstractMembersInInterfaces_31
Jun 16, 2021
Merged

Pack error codes. Disallow UnmanagedCallersOnly for abstract static methods.#54118
jaredpar merged 2 commits intodotnet:features/StaticAbstractMembersInInterfacesfrom
AlekseyTs:StaticAbstractMembersInInterfaces_31

Conversation

@AlekseyTs
Copy link
Contributor

No description provided.

<value>User-defined conversion in an interface must convert to or from a type parameter on the enclosing type constrained to the enclosing type</value>
</data>
<data name="ERR_InterfaceImplementedByUnmanagedCallersOnlyMethod" xml:space="preserve">
<value>'UnmanagedCallersOnly' method '{0}' cannot implement interface member '{1}' in type '{2}'</value>
Copy link
Member

@RikkiGibson RikkiGibson Jun 15, 2021

Choose a reason for hiding this comment

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

It looks like {2} is included in this diagnostic because it may not be the same as the containing type of {0}. Is that correct? #Resolved

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It looks like {2} is included in this diagnostic because it may not be the same as the containing type of {0}. Is that correct?

Correct.

Copy link
Member

@333fred 333fred left a comment

Choose a reason for hiding this comment

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

LGTM (commit 2)

@AlekseyTs
Copy link
Contributor Author

@dotnet/roslyn-infrastructure It looks like all CI jobs succeeded for this PR, but the status isn't updated on the PR page, which is blocking merging.

@AlekseyTs
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 3 pipeline(s).

@AlekseyTs AlekseyTs enabled auto-merge June 16, 2021 14:13
@jaredpar jaredpar disabled auto-merge June 16, 2021 15:21
@jaredpar jaredpar merged commit fc7c990 into dotnet:features/StaticAbstractMembersInInterfaces Jun 16, 2021
@jaredpar
Copy link
Member

AzDO pipelines all succeeded here but we were hitting the bug where it's not updating GH status. Forced merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants