Skip to content

Listener adjustment - #1173

Merged
junaed-optimizely merged 1 commit into
masterfrom
junaed/listener-adjust
Aug 20, 2026
Merged

Listener adjustment#1173
junaed-optimizely merged 1 commit into
masterfrom
junaed/listener-adjust

Conversation

@junaed-optimizely

Copy link
Copy Markdown
Contributor

Summary

This pull request makes a small improvement to the DefaultNotificationCenter class by ensuring that listener removers are properly cleaned up after use.

  • After calling a remover function for a listener, the corresponding entry is deleted from the removers map to prevent memory leaks and ensure accurate state tracking.

Test plan

Existing tests should pass

Issues

@junaed-optimizely junaed-optimizely changed the title adjust Listener adjustment Aug 20, 2026
@datadog-optimizely-experimentation

datadog-optimizely-experimentation Bot commented Aug 20, 2026

Copy link
Copy Markdown

Pipelines

⚠️ Warnings

🚦 1 Pipeline job failed

Jira ticket reference check | jira_ticket_reference_check

View in Datadog · View in GitHub Actions

No ticket was referenced in this pull request.

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 2da24bd | Docs | View more details | Give us feedback!

@junaed-optimizely
junaed-optimizely merged commit ce9aa2d into master Aug 20, 2026
20 of 26 checks passed
@junaed-optimizely
junaed-optimizely deleted the junaed/listener-adjust branch August 20, 2026 14:09
@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 77.631% (+0.001%) from 77.63% — junaed/listener-adjust into master

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.

3 participants