Skip to content

Conversation

@zomglings
Copy link
Contributor

…rate"

The issue was that copy(targetVar[:], intermediateVar) doesn't work if targetVar has not been allocated in memory.

Fixed now, and tested using ZigZagZog game contract: https://testnet.game7.io/tx/0x7405984f6c243a341d75a64571e2546114d7c8d5e002a3cccf333b800c2d3d16

…rate"

The issue was that `copy(targetVar[:], intermediateVar)` doesn't work if `targetVar` has not been allocated
in memory.

Fixed now, and tested using ZigZagZog game contract:
https://testnet.game7.io/tx/0x7405984f6c243a341d75a64571e2546114d7c8d5e002a3cccf333b800c2d3d16
Copy link
Contributor

@karacurt karacurt left a comment

Choose a reason for hiding this comment

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

LGTM

@zomglings zomglings merged commit 8591681 into main Mar 24, 2025
1 check passed
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