Skip to content

Commit

Permalink
Remove CFT Direct Step from path
Browse files Browse the repository at this point in the history
* Remove DirectStepCFT
* Remove CFT from the payment eng
* Handle direct CFT payment in payment transactor
  • Loading branch information
gregtatcam committed Dec 5, 2023
1 parent d0278c2 commit a4f5af2
Show file tree
Hide file tree
Showing 12 changed files with 64 additions and 1,146 deletions.
1 change: 0 additions & 1 deletion Builds/CMake/RippledCore.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -489,7 +489,6 @@ target_sources (rippled PRIVATE
src/ripple/app/paths/impl/AMMOffer.cpp
src/ripple/app/paths/impl/BookStep.cpp
src/ripple/app/paths/impl/DirectStep.cpp
src/ripple/app/paths/impl/DirectStepCFT.cpp
src/ripple/app/paths/impl/PaySteps.cpp
src/ripple/app/paths/impl/XRPEndpointStep.cpp
src/ripple/app/rdb/backend/detail/impl/Node.cpp
Expand Down
19 changes: 0 additions & 19 deletions src/ripple/app/paths/Flow.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -194,25 +194,6 @@ flow(
}

assert(!srcIsXRP && !dstIsXRP);
if (srcIssue.currency.isCFT())
{
assert(dstIssue.currency.isCFT());
return finishFlow(
sb,
srcIssue,
dstIssue,
flow<CFTAmount, CFTAmount>(
sb,
strands,
asDeliver.cft,
partialPayment,
offerCrossing,
limitQuality,
sendMax,
j,
ammContext,
flowDebugInfo));
}
return finishFlow(
sb,
srcIssue,
Expand Down
2 changes: 1 addition & 1 deletion src/ripple/app/paths/Pathfinder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -944,7 +944,7 @@ Pathfinder::addLink(
auto const& uEndCurrency = pathEnd.getCurrency();
auto const& uEndIssuer = pathEnd.getIssuerID();
auto const& uEndAccount = pathEnd.getAccountID();
bool const bOnXRP = uEndCurrency.isXRP();
bool const bOnXRP = isXRP(uEndCurrency);

// Does pathfinding really need to get this to
// a gateway (the issuer of the destination amount)
Expand Down
Loading

0 comments on commit a4f5af2

Please sign in to comment.