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

API memory allocator #429

Open
sylware opened this issue Dec 20, 2024 · 4 comments
Open

API memory allocator #429

sylware opened this issue Dec 20, 2024 · 4 comments
Labels
enhancement New feature or request

Comments

@sylware
Copy link
Contributor

sylware commented Dec 20, 2024

If I recall properly some data returned from the alsa lib API must be free using the C runtime free().

Would it be appropriate to move away from the C allocator to an API install-able memory allocator (like vulkan3D)?

Backward compatibility would be achieved since the "default memory allocator" would be the C one, if the C runtime is around.

Unless that part of the API is already deprecated (and I should not have used it).

@perexg
Copy link
Member

perexg commented Dec 20, 2024

What's the target purpose?

@sylware
Copy link
Contributor Author

sylware commented Dec 21, 2024

Huh? For the API To be cleanly independent of the C runtime, like the vulkan3D API.

@perexg perexg added the enhancement New feature or request label Dec 21, 2024
@perexg
Copy link
Member

perexg commented Dec 21, 2024

If you don't have a real requirement (a code which will benefit from this change), it's really low priority task. You can propose your implementation for the review.

@sylware
Copy link
Contributor Author

sylware commented Dec 22, 2024

What? This is not the goal of this issue.

It is an issue to bring this subject on the table, in the mind of alsa people. Because, I realized the alsa user code I wrote require the usage of free() right while I was coding some vulkan3D stuff (which API is very carefull about just that).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants