Skip to content

[Merged by Bors] - add ReflectDefault to std types#6429

Closed
jakobhellermann wants to merge 2 commits intobevyengine:mainfrom
jakobhellermann:reflect-default-std
Closed

[Merged by Bors] - add ReflectDefault to std types#6429
jakobhellermann wants to merge 2 commits intobevyengine:mainfrom
jakobhellermann:reflect-default-std

Conversation

@jakobhellermann
Copy link
Contributor

Objective

  • ReflectDefault can be used to create default values for reflected types
  • std primitives that are Default-constructable should register ReflectDefault

Solution

  • register ReflectDefault

@jakobhellermann jakobhellermann added C-Feature A new feature, making something new possible A-Reflection Runtime information about types labels Oct 31, 2022
@alice-i-cecile alice-i-cecile added the S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it label Oct 31, 2022
@alice-i-cecile
Copy link
Member

bors r+

bors bot pushed a commit that referenced this pull request Oct 31, 2022
# Objective

- `ReflectDefault` can be used to create default values for reflected types
- `std` primitives that are `Default`-constructable should register `ReflectDefault`

## Solution

- register `ReflectDefault`
@bors bors bot changed the title add ReflectDefault to std types [Merged by Bors] - add ReflectDefault to std types Oct 31, 2022
@bors bors bot closed this Oct 31, 2022
@jakobhellermann jakobhellermann deleted the reflect-default-std branch October 31, 2022 16:52
ItsDoot pushed a commit to ItsDoot/bevy that referenced this pull request Feb 1, 2023
# Objective

- `ReflectDefault` can be used to create default values for reflected types
- `std` primitives that are `Default`-constructable should register `ReflectDefault`

## Solution

- register `ReflectDefault`
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-Feature A new feature, making something new possible S-Ready-For-Final-Review This PR has been approved by the community. It's ready for a maintainer to consider merging it

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants