Skip to content

Record usage of TupleElementNamesAttribute in VB#42048

Merged
AlekseyTs merged 2 commits intodotnet:features/UsedAssemblyReferencesfrom
AlekseyTs:UsedAssemblies_17
Mar 3, 2020
Merged

Record usage of TupleElementNamesAttribute in VB#42048
AlekseyTs merged 2 commits intodotnet:features/UsedAssemblyReferencesfrom
AlekseyTs:UsedAssemblies_17

Conversation

@AlekseyTs
Copy link
Copy Markdown
Contributor

No description provided.

@AlekseyTs AlekseyTs marked this pull request as ready for review February 29, 2020 18:32
@AlekseyTs AlekseyTs requested review from a team as code owners February 29, 2020 18:32
@AlekseyTs
Copy link
Copy Markdown
Contributor Author

dotnet/roslyn-compiler

Dim constructorSymbol = TryCast(GetWellKnownTypeMember(WellKnownMember.System_Runtime_CompilerServices_TupleElementNamesAttribute__ctorTransformNames), MethodSymbol)
Return constructorSymbol IsNot Nothing AndAlso
Binder.GetUseSiteInfoForWellKnownTypeMember(constructorSymbol, WellKnownMember.System_Runtime_CompilerServices_TupleElementNamesAttribute__ctorTransformNames,
embedVBRuntimeUsed:=False).DiagnosticInfo Is Nothing
Copy link
Copy Markdown
Member

@gafter gafter Mar 2, 2020

Choose a reason for hiding this comment

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

DiagnosticInfo [](start = 94, length = 14)

So, if the attribute ctor is obsolete with a warning this returns false? If that is the intended behavior, why? #Resolved

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

So, if the attribute ctor is obsolete with a warning this returns false? If that is the intended behavior, why?

There is no change around this scenario. Obsolete checks are performed separately and presence of an Obsolete attributes on a symbol does not have any effect on DiagnosticInfo. Also, in VB the DiagnosticInfo property never returns warnings (unlike in C#).


In reply to: 386555701 [](ancestors = 386555701)

Copy link
Copy Markdown
Member

@gafter gafter left a comment

Choose a reason for hiding this comment

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

:shipit:

@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@dotnet/roslyn-compiler Please review, need a second sign-off for a small change.

1 similar comment
@AlekseyTs
Copy link
Copy Markdown
Contributor Author

@dotnet/roslyn-compiler Please review, need a second sign-off for a small change.

@AlekseyTs AlekseyTs merged commit c663d64 into dotnet:features/UsedAssemblyReferences Mar 3, 2020
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.

3 participants