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

✨ DynamicCopy Cleanup #474

Open
1 task done
SteveMaier-IRT opened this issue Oct 12, 2023 · 0 comments
Open
1 task done

✨ DynamicCopy Cleanup #474

SteveMaier-IRT opened this issue Oct 12, 2023 · 0 comments
Labels
enhancement New feature or request

Comments

@SteveMaier-IRT
Copy link
Collaborator

Is there an existing issue for this?

  • I have searched the existing issues

Suggestion

In the convert.hpp there are multiple DynamicCopy methods. The ones that allocate strings use "malloc/free" and the ones that work with arrays of data use "new/delete" to allocate out the memory.

This should be using "new/delete" for all of the methods to make it simpler in the C# code for which deleting function to call when the code is done with the data.

Possible Implementation

No response

Anything else?

No response

@SteveMaier-IRT SteveMaier-IRT added the enhancement New feature or request label Oct 12, 2023
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

1 participant