message: Use reference syntax in `MapViewState::transportOreFromMines`
  Previously the `const` with `auto` applied to the pointer, rather than
  the pointed to object. This was confusing. The reference syntax here
  makes more clear that the object must be mutable.
diff: @@ -303,21 +303,21 @@ void MapViewState::transportOreFromMines()
  if (routeIt != routeTable.end())
  {
  const auto& route = routeIt->second;
  - const auto smelter = static_cast<OreRefining*>(static_cast<Tile*>(route.path.back())->structure());
  - const auto mineFacility = static_cast<MineFacility*>(static_cast<Tile*>(route.path.front())->structure());
  + auto& smelter = *static_cast<OreRefining*>(static_cast<Tile*>(route.path.back())->structure());
  + auto& mineFacility = *static_cast<MineFacility*>(static_cast<Tile*>(route.path.front())->structure());
  - if (!smelter->operational()) { break; }
  + if (!smelter.operational()) { break; }
  /* clamp route cost to minimum of 1.0f for next computation to avoid
  unintended multiplication. */
  const float routeCost = std::clamp(routeIt->second.cost, 1.0f, FLT_MAX);
  /* intentional truncation of fractional component*/
  - const int totalOreMovement = static_cast<int>(constants::ShortestPathTraversalCount / routeCost) * mineFacility->assignedTrucks();
  + const int totalOreMovement = static_cast<int>(constants::ShortestPathTraversalCount / routeCost) * mineFacility.assignedTrucks();
  const int oreMovementPart = totalOreMovement / 4;
  const int oreMovementRemainder = totalOreMovement % 4;
  - auto& stored = mineFacility->storage();
  + auto& stored = mineFacility.storage();
  StorableResources moved
  {
  std::clamp(stored.resources[0], 0, oreMovementPart),
  @@ -328,7 +328,7 @@ void MapViewState::transportOreFromMines()
  stored -= moved;
  - auto& smelterProduction = smelter->production();
  + auto& smelterProduction = smelter.production();
  auto newResources = smelterProduction + moved;
  auto capped = newResources.cap(250);
  smelterProduction = capped;
  