search

NethermindEth / cairo-vm-go

A virtual machine for Cairo written in Go
MIT License
73 stars 43 forks source link

Move `WriteToNthStructField` function to Memory package #426

Closed TAdev0 closed 1 month ago

TAdev0 commented 1 month ago

WriteToNthStructField function is currently defined in Hinter package in operand.go file. This is not the right place for this function, which writes some value to memory. Nothing related to operands.

  1. Move the function to the Memory package in the appropriate file
  2. Update the codebase where it is needed
princeibs commented 1 month ago

Can I hop on this?

charans29 commented 1 month ago

Hello, @TAdev0 shall I work on this, if it's still open for assignment.

EmmanuelAR commented 1 month ago

Hello team @TAdev0 I already covert this function because i have to move WriteToNthStructField too, because inside in the WriteUint256ToAddress i have to call WriteToNthStructField

Heres the PR #428

TAdev0 commented 1 month ago

428