Skip to content

Add 0.15 release notes for function reflection#1768

Merged
alice-i-cecile merged 4 commits into
bevyengine:mainfrom
MrGVSV:mrgvsv/notes/function-reflection
Nov 5, 2024
Merged

Add 0.15 release notes for function reflection#1768
alice-i-cecile merged 4 commits into
bevyengine:mainfrom
MrGVSV:mrgvsv/notes/function-reflection

Conversation

@MrGVSV
MrGVSV force-pushed the mrgvsv/notes/function-reflection branch from a57265c to ad6897f Compare November 3, 2024 19:33
Comment thread release-content/0.15/release-notes/function_reflection.md Outdated
Comment thread release-content/0.15/release-notes/function_reflection.md Outdated
Comment thread release-content/0.15/release-notes/function_reflection.md Outdated
Comment thread release-content/0.15/release-notes/function_reflection.md Outdated
Comment thread release-content/0.15/release-notes/function_reflection.md Outdated
Comment thread release-content/0.15/release-notes/function_reflection.md Outdated
@MiniaczQ

MiniaczQ commented Nov 3, 2024

Copy link
Copy Markdown
Contributor

I didn't see anything about overloading, will that be in separate notes or did that not land?

@alice-i-cecile

Copy link
Copy Markdown
Member

Overloading didn't land IIRC.

@MrGVSV

MrGVSV commented Nov 4, 2024

Copy link
Copy Markdown
Member Author

Overloading didn't land IIRC.

Yeah unfortunately. But not too big a deal since that at least gives us more time to test it out.

I did briefly mention it in the Limitations section, but I forgot to link to the PR. I'll be sure to do that!

MrGVSV and others added 3 commits November 5, 2024 11:22
Co-authored-by: Alice Cecile <alice.i.cecile@gmail.com>
Co-authored-by: MiniaczQ <xnetroidpl@gmail.com>
Comment on lines +99 to +105
Using `FunctionInfo`, a `DynamicFunction` will print out its signature when debug-printed.

```rust
dbg!(String::len.into_function());
// Outputs:
// DynamicFunction(fn alloc::string::String::len(_: &alloc::string::String) -> usize)
```

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just added this because I think it's really cool we can still print out the full signature, but let me know if it's too much to include for the release post.

@alice-i-cecile
alice-i-cecile added this pull request to the merge queue Nov 5, 2024
Merged via the queue into bevyengine:main with commit 5fb7cf8 Nov 5, 2024
@MrGVSV
MrGVSV deleted the mrgvsv/notes/function-reflection branch November 5, 2024 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants