aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAmir Bandeali <abandeali1@gmail.com>2018-08-28 01:30:37 +0800
committerAmir Bandeali <abandeali1@gmail.com>2018-08-28 02:54:20 +0800
commitf60adbdd723dcf1e5c980c357a8c2c4c337d3a87 (patch)
treeecc352a4ff4ff62c33d52f3ef56d9fa4e43ad97b
parent1be310cef49da42e4534a945b95b633d5846d864 (diff)
downloaddexon-sol-tools-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar
dexon-sol-tools-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.gz
dexon-sol-tools-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.bz2
dexon-sol-tools-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.lz
dexon-sol-tools-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.xz
dexon-sol-tools-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.tar.zst
dexon-sol-tools-f60adbdd723dcf1e5c980c357a8c2c4c337d3a87.zip
Remove redundant mstores from fillOrderNoThrow
-rw-r--r--packages/contracts/src/2.0.0/extensions/Forwarder/MixinExchangeWrapper.sol9
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)))