Rephrased some of the math in MixinMatchOrders to improve readability

1 files changed, 3 insertions, 3 deletions

diff --git a/packages/contracts/src/2.0.0/protocol/Exchange/MixinMatchOrders.sol b/packages/contracts/src/2.0.0/protocol/Exchange/MixinMatchOrders.sol
index 8f289ee85..226a1bfb6 100644
--- a/packages/contracts/src/2.0.0/protocol/Exchange/MixinMatchOrders.sol
+++ b/packages/contracts/src/2.0.0/protocol/Exchange/MixinMatchOrders.sol
@@ -184,12 +184,12 @@ contract MixinMatchOrders is
             // Case 1: Right order is fully filled
             matchedFillResults.right.makerAssetFilledAmount = rightMakerAssetAmountRemaining;
             matchedFillResults.right.takerAssetFilledAmount = rightTakerAssetAmountRemaining;
+            matchedFillResults.left.takerAssetFilledAmount = matchedFillResults.right.makerAssetFilledAmount;
             matchedFillResults.left.makerAssetFilledAmount = getPartialAmountFloor(
                 leftOrder.makerAssetAmount,
                 leftOrder.takerAssetAmount,
-                matchedFillResults.right.makerAssetFilledAmount
+                matchedFillResults.left.takerAssetFilledAmount
             );
-            matchedFillResults.left.takerAssetFilledAmount = matchedFillResults.right.makerAssetFilledAmount;
         } else {
             // Case 2: Left order is fully filled
             matchedFillResults.left.makerAssetFilledAmount = leftMakerAssetAmountRemaining;
@@ -198,7 +198,7 @@ contract MixinMatchOrders is
             matchedFillResults.right.takerAssetFilledAmount = getPartialAmountCeil(
                 rightOrder.takerAssetAmount,
                 rightOrder.makerAssetAmount,
-                matchedFillResults.left.takerAssetFilledAmount
+                matchedFillResults.right.makerAssetFilledAmount
             );
         }