Skip to content

Extension indexers: flesh out the spec#9869

Merged
jcouv merged 3 commits intodotnet:mainfrom
jcouv:extension-indexers
Jan 12, 2026
Merged

Extension indexers: flesh out the spec#9869
jcouv merged 3 commits intodotnet:mainfrom
jcouv:extension-indexers

Conversation

@jcouv
Copy link
Member

@jcouv jcouv commented Dec 11, 2025

No description provided.

### `IndexerName` attribute

`IndexerNameAttribute` may be applied to an extension indexer. The attribute is
carried over to the extension member in metadata and its value determines
Copy link
Contributor

@AlekseyTs AlekseyTs Dec 11, 2025

Choose a reason for hiding this comment

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

This feels somewhat misleading. It looks like the attribute isn't actually emitted, but "carried over" can be interpreted as it is. #Closed

@AlekseyTs
Copy link
Contributor

AlekseyTs commented Dec 11, 2025

Done with review pass (commit 1) #Closed

@jcouv
Copy link
Member Author

jcouv commented Dec 30, 2025

@AlekseyTs for another look. Thanks

1 similar comment
@jcouv
Copy link
Member Author

jcouv commented Jan 12, 2026

@AlekseyTs for another look. Thanks

Copy link
Contributor

@AlekseyTs AlekseyTs 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 3)

@jcouv jcouv merged commit 7487550 into dotnet:main Jan 12, 2026
1 check passed
@jcouv jcouv deleted the extension-indexers branch January 12, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants