Added comments

contracts/orderbook/src/order.rs

@@ -73,10 +73,14 @@ pub fn place_limit(
     let mut response = Response::default();
     // Run order fill if criteria met
     if should_fill {
+        // Convert order to market order for filling
         let mut market_order: MarketOrder = limit_order.clone().into();
+        // Generate vector of fulfillments for current orders and a payment for the placed order
         let (fulfillments, placed_order_payment) =
             run_market_order(deps.storage, &mut market_order, Some(tick_id))?;
+        // Resolve given fulfillments and current placed order
         let fulfillment_msgs = resolve_fulfillments(deps.storage, fulfillments)?;
+        // Update limit order quantity to reflect fulfilled amount
         limit_order.quantity = market_order.quantity;
 
         response = response
@@ -299,7 +303,6 @@ pub fn resolve_fulfillments(
         ensure_eq!(
             fulfillment.order.book_id,
             orderbook.book_id,
-            // TODO: Error not expressive
             ContractError::InvalidFulfillment {
                 order_id: fulfillment.order.order_id,
                 book_id: fulfillment.order.book_id,

contracts/orderbook/src/state.rs

@@ -18,7 +18,6 @@ pub const ORDERBOOKS: Map<&u64, Orderbook> = Map::new("orderbooks");
 /// Key: (orderbook_id, tick)
 pub const TICK_LIQUIDITY: Map<&(u64, i64), Uint128> = Map::new("tick_liquidity");
 
-// TODO: Check additional gas fee for adding more indexes
 pub struct OrderIndexes {
     // Index by owner; Generic types: MultiIndex<Index Key: owner, Input Data: LimitOrder, Map Key: (orderbook_id, tick, order_id)>
     pub owner: MultiIndex<'static, Addr, LimitOrder, (u64, i64, u64)>,
@@ -87,7 +86,7 @@ pub fn reduce_tick_liquidity(
         TICK_LIQUIDITY.remove(storage, &(book_id, tick_id));
     } else {
         TICK_LIQUIDITY.save(storage, &(book_id, tick_id), &new_liquidity)?;
-    }
+    };
     Ok(())
 }