Skip to content

Call closure_exception destructor#79

Open
VoxSciurorum wants to merge 1 commit into
OpenCilk:devfrom
VoxSciurorum:jfc/destruct
Open

Call closure_exception destructor#79
VoxSciurorum wants to merge 1 commit into
OpenCilk:devfrom
VoxSciurorum:jfc/destruct

Conversation

@VoxSciurorum
Copy link
Copy Markdown
Contributor

This doesn't matter now. Some day the destructor might do work.

@neboat
Copy link
Copy Markdown
Contributor

neboat commented May 29, 2026

Can this method simply call delete exn_r instead of explicitly calling the destructor followed by free?

@VoxSciurorum
Copy link
Copy Markdown
Contributor Author

Can this method simply call delete exn_r instead of explicitly calling the destructor followed by free?

The memory was allocated by a malloc family function and should not be freed by delete.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants