search

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

Constructor visibility warning issue #61

Closed MXmaster2s closed 3 years ago

MXmaster2s commented 3 years ago

When a constructor visibility is set to public, a warning is shown" Visibility for constructor is ignored. If you want the contract to be non-deployable, making it "abstract" is sufficient However, explicitly mentioning is needed; else there is an error when trying to compile: SyntaxError: No visibility specified. Did you intend to add "public"?

I'm using Truffle v5.1.40 (core: 5.1.40) and Node v14.8.0

tintinweb commented 3 years ago

Hi @MXmaster2s,

this appears to be a solidity compiler warning/error message (see solidity which is not emitted by this extension.

Please refer to the solidity compiler for more information but this may be a good indicator that the compiler has problems with your source code.

cheers 🙌