ERC777 icon indicating copy to clipboard operation
ERC777 copied to clipboard

removed gas loops, does not deploy due "This contract does not implement all functions and thus cannot be created."

Open lrgeoemtry opened this issue 7 years ago • 4 comments

hi 777 gets roughly 27 static warnings in remix. you can remove 12 of them by defining a boundry on bytes (at line 252 of this contract: https://gist.github.com/lrgeoemtry/df096412147a1859bae07cc31f51b267) of "bytes32"

doing this though makes it so i cannot deploy the contract getting this error: "This contract does not implement all functions and thus cannot be created." I'm going to keep hacking away at it, but im hopeful the community can help.

the other 15 static analysis warnings are 2 reentrancy issues which we modified with mutex (492-505 & 528-509) and "similar variable names" which is readability deprecation but w/e

any help with this is much appreciated as currently sending a 777 token is like .1 rEth and as expensive as hell.

lrgeoemtry avatar Oct 28 '18 05:10 lrgeoemtry

there must be something with the original contract i deployed to here : https://ropsten.etherscan.io/address/0xc443f11cfa23c1b5a098a46cefb76cc998089a46

because i took the same logic and deployed it here: https://ropsten.etherscan.io/address/0x77752a746a3433b27a21413978b12d145bf2efc7

and the transfers aren't as expensive anymore , and theyre not failing.

lrgeoemtry avatar Oct 28 '18 06:10 lrgeoemtry

i tested the waters to deploy this logic on mainnet getting a warning that it might fail, which i don't receive with ropsten. potentially something fatal?

lrgeoemtry avatar Oct 28 '18 06:10 lrgeoemtry

Actually have same errors. Contracts from here: https://github.com/0xjac/ERC777 Adopted contracts for Solidity EVM v.0.5.6 from that repo (and same) drop in error...

udartsev avatar Mar 19 '19 07:03 udartsev

Thanks for the report, I will investigate. A quick check in Remix revealed nothing so far for me.

0xjac avatar Apr 06 '19 12:04 0xjac