diff --git a/programs/manifest/src/program/processor/global_deposit.rs b/programs/manifest/src/program/processor/global_deposit.rs index 4b9de3cde..707ae95c6 100644 --- a/programs/manifest/src/program/processor/global_deposit.rs +++ b/programs/manifest/src/program/processor/global_deposit.rs @@ -16,7 +16,10 @@ use super::invoke; #[derive(BorshDeserialize, BorshSerialize)] pub struct GlobalDepositParams { pub amount_atoms: u64, - // TODO: Use trader_index_hint. Should not be very impactful, but for consistency. + // No trader index hint because global account is small so there is not much + // benefit from hinted indices, unlike the market which can get large. Also, + // seats are not permanent like on a market due to eviction, so it is more + // likely that a client could send a bad request. Just look it up for them. } impl GlobalDepositParams { diff --git a/programs/manifest/src/program/processor/global_withdraw.rs b/programs/manifest/src/program/processor/global_withdraw.rs index 270209544..c15f18132 100644 --- a/programs/manifest/src/program/processor/global_withdraw.rs +++ b/programs/manifest/src/program/processor/global_withdraw.rs @@ -17,6 +17,10 @@ use crate::{ #[derive(BorshDeserialize, BorshSerialize)] pub struct GlobalWithdrawParams { pub amount_atoms: u64, + // No trader index hint because global account is small so there is not much + // benefit from hinted indices, unlike the market which can get large. Also, + // seats are not permanent like on a market due to eviction, so it is more + // likely that a client could send a bad request. Just look it up for them. } impl GlobalWithdrawParams {