Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Make repr_c::Box<dyn FnMut…> and by extension fn pointers, Optionable #239

Merged
merged 2 commits into from
Sep 17, 2024
Merged

Make repr_c::Box<dyn FnMut…> and by extension fn pointers, Optionable #239

merged 2 commits into from
Sep 17, 2024

Conversation

danielhenrymantilla
Copy link
Collaborator

@danielhenrymantilla danielhenrymantilla commented Sep 17, 2024 •
edited
Loading

We should do something similar for Arc, btw

@danielhenrymantilla danielhenrymantilla self-assigned this Sep 17, 2024
@danielhenrymantilla danielhenrymantilla force-pushed the dhm/niche-for-box-dyn-fn-and-fns branch from f14bad4 to 56e5088 Compare September 17, 2024 18:01
@danielhenrymantilla danielhenrymantilla merged commit 501b1d4 into master Sep 17, 2024
206 checks passed
@danielhenrymantilla danielhenrymantilla deleted the dhm/niche-for-box-dyn-fn-and-fns branch September 17, 2024 18:06
@p-avital
Copy link
Contributor

Got there after the battle, but yes, this is a good thing that should also be ported to ArcDynFn :D

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@p-avital p-avital Awaiting requested review from p-avital

Assignees

@danielhenrymantilla danielhenrymantilla

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@danielhenrymantilla @p-avital