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.

Sign up for GitHub

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

Jump to bottom

close memory leak with testing allocator #16

Merged

hendriknielaender merged 4 commits into hendriknielaender:main from FObersteiner:main

Nov 17, 2023

Collaborator

FObersteiner commented Nov 17, 2023

pass allocator by value instead of reference
deinitialize the allocator after run
update examples, showing how to deinitialize the results memory

FObersteiner added 4 commits

November 17, 2023 17:10


          pass allocator by value, deinit after run

ef9666c


          initialize start time with nanoseconds

f875b6f


          Merge branch 'hendriknielaender:main' into main

609735d


          use std.testing.allocator

75aa718

FObersteiner mentioned this pull request

api: remove custom allocator creation #9

Closed

hendriknielaender approved these changes

View reviewed changes

Owner

hendriknielaender left a comment

Great work! The changes to the allocator management, including passing by value and ensuring it's deinitialized, are crucial fixes.

hendriknielaender merged commit 0bfd057 into hendriknielaender:main

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

Labels

None yet