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.

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

implement Reflect for Box<T> where T: Reflect #6098

Closed

Conversation

jakobhellermann
Copy link
Contributor

TODO: write the PR description

Objective

  • it would be nice to be able to reflect types containing Box<T> where T implements reflect

Solution

  • TODO: describe why Wrapper is introduced instead of passing everything through the inner value

@alice-i-cecile alice-i-cecile added C-Usability A targeted quality-of-life change that makes Bevy easier to use A-Reflection Runtime information about types labels Sep 28, 2022
@Shatur
Copy link
Contributor

Shatur commented Nov 21, 2022

@jakobhellermann Looks very useful. What is the state of it? Are you planning to continue working on it?

@jakobhellermann
Copy link
Contributor Author

@jakobhellermann Looks very useful. What is the state of it? Are you planning to continue working on it?

I think I ran into some problem and didn't continue further at the time, but I don't remember what it was. I'll rebase this over the next week if I have the time and motivation

@richchurcher
Copy link
Contributor

Backlog cleanup: closing due to inactivity, and in favour of #15532. It seems this has had many attempts over the years!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Reflection Runtime information about types C-Usability A targeted quality-of-life change that makes Bevy easier to use
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants