Skip to content

[Helper] Replace console message handler instantiation with singleton usage#5978

Merged
fredroy merged 1 commit into
sofa-framework:masterfrom
alxbilger:consolemessagehandler
Mar 19, 2026
Merged

[Helper] Replace console message handler instantiation with singleton usage#5978
fredroy merged 1 commit into
sofa-framework:masterfrom
alxbilger:consolemessagehandler

Conversation

@alxbilger
Copy link
Copy Markdown
Contributor

@alxbilger alxbilger commented Feb 26, 2026

The behaviour stays the same and it reduces a bit the memory leaks. Related to sofa-framework/SofaPython3#589

[with-all-tests]


By submitting this pull request, I acknowledge that
I have read, understand, and agree SOFA Developer Certificate of Origin (DCO).


Reviewers will merge this pull-request only if

  • it builds with SUCCESS for all platforms on the CI.
  • it does not generate new warnings.
  • it does not generate new unit test failures.
  • it does not generate new scene test failures.
  • it does not break API compatibility.
  • it is more than 1 week old (or has fast-merge label).

@alxbilger alxbilger added pr: fix Fix a bug pr: status to review To notify reviewers to review this pull-request labels Feb 26, 2026
@alxbilger
Copy link
Copy Markdown
Contributor Author

[ci-build][with-all-tests]

@bakpaul
Copy link
Copy Markdown
Contributor

bakpaul commented Mar 12, 2026

[ci-build][force-full-build]

@fredroy fredroy force-pushed the consolemessagehandler branch from 558d955 to 281c58d Compare March 18, 2026 22:32
@fredroy fredroy force-pushed the consolemessagehandler branch from 281c58d to 7589567 Compare March 19, 2026 01:50
@fredroy fredroy added pr: status ready Approved a pull-request, ready to be squashed and removed pr: status to review To notify reviewers to review this pull-request labels Mar 19, 2026
@fredroy fredroy merged commit ea89904 into sofa-framework:master Mar 19, 2026
15 of 17 checks passed
@hugtalbot hugtalbot added this to the v26.06 milestone May 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: fix Fix a bug pr: status ready Approved a pull-request, ready to be squashed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants