aboutsummaryrefslogtreecommitdiffstats
path: root/docs/style-guide.rst
diff options
context:
space:
mode:
authorDax Bondye <beckwithdylan@gmail.com>2018-02-23 02:18:09 +0800
committerDax Bondye <beckwithdylan@gmail.com>2018-02-23 02:23:20 +0800
commit2ac49b3c2d10eb43deb716cdf75ae48cd64840f8 (patch)
tree6b3cc2e068dadeb5fafefed4dc24410eb7670bed /docs/style-guide.rst
parent187e50b14c063cdfc56e085d308f2cb8dacc13bd (diff)
downloaddexon-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.rst18
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::