Skip to content

Property support: Use static unmodifiable Maps for common defaults - #2322

Merged
matthiaso merged 1 commit into
releases/27.1from
features/matthiaso/27.1/449281_improveProperySupportWithDefaultValues
Sep 14, 2026
Merged

matthiaso merged 1 commit into
releases/27.1from
features/matthiaso/27.1/449281_improveProperySupportWithDefaultValues

Conversation

@matthiaso

Copy link
Copy Markdown
Member

Goal: Decrease memory usage when lots of property supports are in use.

449281

@matthiaso matthiaso self-assigned this Aug 31, 2026
@matthiaso
matthiaso marked this pull request as draft August 31, 2026 04:02
@matthiaso
matthiaso force-pushed the features/matthiaso/27.1/449281_improveProperySupportWithDefaultValues branch 11 times, most recently from 3f51b96 to fea7a4b Compare September 1, 2026 11:33
@matthiaso
matthiaso marked this pull request as ready for review September 1, 2026 11:34
@matthiaso
matthiaso requested a review from paolobazzi September 9, 2026 10:00
@matthiaso
matthiaso force-pushed the features/matthiaso/27.1/449281_improveProperySupportWithDefaultValues branch 2 times, most recently from 680f946 to 6299e4b Compare September 9, 2026 19:44
@paolobazzi
paolobazzi force-pushed the features/matthiaso/27.1/449281_improveProperySupportWithDefaultValues branch from 6299e4b to 972c9ab Compare September 14, 2026 11:00
Goal: Decrease memory usage when lots of property supports are in use.

449281
@matthiaso
matthiaso force-pushed the features/matthiaso/27.1/449281_improveProperySupportWithDefaultValues branch from 18f634b to bd0cebd Compare September 14, 2026 14:15

@paolobazzi paolobazzi left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good!

@matthiaso
matthiaso merged commit b6e0b3b into releases/27.1 Sep 14, 2026
4 checks passed
@matthiaso
matthiaso deleted the features/matthiaso/27.1/449281_improveProperySupportWithDefaultValues branch September 14, 2026 14:57
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