Election-Tech-Initiative / Election-Tech-Initiative/electionguard-core2
✨ DynamicCopy Cleanup
Open
enhancement
- Dominant language
- C#
- Stars
- 14
- Forks
- 12
- PR merge metrics
- No merged PRs in 30d
Description
### Is there an existing issue for this?
- [X] 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_
Contributor guide
Assessment
This issue has not been assessed yet.