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

Reset init guard flag in destructor #138

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

raguridan
Copy link

Allows resurrection of dead singletons by enabling pthread_once to execute after the current instance has been destroyed.

@FussyDuck
Copy link

FussyDuck commented Jul 17, 2024

CLA assistant check
All committers have signed the CLA.

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 61.94%. Comparing base (7b974be) to head (ea6f6e3).
Report is 818 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #138      +/-   ##
===========================================
- Coverage    64.33%   61.94%   -2.39%     
===========================================
  Files          855      900      +45     
  Lines        48000    49639    +1639     
  Branches         0     3729    +3729     
===========================================
- Hits         30879    30751     -128     
- Misses       17121    18888    +1767     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link

Private downstream CI failed.
Workflow name: private-downstream-ci-hpc
View the logs at https://github.com/ecmwf-actions/private-downstream-ci/actions/runs/9974619856.

@danovaro danovaro merged commit 3d68516 into develop Jul 17, 2024
193 of 194 checks passed
@danovaro danovaro deleted the fix/allow-singleton-resurrection branch July 17, 2024 13:34
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.

5 participants