remove unnecessary check

yinyiqian1 · Sep 19, 2024 · 384a73c · 384a73c
1 parent 7a4429e
commit 384a73c
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 8 deletions.
diff --git a/src/test/app/AMMClawback_test.cpp b/src/test/app/AMMClawback_test.cpp
@@ -1469,7 +1469,9 @@ class AMMClawback_test : public jtx::AMMTest
     void
     testNotHoldingLptoken(FeatureBitset features)
     {
-        testcase("test AMMClawback from account which does not own any lptoken in the pool");
+        testcase(
+            "test AMMClawback from account which does not own any lptoken in "
+            "the pool");
         using namespace jtx;
 
         Env env(*this, features);

diff --git a/src/xrpld/app/tx/detail/AMMWithdraw.cpp b/src/xrpld/app/tx/detail/AMMWithdraw.cpp
@@ -399,7 +399,8 @@ AMMWithdraw::applyGuts(Sandbox& sb)
         {
             TER result;
             STAmount newLPTokenBalance;
-            bool withdrawAll = ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
+            bool withdrawAll =
+                ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
             std::tie(result, newLPTokenBalance, std::ignore, std::ignore) =
                 equalWithdrawTokens(
                     sb,
@@ -570,7 +571,8 @@ AMMWithdraw::equalWithdrawLimit(
     STAmount newLPTokenBalance;
     auto frac = Number{amount} / amountBalance;
     auto const amount2Withdraw = amount2Balance * frac;
-    bool withdrawAll = ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
+    bool withdrawAll =
+        ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
     if (amount2Withdraw <= amount2)
     {
         std::tie(result, newLPTokenBalance, std::ignore, std::ignore) =
@@ -632,7 +634,8 @@ AMMWithdraw::singleWithdraw(
 
     TER result;
     STAmount newLPTokenBalance;
-    bool withdrawAll = ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
+    bool withdrawAll =
+        ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
     std::tie(result, newLPTokenBalance, std::ignore, std::ignore) = withdraw(
         view,
         ammAccount,
@@ -678,7 +681,8 @@ AMMWithdraw::singleWithdrawTokens(
     {
         TER result;
         STAmount newLPTokenBalance;
-        bool withdrawAll = ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
+        bool withdrawAll =
+            ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
         std::tie(result, newLPTokenBalance, std::ignore, std::ignore) =
             withdraw(
                 view,
@@ -752,7 +756,8 @@ AMMWithdraw::singleWithdrawEPrice(
     {
         TER result;
         STAmount newLPTokenBalance;
-        bool withdrawAll = ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
+        bool withdrawAll =
+            ctx_.tx[sfFlags] & (tfWithdrawAll | tfOneAssetWithdrawAll);
         std::tie(result, newLPTokenBalance, std::ignore, std::ignore) =
             withdraw(
                 view,

diff --git a/src/xrpld/app/tx/detail/InvariantCheck.h b/src/xrpld/app/tx/detail/InvariantCheck.h
@@ -465,8 +465,7 @@ using InvariantChecks = std::tuple<
     ValidNewAccountRoot,
     ValidNFTokenPage,
     NFTokenCountTracking,
-    ValidClawback,
-    ValidAMMClawback>;
+    ValidClawback>;
 
 /**
  * @brief get a tuple of all invariant checks