diff options
author | Dax Bondye <beckwithdylan@gmail.com> | 2018-02-23 02:18:09 +0800 |
---|---|---|
committer | Dax Bondye <beckwithdylan@gmail.com> | 2018-02-23 02:23:20 +0800 |
commit | 2ac49b3c2d10eb43deb716cdf75ae48cd64840f8 (patch) | |
tree | 6b3cc2e068dadeb5fafefed4dc24410eb7670bed /docs/style-guide.rst | |
parent | 187e50b14c063cdfc56e085d308f2cb8dacc13bd (diff) | |
download | dexon-solidity-2ac49b3c2d10eb43deb716cdf75ae48cd64840f8.tar.gz dexon-solidity-2ac49b3c2d10eb43deb716cdf75ae48cd64840f8.tar.zst dexon-solidity-2ac49b3c2d10eb43deb716cdf75ae48cd64840f8.zip |
Reccomend to explicitly label the visibility of functions.
Diffstat (limited to 'docs/style-guide.rst')
-rw-r--r-- | docs/style-guide.rst | 18 |
1 files changed, 16 insertions, 2 deletions
diff --git a/docs/style-guide.rst b/docs/style-guide.rst index 7d513d7a..9249f3e1 100644 --- a/docs/style-guide.rst +++ b/docs/style-guide.rst @@ -494,7 +494,7 @@ function body to be kept on the same line as the function declaration. The closing brace should be at the same indentation level as the function declaration. -The opening brace should be preceeded by a single space. +The opening brace should be preceded by a single space. Yes:: @@ -524,7 +524,21 @@ No:: function increment(uint x) public pure returns (uint) { return x + 1;} -The visibility modifiers for a function should come before any custom +You should explicitly label the visibility of all functions, including constructors. + +Yes:: + + function explicitlyPublic(uint val) public { + doSomething(); + } + +No:: + + function implicitlyPublic(uint val) { + doSomething(); + } + +The visibility modifier for a function should come before any custom modifiers. Yes:: |