Skip to content
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

Add a method to allow LifecycleManager to free keys #138

Merged
merged 2 commits into from
May 31, 2024
Merged

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented May 31, 2024

Currently there is no way to ever remove a key from the map, this can lead to accumulation of memory as it is strongly referencing the class.

This adds a new method so keys can be removed from the map

Fixes #74

laeubi and others added 2 commits May 31, 2024 18:21
Currently there is no way to ever remove a key from the map, this can
lead to accumulation of memory as it is strongly referencing the class.

This adds a new method so keys can be removed from the map

See eclipse#74

Signed-off-by: Christoph Läubrich <laeubi@laeubi-soft.de>
Copy link

sonarcloud bot commented May 31, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@cstamas cstamas merged commit f7bd80c into main May 31, 2024
13 checks passed
@cstamas cstamas deleted the laeubi-issue_74 branch May 31, 2024 16:26
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.

LifecycleManager reatains classes forever
2 participants