Consensys / vscode-solidity-auditor

Solidity language support and visual security auditor for Visual Studio Code
https://marketplace.visualstudio.com/items?itemName=tintinweb.solidity-visual-auditor
GNU General Public License v3.0
569 stars 60 forks source link

Overloaded function names not displayed as expected #138

Closed dmuhs closed 8 months ago

dmuhs commented 10 months ago

For example, these two functions have the same name but different parameters:

https://github.com/defi-wonderland/xTokens/blob/dev/solidity/contracts/XERC20Lockbox.sol#L45-L65

The public state-changing methods context view only shows one deposit function, which always indicates the last occurrence.

Similarly, the external calls context only has one deposit function entry. That entry shows the external calls from both functions mixed together.

vquelque commented 8 months ago

ack 👍 - will be fixed in the next release.