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

#[godot_api]: calls to std::mem::forget with a value that implements Copy. Forgetting a copy leaves the original intact #80

Closed
bayou-brogrammer opened this issue Jan 21, 2023 · 2 comments
Labels
bug c: register Register classes, functions and other symbols to GDScript

Comments

@bayou-brogrammer
Copy link

Going off the dodge-the-creeps example, deriving #[godot_api] causes the following warning message to display:

calls to std::mem::forgetwith a value that implementsCopy. Forgetting a copy leaves the original intact

image

@Bromeon
Copy link
Member

Bromeon commented Jan 22, 2023

Thanks for the report! I should probably include cargo check (or even cargo clippy) for examples in CI.

@Bromeon Bromeon added bug c: register Register classes, functions and other symbols to GDScript labels Jan 22, 2023
@Bromeon
Copy link
Member

Bromeon commented Feb 5, 2023

This should be fixed with #74.

@Bromeon Bromeon closed this as completed Feb 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug c: register Register classes, functions and other symbols to GDScript
Projects
None yet
Development

No branches or pull requests

2 participants