Skip to content

Avoid some unneeded dictionary operations - #997

Merged
KevinJump merged 1 commit into
KevinJump:v17/mainfrom
Henr1k80:v17/main
Aug 5, 2026
Merged

Avoid some unneeded dictionary operations#997
KevinJump merged 1 commit into
KevinJump:v17/mainfrom
Henr1k80:v17/main

Conversation

@Henr1k80

@Henr1k80 Henr1k80 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Every lookup & write requires the creation of a hashkey and lookup in the buckets, so reduce that as much as possible

@KevinJump
KevinJump merged commit 1d06d7c into KevinJump:v17/main Aug 5, 2026
4 checks passed
KevinJump added a commit that referenced this pull request Aug 5, 2026
* Avoid some unneeded dictionary operations (#997)

(cherry picked from commit 1d06d7c)

* Use new Lock & make internal + private classes sealed to avoid virtual dispatch (#998)

(cherry picked from commit 3109ab6)

* Changelog for the v17 perf ports (#997, #998)

Records the SyncChangeInfo sealing, which is a (minor) extender API change:
it stays protected so handlers can still construct one from an
IsItemCurrentAsync override, but can no longer be derived from.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

---------

Co-authored-by: Henrik <hg@impact.dk>
Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
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