Rather than passing the exchange context here, it would probably be better to just pass (by reference) the requests_by_commod map contained in the context.  This simplifies the api and prevents traders from doing something stupid like calling the ExchangeContext::AddBid method themselves.  This same comment applies to the GetGenRsrcBids method.