message: Initialize a new StorageResources object when recalculating
  This is mostly a stylistic choice. Rather than clearing and re-using the
  old variable, leaving it in an intermediate state during the
  calculation, it often makes sense to initialize a new value, and copy
  the result over at the end.
diff: @@ -269,12 +269,12 @@ void MapViewState::updatePlayerResources()
  */
  auto& playerResources = const_cast<StorableResources&>(mPlayerResources);
  - playerResources.clear();
  -
  + StorableResources resources;
  for (auto structure : storage)
  {
  - playerResources += structure->storage();
  + resources += structure->storage();
  }
  + playerResources = resources;
  }
  