Skip to content

Use single dictionary in DefaultMemberAccessStrategy#782

Merged
sebastienros merged 1 commit intosebastienros:mainfrom
lahma:use-single-level-member-access-dictionary
Apr 21, 2025
Merged

Use single dictionary in DefaultMemberAccessStrategy#782
sebastienros merged 1 commit intosebastienros:mainfrom
lahma:use-single-level-member-access-dictionary

Conversation

@lahma
Copy link
Copy Markdown
Collaborator

@lahma lahma commented Apr 21, 2025

It's wasteful do do a doble lookup to get onto type's member level. Now using single key. Added also simple Lock shim for non NET 9 targets.

@sebastienros
Copy link
Copy Markdown
Owner

Check what I did recently, almost identical. Currently refactoring the whole thing for 3.0 so all members of the classes you pass will be accessible by default.

main...sebros/memberstrategy#diff-11accc50705ae6f28b4b24b0ea6ab4de5c19ee01ae37dc7084aaca29f03d0e20R9

@sebastienros sebastienros merged commit d39f793 into sebastienros:main Apr 21, 2025
1 check passed
@lahma lahma deleted the use-single-level-member-access-dictionary branch April 21, 2025 16:43
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.

2 participants