diff options
author | Chris Ward <chris.ward@ethereum.org> | 2019-01-16 23:11:23 +0800 |
---|---|---|
committer | Chris Ward <chris.ward@ethereum.org> | 2019-01-16 23:11:23 +0800 |
commit | eab9e58e7e6b815e7dfb288510b49d52fe579fe2 (patch) | |
tree | 8f293280285a84c55cd87fd2ae110e808012c252 | |
parent | 778b14de260a7eeaea88867e39cfc226f1494e63 (diff) | |
download | dexon-solidity-eab9e58e7e6b815e7dfb288510b49d52fe579fe2.tar.gz dexon-solidity-eab9e58e7e6b815e7dfb288510b49d52fe579fe2.tar.zst dexon-solidity-eab9e58e7e6b815e7dfb288510b49d52fe579fe2.zip |
Add call warning into main docs and remove FAQ item
-rw-r--r-- | docs/frequently-asked-questions.rst | 12 | ||||
-rw-r--r-- | docs/units-and-global-variables.rst | 4 |
2 files changed, 4 insertions, 12 deletions
diff --git a/docs/frequently-asked-questions.rst b/docs/frequently-asked-questions.rst index 4635d577..3b59af81 100644 --- a/docs/frequently-asked-questions.rst +++ b/docs/frequently-asked-questions.rst @@ -54,18 +54,6 @@ Yes, you can use ``abi.encodePacked``:: } } - -Why is the low-level function ``.call()`` less favorable than instantiating a contract with a variable (``ContractB b;``) and executing its functions (``b.doSomething();``)? -============================================================================================================================================================================= - -If you use actual functions, the compiler will tell you if the types -or your arguments do not match, if the function does not exist -or is not visible and it will do the packing of the -arguments for you. - -See `ping.sol <https://github.com/fivedogit/solidity-baby-steps/blob/master/contracts/45_ping.sol>`_ and -`pong.sol <https://github.com/fivedogit/solidity-baby-steps/blob/master/contracts/45_pong.sol>`_. - What happens if you send ether along with a function call to a contract? ======================================================================== diff --git a/docs/units-and-global-variables.rst b/docs/units-and-global-variables.rst index 59acfcff..e44caa8f 100644 --- a/docs/units-and-global-variables.rst +++ b/docs/units-and-global-variables.rst @@ -200,6 +200,10 @@ Members of Address Types For more information, see the section on :ref:`address`. .. warning:: + You should avoid using ``.call()`` when executing another contract function as it bypasses the compiler type checking, + function existence check, and argument packing. + +.. warning:: There are some dangers in using ``send``: The transfer fails if the call stack depth is at 1024 (this can always be forced by the caller) and it also fails if the recipient runs out of gas. So in order to make safe Ether transfers, always check the return value of ``send``, use ``transfer`` or even better: |