message: Remove comment about brute force concerns
  Unless we keep a running count of available space, this needs to be
  brute force. I don't think there is much incentive to change this at
  present.
diff: @@ -411,8 +411,6 @@ void transferProductsPool(ProductPool& source, ProductPool& destination)
  bool simulateMoveProducts(Warehouse* sourceWarehouse)
  {
  ProductPool sourcePool = sourceWarehouse->products();
  -
  - /** \fixme This is a brute force approach. It works but it's not elegant. */
  const auto& structures = Utility<StructureManager>::get().structureList(Structure::StructureClass::Warehouse);
  for (auto structure : structures)
  {
  