Skip to content

GodotFfi::variant_type can be const#1090

Merged
Bromeon merged 1 commit intogodot-rust:masterfrom
TitanNano:jovan/const_variant_type
Mar 21, 2025
Merged

GodotFfi::variant_type can be const#1090
Bromeon merged 1 commit intogodot-rust:masterfrom
TitanNano:jovan/const_variant_type

Conversation

@TitanNano
Copy link
Contributor

We can turn GodotFfi::variant_type into a const VARIANT_TYPE.

@GodotRust
Copy link

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdext/pr-1090

Copy link
Member

@Bromeon Bromeon left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

This isn't a breaking change btw, as GodotFfi isn't a public API. So we're quite free to change things here, also in the future.

@Bromeon Bromeon added this pull request to the merge queue Mar 21, 2025
@Bromeon Bromeon added quality-of-life No new functionality, but improves ergonomics/internals c: ffi Low-level components and interaction with GDExtension API labels Mar 21, 2025
Merged via the queue into godot-rust:master with commit 75a36ca Mar 21, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: ffi Low-level components and interaction with GDExtension API quality-of-life No new functionality, but improves ergonomics/internals

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants