diff options
| author | Amir Bandeali <abandeali1@gmail.com> | 2018-08-28 01:30:37 +0800 |
|---|---|---|
| committer | Amir Bandeali <abandeali1@gmail.com> | 2018-08-28 02:54:20 +0800 |
| commit | f60adbdd723dcf1e5c980c357a8c2c4c337d3a87 (patch) | |
| tree | ecc352a4ff4ff62c33d52f3ef56d9fa4e43ad97b | |
| parent | 1be310cef49da42e4534a945b95b633d5846d864 (diff) | |
| download | dexon-0x-contracts-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar dexon-0x-contracts-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.gz dexon-0x-contracts-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.bz2 dexon-0x-contracts-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.lz dexon-0x-contracts-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.xz dexon-0x-contracts-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.zst dexon-0x-contracts-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.zip | |
Remove redundant mstores from fillOrderNoThrow
| -rw-r--r-- | packages/contracts/src/2.0.0/extensions/Forwarder/MixinExchangeWrapper.sol | 9 |
1 files changed, 1 insertions, 8 deletions
diff --git a/packages/contracts/src/2.0.0/extensions/Forwarder/MixinExchangeWrapper.sol b/packages/contracts/src/2.0.0/extensions/Forwarder/MixinExchangeWrapper.sol index a7ff400b9..b5b8fc3e0 100644 --- a/packages/contracts/src/2.0.0/extensions/Forwarder/MixinExchangeWrapper.sol +++ b/packages/contracts/src/2.0.0/extensions/Forwarder/MixinExchangeWrapper.sol @@ -69,14 +69,7 @@ contract MixinExchangeWrapper is fillOrderCalldata, // write output over input 128 // output size is 128 bytes ) - switch success - case 0 { - mstore(fillResults, 0) - mstore(add(fillResults, 32), 0) - mstore(add(fillResults, 64), 0) - mstore(add(fillResults, 96), 0) - } - case 1 { + if success { mstore(fillResults, mload(fillOrderCalldata)) mstore(add(fillResults, 32), mload(add(fillOrderCalldata, 32))) mstore(add(fillResults, 64), mload(add(fillOrderCalldata, 64))) |