Andrei Toma
Andrei Toma
I think the addStakeholder and removeStakeholder should be internal functions rather than public, as they are only created for the other fucntions in the contract to use them.
When generating a call graph, the extensions sees typecasting like `address(this)` or `address(some contract/interface)` as a function and this is not only incorrect, but also visually clutters the chart. See...
https://github.com/m1guelpf/nft-token-drop/blob/fb3c2c5075367166e8eed6058505198175442321/src/CurrencyToken.sol#L5 The Smart contract only needs a Interface for the ERC721 smart contract(and the address) to interact with it, so you could spend less gas on deployment by replacing the...