Skip to content

Add missing enum & ctor for PackedVector4Array implementation#1497

Merged
dsnopek merged 1 commit into
godotengine:masterfrom
Naros:fix-variant-hpp-packed-vector4-array
Jun 25, 2024
Merged

Add missing enum & ctor for PackedVector4Array implementation#1497
dsnopek merged 1 commit into
godotengine:masterfrom
Naros:fix-variant-hpp-packed-vector4-array

Conversation

@Naros

@Naros Naros commented Jun 19, 2024

Copy link
Copy Markdown
Contributor

@dsnopek I think this may have been overlooked in #1456

Comment thread include/godot_cpp/variant/variant.hpp Outdated
@dsnopek

dsnopek commented Jun 19, 2024

Copy link
Copy Markdown
Collaborator

Thanks!

However, this only appears to update variant.hpp - shouldn't the implementations for these methods need to be added to variant.cpp as well?

@Naros

Naros commented Jun 19, 2024

Copy link
Copy Markdown
Contributor Author

Thanks!

However, this only appears to update variant.hpp - shouldn't the implementations for these methods need to be added to variant.cpp as well?

Ya it does, fixed the PR.

@Naros
Naros force-pushed the fix-variant-hpp-packed-vector4-array branch from 0167d65 to 78b6320 Compare June 19, 2024 15:21
@Naros
Naros requested a review from akien-mga June 19, 2024 15:21

@dsnopek dsnopek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

I haven't tested it, but the code looks good to me. Assuming this passes CI, I think it's good to go :-)

@dsnopek
dsnopek merged commit 90c6ea2 into godotengine:master Jun 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants