Skip to content

Commit

Permalink
Handle resplay for feebumptransactionframe
Browse files Browse the repository at this point in the history
  • Loading branch information
ThomasBrady committed Nov 19, 2024
1 parent d6041b4 commit 42242c0
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
7 changes: 2 additions & 5 deletions src/transactions/MutableTransactionResult.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -485,21 +485,18 @@ FeeBumpMutableTransactionResult::setReplayFailingTransactionResult(
TransactionResult const& failing)
{
// no-op
// TODO maybe this should be no-op
mReplayFailingTransactionResult = std::make_optional(failing);
}

std::optional<TransactionResult> const&
FeeBumpMutableTransactionResult::getReplaySuccessfulTransactionResult() const
{
// no -op?
return mReplaySuccessfulTransactionResult;
return std::nullopt;
}

std::optional<TransactionResult> const&
FeeBumpMutableTransactionResult::getReplayFailingTransactionResult() const
{
return mReplayFailingTransactionResult;
return std::nullopt;
}

}
1 change: 0 additions & 1 deletion src/transactions/TransactionFrame.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1581,7 +1581,6 @@ TransactionFrame::applyOperations(AbstractSignatureChecker& signatureChecker,
{
// Sub-zone for skips
ZoneScopedN("skipped failed");

txResult.setResultCode(failingResult->result.code());
txResult.getResult().result.results() = failingResult->result.results();
return false;
Expand Down

0 comments on commit 42242c0

Please sign in to comment.