update TODO with new information

sequencer/src/state.rs

@@ -631,13 +631,17 @@ impl ValidatedState {
 
         // TODO we will need something similar to `charge_fee` but
         // that charges `BidTx` to the account. We will also validate
-        // we are within bid phase time constraints. Questions: * Is
-        // there a single bid per block or multiple?  * What
-        // account(s) are we charging?
+        // against bid phase time constraints.
         //
-        // Assuming multiple bids we might have a `bids_map` might be
-        // a hashmap of accounts => `BidTx`
-        // charge_bids(&mut validated_state, bids_map);
+        // According to discussion we may receive multiple bids per
+        // header (up to configurable maximum), so this will likely be
+        // represented as `Vec<BidTx>` where `BidTx` holds account and
+        // amount.
+        //
+        // Note that different from `charge_fee` fee for the bid (sent
+        // to fee_recipient) whilch the bid itself will be held in an
+        // escrow account.
+        // charge_bids(&mut validated_state, bids_vec, fee_recipient, escrow_account);
         charge_fee(
             &mut validated_state,
             &mut delta,