Skip to content

Conversation

@lizhenneng
Copy link
Contributor

Make sure to read the contributing guidelines before submitting a PR

@ngxson
Copy link
Collaborator

ngxson commented Dec 9, 2025

I don't think that will be a valid fix as-is, because technically both smpl and ctx must be freed.

Since simple.cpp is already written in cpp, I think it's probably be better to refactor this example to be RAII, rather than adding *_free() everywhere

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants