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

Fix missing MAKE_TYPED_ARRAY_INFO for Packed*Arrays #1558

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

aaronfranke
Copy link
Member

@aaronfranke aaronfranke commented Aug 22, 2024

Without this fix, it is impossible to bind a method using TypedArray<Packed*Array> where * can be several things.

Can be cherry-picked to 4.3 as-is, but needs to have PackedVector4Array removed for 4.2 and earlier. #1559

@aaronfranke aaronfranke added bug This has been identified as a bug cherrypick:4.3 labels Aug 22, 2024
@aaronfranke aaronfranke added this to the 4.4 milestone Aug 22, 2024
@aaronfranke aaronfranke requested a review from a team as a code owner August 22, 2024 01:34
@dsnopek
Copy link
Collaborator

dsnopek commented Aug 22, 2024

Thanks! The code looks good to me :-)

However, could you bind a function in the test project that returns a TypedArray<PackedInt32Array> (or one of the other packed array types) so that we have this represented somewhat in our automated tests?

Copy link
Collaborator

@dsnopek dsnopek left a comment

Choose a reason for hiding this comment

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

Thanks!

@dsnopek dsnopek merged commit 16bc0a0 into godotengine:master Aug 26, 2024
12 checks passed
@aaronfranke aaronfranke deleted the fix-typed-packed-array-bind branch August 26, 2024 20:55
@dsnopek
Copy link
Collaborator

dsnopek commented Sep 3, 2024

Cherry-picked for 4.3 in PR #1569

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This has been identified as a bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants