Comments on why there is no hint for global deposit and withdraw (#165)

* comments on why there is no hint for global deposit and withdraw

* fmt

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 {

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 {