Skip to content

Engine deinitialization fix - #13

Merged
na2axl merged 9 commits into
developfrom
fix/deinitialization
Apr 9, 2026
Merged

Engine deinitialization fix#13
na2axl merged 9 commits into
developfrom
fix/deinitialization

Conversation

@na2axl

@na2axl na2axl commented Apr 9, 2026

Copy link
Copy Markdown
Member

No description provided.

na2axl added 2 commits April 9, 2026 11:27
Replaced direct ref counter decrements with centralized Dereference* functions to ensure proper cleanup and halt playing channels when references reach zero. This prevents dangling pointers and ensures resources are released correctly when objects are removed from maps.
@na2axl na2axl self-assigned this Apr 9, 2026
na2axl added 4 commits April 9, 2026 13:38
Change various EngineInternalState helper signatures to accept
const std::shared_ptr<EngineInternalState>& to avoid unnecessary
shared_ptr copies and refcount churn.

Also fixed an issue with compressor plugin not unregistered properly.
This ensure better detection of CPU architectures.
@codecov

codecov Bot commented Apr 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 73.40426% with 25 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/Core/Engine.cpp 72.23% 5 Missing and 10 partials ⚠️
src/Sound/SwitchContainer.cpp 66.67% 3 Missing and 2 partials ⚠️
src/Sound/Sound.cpp 33.34% 3 Missing and 1 partial ⚠️
src/Sound/Collection.cpp 66.67% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/Core/EngineInternalState.h 100.00% <100.00%> (ø)
src/Sound/SoundBank.cpp 63.14% <100.00%> (-0.20%) ⬇️
src/Sound/Collection.cpp 61.72% <66.67%> (-1.07%) ⬇️
src/Sound/Sound.cpp 55.56% <33.34%> (-3.19%) ⬇️
src/Sound/SwitchContainer.cpp 54.97% <66.67%> (+0.83%) ⬆️
src/Core/Engine.cpp 65.18% <72.23%> (+1.76%) ⬆️

... and 43 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@na2axl
na2axl merged commit d7e1632 into develop Apr 9, 2026
51 of 57 checks passed
@na2axl
na2axl deleted the fix/deinitialization branch April 9, 2026 16:46
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.

1 participant