Skip to content

fix: release temporary factory host application references in VST3 - #548

Open
rmt wants to merge 1 commit into
DISTRHO:developfrom
rmt:fix/vst3-factory-host-reference-leak
Open

rmt wants to merge 1 commit into
DISTRHO:developfrom
rmt:fix/vst3-factory-host-reference-leak

Conversation

@rmt

@rmt rmt commented Oct 6, 2026

Copy link
Copy Markdown

Fixes #547

@falkTX

falkTX commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

Thanks for the PR, makes sense to me. But I dont think the comments there are necessary.

When I wrote that part it was beginning of vst3 details and DPF, I didnt know querying something would automatically assign ++refcount internally.

@rmt
rmt force-pushed the fix/vst3-factory-host-reference-leak branch from fe83bad to 120e7e8 Compare October 6, 2026 19:50
@falkTX

falkTX commented Oct 6, 2026

Copy link
Copy Markdown
Contributor

thank you.

I forgot but please rebase against develop instead of main. new commits always go to develop first before reaching main

@rmt
rmt force-pushed the fix/vst3-factory-host-reference-leak branch from 120e7e8 to e27fa13 Compare October 6, 2026 21:16
@dromer

dromer commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

And change the target branch to develop, not main :)

@rmt
rmt changed the base branch from main to develop October 7, 2026 12:43

This branch has not been deployed

No deployments
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.

Reference leak in DistrhoPluginVST3.cpp

3 participants