From 46d79e5115421285faa709e16ba6156f2c672f38 Mon Sep 17 00:00:00 2001 From: Greg Hysen Date: Thu, 28 Jun 2018 15:41:06 -0700 Subject: Updated hashing comments as-per PR --- .../contracts/current/protocol/Exchange/libs/LibOrder.sol | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'packages') diff --git a/packages/contracts/src/contracts/current/protocol/Exchange/libs/LibOrder.sol b/packages/contracts/src/contracts/current/protocol/Exchange/libs/LibOrder.sol index d1e5da580..954f94f76 100644 --- a/packages/contracts/src/contracts/current/protocol/Exchange/libs/LibOrder.sol +++ b/packages/contracts/src/contracts/current/protocol/Exchange/libs/LibOrder.sol @@ -99,11 +99,7 @@ contract LibOrder is bytes32 makerAssetDataHash = keccak256(order.makerAssetData); bytes32 takerAssetDataHash = keccak256(order.takerAssetData); // Assembly for more efficiently computing: - // keccak256(abi.encodePacked( - // DOMAIN_SEPARATOR_SCHEMA_HASH, - // keccak256(abi.encodePacked(address(this))), - // ORDER_SCHEMA_HASH, - // keccak256(abi.encodePacked( + // keccak256(abi.encode( // order.makerAddress, // order.takerAddress, // order.feeRecipientAddress, @@ -114,10 +110,9 @@ contract LibOrder is // order.takerFee, // order.expirationTimeSeconds, // order.salt, - // keccak256(abi.encodePacked(order.makerAssetData)), - // keccak256(abi.encodePacked(order.takerAssetData)) - // )) - // )); + // keccak256(order.makerAssetData), + // keccak256(order.takerAssetData) + // )); assembly { // Backup let temp1 := mload(sub(order, 32)) -- cgit v1.2.3