forest icon indicating copy to clipboard operation
forest copied to clipboard

Published 20 hours ago • verified publisher icon ChainSafe

interpreter tests are just copy-pasted

Open aatifsyed opened this issue 2 years ago • 1 comments

https://github.com/ChainSafe/forest/blob/f54d8d7f31ed5f1c8f96819e0a800c5466da71a6/src/interpreter/fvm2.rs#L297-L327 https://github.com/ChainSafe/forest/blob/f54d8d7f31ed5f1c8f96819e0a800c5466da71a6/src/interpreter/fvm3.rs#L319-L349 https://github.com/ChainSafe/forest/blob/f54d8d7f31ed5f1c8f96819e0a800c5466da71a6/src/interpreter/fvm4.rs#L319-L349

Needs refactoring

aatifsyed avatar Oct 03 '23 16:10 aatifsyed

How much of the fvm code is copy-paste? I suspect it needs a bunch of refactoring

aatifsyed avatar Oct 03 '23 16:10 aatifsyed