From 695cf943966aebacc7924eed2a8cfb37173e9d53 Mon Sep 17 00:00:00 2001
From: Tal Derei <talderei99@gmail.com>
Date: Tue, 7 May 2024 13:37:30 -0700
Subject: [PATCH] fix

---
 crates/view/src/planner.rs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/crates/view/src/planner.rs b/crates/view/src/planner.rs
index b5480fe84d..8094dbee36 100644
--- a/crates/view/src/planner.rs
+++ b/crates/view/src/planner.rs
@@ -45,8 +45,7 @@ use penumbra_tct as tct;
 use penumbra_transaction::{
     memo::MemoPlaintext,
     plan::{ActionPlan, MemoPlan, TransactionPlan},
-    TransactionParameters,
-    ActionList
+    ActionList, TransactionParameters,
 };
 
 /// A planner for a [`TransactionPlan`] that can fill in the required spends and change outputs upon
@@ -473,6 +472,7 @@ impl<R: RngCore + CryptoRng> Planner<R> {
     /// capital losses when possible, using cost basis information retained by the
     /// view server.
     fn prioritize_and_filter_spendable_notes(
+        &mut self,
         records: Vec<SpendableNoteRecord>,
     ) -> Vec<SpendableNoteRecord> {
         // Filter out zero valued notes.
@@ -536,8 +536,6 @@ impl<R: RngCore + CryptoRng> Planner<R> {
                 .await?;
             notes_by_asset_id.insert(
                 required.asset_id,
-                // TODO: reorganize later, the important thing here is that this
-                // is separate from the Planner logic (will never be commonized)
                 self.prioritize_and_filter_spendable_notes(records),
             );
         }