diff options
author | Amir Bandeali <abandeali1@gmail.com> | 2018-07-18 07:58:00 +0800 |
---|---|---|
committer | Amir Bandeali <abandeali1@gmail.com> | 2018-07-23 23:00:23 +0800 |
commit | 4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9 (patch) | |
tree | cb5b73b17cd8da071df2e34a8b43769a4b95b0fa /packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol | |
parent | 3506ec1caa2a6c7ee2d68970553ca53d8ceb5c64 (diff) | |
download | dexon-sol-tools-4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9.tar dexon-sol-tools-4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9.tar.gz dexon-sol-tools-4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9.tar.bz2 dexon-sol-tools-4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9.tar.lz dexon-sol-tools-4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9.tar.xz dexon-sol-tools-4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9.tar.zst dexon-sol-tools-4636d5fbc2ad3350278dc9fa2e780c2afa08d3d9.zip |
Store orders length in varible before looping over orders
Diffstat (limited to 'packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol')
-rw-r--r-- | packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol b/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol index 9dc203373..561507ce4 100644 --- a/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol +++ b/packages/contracts/src/2.0.0/forwarder/MixinForwarderCore.sol @@ -245,7 +245,8 @@ contract MixinForwarderCore is orders[0].takerAssetData = WETH_ASSET_DATA; // All orders are required to have the same makerAssetData. We save on gas by reusing the makerAssetData of the first order. - for (uint256 i = 0; i < orders.length; i++) { + uint256 ordersLength = orders.length; + for (uint256 i = 0; i < ordersLength; i++) { orders[i].makerAssetData = orders[0].makerAssetData; } @@ -274,7 +275,8 @@ contract MixinForwarderCore is bytes memory wethAssetData = WETH_ASSET_DATA; // All orders are required to have WETH as takerAssetData. We save on gas by populating the orders here, rather than passing in any extra calldata. - for (uint256 i = 0; i < orders.length; i++) { + uint256 ordersLength = orders.length; + for (uint256 i = 0; i < ordersLength; i++) { orders[i].takerAssetData = wethAssetData; } @@ -311,7 +313,8 @@ contract MixinForwarderCore is bytes memory wethAssetData = WETH_ASSET_DATA; uint256 zrxPurchased = 0; - for (uint256 i = 0; i < orders.length; i++) { + uint256 ordersLength = orders.length; + for (uint256 i = 0; i < ordersLength; i++) { // All of these are ZRX/WETH, so we can drop the respective assetData from calldata. orders[i].makerAssetData = zrxAssetData; |