Skip to content

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

Closed
matthiaso wants to merge 1 commit into
releases/26.2from
features/matthiaso/26.2/449281_improveProperySupportWithDefaultValues
Closed

Property support: Use static unmodifiable Maps for common defaults#2041
matthiaso wants to merge 1 commit into
releases/26.2from
features/matthiaso/26.2/449281_improveProperySupportWithDefaultValues

Conversation

@matthiaso

@matthiaso matthiaso commented Feb 20, 2026

Copy link
Copy Markdown
Member

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

449281

@matthiaso matthiaso self-assigned this Feb 20, 2026
@matthiaso
matthiaso force-pushed the features/matthiaso/26.2/449281_improveProperySupportWithDefaultValues branch 7 times, most recently from 0828d57 to 4d8d0f1 Compare February 24, 2026 12:01
@matthiaso
matthiaso requested a review from paolobazzi February 24, 2026 18:43
@matthiaso @paolobazzi
Property support: Use static unmodifiable Maps for common defaults
7a101bf 
Goal: Decrease memory usage when lots of property supports are in use.

449281
@paolobazzi
paolobazzi force-pushed the features/matthiaso/26.2/449281_improveProperySupportWithDefaultValues branch from 4d8d0f1 to 7a101bf Compare June 25, 2026 14:42
paolobazzi
paolobazzi reviewed Jun 25, 2026
View reviewed changes

@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.

See inputs for minor improvements

Comment thread ...t.platform.test/src/test/java/org/eclipse/scout/rt/platform/reflect/DefaultValueMapTest.java
Comment thread ...lient.test/src/test/java/org/eclipse/scout/rt/client/ui/BasicPropertySupportFactoryTest.java
Comment thread ...cout.rt.client/src/main/java/org/eclipse/scout/rt/client/ui/BasicPropertySupportFactory.java
Comment thread ...cout.rt.client/src/main/java/org/eclipse/scout/rt/client/ui/BasicPropertySupportFactory.java
Comment thread ...cout.rt.client/src/main/java/org/eclipse/scout/rt/client/ui/BasicPropertySupportFactory.java
Comment thread ...e.scout.rt.platform/src/main/java/org/eclipse/scout/rt/platform/reflect/DefaultValueMap.java
Comment thread ...e.scout.rt.platform/src/main/java/org/eclipse/scout/rt/platform/reflect/DefaultValueMap.java
Comment thread ...t.platform.test/src/test/java/org/eclipse/scout/rt/platform/reflect/DefaultValueMapTest.java
Comment thread ...t.platform.test/src/test/java/org/eclipse/scout/rt/platform/reflect/DefaultValueMapTest.java
Comment thread ...t.platform.test/src/test/java/org/eclipse/scout/rt/platform/reflect/DefaultValueMapTest.java
@matthiaso
matthiaso marked this pull request as draft August 31, 2026 04:02
@matthiaso

matthiaso commented Aug 31, 2026 •
edited
Loading

Copy link
Copy Markdown
Member Author

Added cherry-pick for 27/1, see #2322

  • Apply review comments to newly created pull request
  • Close this pull request / remove branch

@matthiaso matthiaso closed this Sep 14, 2026
@matthiaso
matthiaso deleted the features/matthiaso/26.2/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

Reviewers

@paolobazzi paolobazzi paolobazzi left review comments

Assignees

@matthiaso matthiaso

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@matthiaso @paolobazzi