Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Commit

Permalink
Remove unnecessary unwrap from simulate_transaction_unchecked() (#3…
Browse files Browse the repository at this point in the history
…5375)

Remove unnecessary unwrap from simulate_transaction_unchecked()

(cherry picked from commit cb260f1)
  • Loading branch information
pgarg66 authored and mergify[bot] committed Mar 1, 2024
1 parent 1390ec9 commit c326ac6
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions runtime/src/bank.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4384,9 +4384,7 @@ impl Bank {
let post_simulation_accounts = loaded_transactions
.into_iter()
.next()
.unwrap()
.0
.ok()
.and_then(|(loaded_transactions_res, _)| loaded_transactions_res.ok())
.map(|loaded_transaction| {
loaded_transaction
.accounts
Expand All @@ -4406,7 +4404,12 @@ impl Bank {

debug!("simulate_transaction: {:?}", timings);

let execution_result = execution_results.pop().unwrap();
let execution_result =
execution_results
.pop()
.unwrap_or(TransactionExecutionResult::NotExecuted(
TransactionError::InvalidProgramForExecution,
));
let flattened_result = execution_result.flattened_result();
let (logs, return_data) = match execution_result {
TransactionExecutionResult::Executed { details, .. } => {
Expand Down

0 comments on commit c326ac6

Please sign in to comment.