Election-Tech-Initiative / Election-Tech-Initiative/electionguard-core2

✨ DynamicCopy Cleanup

Open
#474 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.