-
Notifications
You must be signed in to change notification settings - Fork 3.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support overriding virtual indexer property in proxies #22258
Conversation
@Pilchie RC1 |
May be should also remove line from #21948 (comment) |
@smitpatel Dictionary is not proxyable. We might want to throw if people try to use it as a proxy type, but that would also fail all our current tests. We should discuss in the context of #22009. |
Approved for RC1. |
1a083d7
to
5b398ce
Compare
@smitpatel @AndriySvyryd Added another commit which improves this significantly by handling null values returned from indexer properties in the same way we handle null values in the backing field of a non-nullable property. |
Fixes #21985