From 3c683eed240ba8530512dd37b36b5c28f8c00c2f Mon Sep 17 00:00:00 2001
From: barak manos <>
Date: Thu, 15 Aug 2024 13:09:52 +0300
Subject: [PATCH] Remove unused input argument `forceLegacyTradeBySourceRange`

---
 src/trade-matcher/trade.ts | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/trade-matcher/trade.ts b/src/trade-matcher/trade.ts
index 8177efd..9a15935 100644
--- a/src/trade-matcher/trade.ts
+++ b/src/trade-matcher/trade.ts
@@ -37,8 +37,7 @@ const getEncodedTradeBySourceAmount = (
   z: BigNumber,
   A: BigNumber,
   B: BigNumber,
-  simulated: boolean,
-  forceLegacyTradeBySourceRange: boolean = false
+  simulated: boolean
 ): BigNumber => {
   if (simulated) {
     const temp1 = z.mul(C);
@@ -69,7 +68,7 @@ const getEncodedTradeBySourceAmount = (
   // Before - in a case of overflow it would end up with a 0 rate and not get
   // picked up for trade. This added check is to avoid the order being picked up
   // for trade - just to be reverted in the case of overflow by the contract.
-  if (forceLegacyTradeBySourceRange || isLegacyTradeBySourceRange || temp4.add(temp5).lte(MAX_UINT256)) {
+  if (isLegacyTradeBySourceRange || temp4.add(temp5).lte(MAX_UINT256)) {
     return mulDivF(temp2, temp3.div(factor), temp4.add(temp5));
   }
   return temp2.div(add(A, mulDivC(temp1, temp1, temp3)));