diff options
author | DYLAN BECKWITH <beckwithdylan@gmail.com> | 2018-02-15 05:09:52 +0800 |
---|---|---|
committer | Dax Bondye <beckwithdylan@gmail.com> | 2018-02-22 23:01:07 +0800 |
commit | 187e50b14c063cdfc56e085d308f2cb8dacc13bd (patch) | |
tree | 883392b1f8aab1853259b0ae35340be4c290f8fc /docs/style-guide.rst | |
parent | 834dac79895811cde1e41d8b4de5d4ef6d758303 (diff) | |
download | dexon-solidity-187e50b14c063cdfc56e085d308f2cb8dacc13bd.tar.gz dexon-solidity-187e50b14c063cdfc56e085d308f2cb8dacc13bd.tar.zst dexon-solidity-187e50b14c063cdfc56e085d308f2cb8dacc13bd.zip |
Recommend consistent style for wrapping long lines.
Diffstat (limited to 'docs/style-guide.rst')
-rw-r--r-- | docs/style-guide.rst | 103 |
1 files changed, 103 insertions, 0 deletions
diff --git a/docs/style-guide.rst b/docs/style-guide.rst index 4c0d44f0..7d513d7a 100644 --- a/docs/style-guide.rst +++ b/docs/style-guide.rst @@ -112,6 +112,109 @@ No:: } } +Maximum Line Length +=================== + +Keeping lines under the `PEP 8 recommendation <https://www.python.org/dev/peps/pep-0008/#maximum-line-length>`_ of 79 (or 99) +characters helps readers easily parse the code. + +Wrapped lines should conform to the following guidelines. + +1. The first argument should not be attached to the opening parenthesis. +2. One, and only one, indent should be used. +3. Each argument should fall on its own line. +4. The terminating element, :code:`);`, should be placed on the final line by itself. + +Function Calls + +Yes:: + + thisFunctionCallIsReallyLong( + longArgument1, + longArgument2, + longArgument3 + ); + +No:: + + thisFunctionCallIsReallyLong(longArgument1, + longArgument2, + longArgument3 + ); + + thisFunctionCallIsReallyLong(longArgument1, + longArgument2, + longArgument3 + ); + + thisFunctionCallIsReallyLong( + longArgument1, longArgument2, + longArgument3 + ); + + thisFunctionCallIsReallyLong( + longArgument1, + longArgument2, + longArgument3 + ); + + thisFunctionCallIsReallyLong( + longArgument1, + longArgument2, + longArgument3); + +Assignment Statements + +Yes:: + + thisIsALongNestedMapping[being][set][to_some_value] = someFunction( + argument1, + argument2, + argument3, + argument4 + ); + +No:: + + thisIsALongNestedMapping[being][set][to_some_value] = someFunction(argument1, + argument2, + argument3, + argument4); + +Event Definitions and Event Emitters + +Yes:: + + event LongAndLotsOfArgs( + adress sender, + adress recipient, + uint256 publicKey, + uint256 amount, + bytes32[] options + ); + + LongAndLotsOfArgs( + sender, + recipient, + publicKey, + amount, + options + ); + +No:: + + event LongAndLotsOfArgs(adress sender, + adress recipient, + uint256 publicKey, + uint256 amount, + bytes32[] options); + + LongAndLotsOfArgs(sender, + recipient, + publicKey, + amount, + options); + Source File Encoding ==================== |