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.

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

.free in debug mode should set the memory to undefined #4087

Closed
daurnimator opened this issue Jan 5, 2020 · 1 comment · Fixed by #4133
Closed

.free in debug mode should set the memory to undefined #4087

daurnimator opened this issue Jan 5, 2020 · 1 comment · Fixed by #4133
Labels
accepted This proposal is planned. contributor friendly This issue is limited in scope and/or knowledge of Zig internals. proposal This issue suggests modifications. If it also has the "accepted" label then it is planned. standard library This issue involves writing Zig code for the standard library.
Milestone
0.6.0

Comments

@daurnimator
Copy link
Contributor

Originally posted by @daurnimator in #2498 (comment)

One thing that is missing is that .free in debug mode should set the memory to undefined.
@andrewrk andrewrk added proposal This issue suggests modifications. If it also has the "accepted" label then it is planned. standard library This issue involves writing Zig code for the standard library. labels Jan 7, 2020
@andrewrk andrewrk added this to the 0.6.0 milestone Jan 7, 2020
@andrewrk
Copy link
Member

andrewrk commented Jan 7, 2020

Yes it makes sense to do it in the interface rather than in allocator implementations.

@andrewrk andrewrk added accepted This proposal is planned. contributor friendly This issue is limited in scope and/or knowledge of Zig internals. labels Jan 7, 2020
@daurnimator daurnimator mentioned this issue Jan 10, 2020
Merged
@andrewrk andrewrk mentioned this issue Jan 27, 2020
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
accepted This proposal is planned. contributor friendly This issue is limited in scope and/or knowledge of Zig internals. proposal This issue suggests modifications. If it also has the "accepted" label then it is planned. standard library This issue involves writing Zig code for the standard library.
Projects
None yet
Milestone
0.6.0
Development

Successfully merging a pull request may close this issue.

Sets memory to undefined when freed from allocator daurnimator/zig
2 participants
@andrewrk @daurnimator