diff options
Diffstat (limited to 'source/game')
26 files changed, 120 insertions, 153 deletions
diff --git a/source/game/StarActorMovementController.cpp b/source/game/StarActorMovementController.cpp index 00e3726..0badc7a 100644 --- a/source/game/StarActorMovementController.cpp +++ b/source/game/StarActorMovementController.cpp @@ -664,7 +664,7 @@ void ActorMovementController::controlFly(Vec2F const& velocity) { m_controlFly = velocity; } -Maybe<pair<Vec2F, bool>> ActorMovementController::pathMove(Vec2F const& position, bool run, Maybe<PlatformerAStar::Parameters> const& parameters) { +Maybe<pair<Vec2F, bool>> ActorMovementController::pathMove(Vec2F const& position, bool, Maybe<PlatformerAStar::Parameters> const& parameters) { if (!m_pathController) m_pathController = make_shared<PathController>(world()); @@ -1214,7 +1214,7 @@ Maybe<bool> PathController::findPath(ActorMovementController& movementController if (!merged) { // try to splice the new path onto the current path auto& newPathStart = path.at(0); - for (size_t i = m_edgeIndex; i < m_path->size(); ++i) { + for (size_t i = m_edgeIndex; i < m_path->size(); i += 2) { auto& edge = m_path->at(i); if (edge.target.position == newPathStart.source.position) { // splice the new path onto our current path up to this index @@ -1228,7 +1228,6 @@ Maybe<bool> PathController::findPath(ActorMovementController& movementController merged = true; break; } - i++; } } } @@ -1456,7 +1455,7 @@ bool PathController::validateEdge(ActorMovementController& movementController, P bool PathController::movingCollision(ActorMovementController& movementController, PolyF const& collisionPoly) { bool collided = false; - movementController.forEachMovingCollision(collisionPoly.boundBox(), [&](MovingCollisionId id, PhysicsMovingCollision mc, PolyF poly, RectF bounds) { + movementController.forEachMovingCollision(collisionPoly.boundBox(), [&](MovingCollisionId, PhysicsMovingCollision, PolyF poly, RectF) { if (poly.intersects(collisionPoly)) { // set collided and stop iterating collided = true; diff --git a/source/game/StarDrawable.cpp b/source/game/StarDrawable.cpp index 2c527a5..a3826e0 100644 --- a/source/game/StarDrawable.cpp +++ b/source/game/StarDrawable.cpp @@ -70,7 +70,7 @@ Drawable::ImagePart& Drawable::ImagePart::removeDirectives(bool keepImageCenterP Drawable Drawable::makeLine(Line2F const& line, float lineWidth, Color const& color, Vec2F const& position) { Drawable drawable; - drawable.part = LinePart{std::move(line), lineWidth}; + drawable.part = LinePart{std::move(line), lineWidth, {}}; drawable.color = color; drawable.position = position; @@ -110,7 +110,7 @@ Drawable::Drawable() Drawable::Drawable(Json const& json) { if (auto line = json.opt("line")) { - part = LinePart{jsonToLine2F(*line), json.getFloat("width")}; + part = LinePart{jsonToLine2F(*line), json.getFloat("width"), {}}; } else if (auto poly = json.opt("poly")) { part = PolyPart{jsonToPolyF(*poly)}; } else if (auto image = json.opt("image")) { diff --git a/source/game/StarHumanoid.cpp b/source/game/StarHumanoid.cpp index 78b615b..c69872b 100644 --- a/source/game/StarHumanoid.cpp +++ b/source/game/StarHumanoid.cpp @@ -935,7 +935,7 @@ List<Drawable> Humanoid::renderSkull() const { Root::singleton().speciesDatabase()->species(m_identity.species)->skull(), 1.0f, true, Vec2F())}; } -Humanoid Humanoid::makeDummy(Gender gender) { +Humanoid Humanoid::makeDummy(Gender) { auto assets = Root::singleton().assets(); Humanoid humanoid(assets->json("/humanoid.config")); diff --git a/source/game/StarImageMetadataDatabase.cpp b/source/game/StarImageMetadataDatabase.cpp index 47ff97f..b3a3983 100644 --- a/source/game/StarImageMetadataDatabase.cpp +++ b/source/game/StarImageMetadataDatabase.cpp @@ -238,7 +238,7 @@ Vec2U ImageMetadataDatabase::calculateImageSize(AssetPath const& path) const { for (auto& directives : path.directives.list()) directives.loadOperations(); - bool complete = path.directives.forEachAbortable([&](auto const& entry, Directives const& directives) -> bool { + bool complete = path.directives.forEachAbortable([&](auto const& entry, Directives const&) -> bool { entry.operation.call(osa); return !osa.hasError; }); diff --git a/source/game/StarMovementController.cpp b/source/game/StarMovementController.cpp index 88ac69f..a06ead5 100644 --- a/source/game/StarMovementController.cpp +++ b/source/game/StarMovementController.cpp @@ -624,9 +624,7 @@ void MovementController::tickMaster(float dt) { Vec2F newVelocity = relativeVelocity + m_surfaceVelocity; - Vec2F pos = position(); PolyF body = collisionBody(); - RectF boundBox = body.boundBox(); updateLiquidPercentage(); @@ -736,7 +734,7 @@ void MovementController::forEachMovingCollision(RectF const& region, function<bo } } -void MovementController::updateForceRegions(float dt) { +void MovementController::updateForceRegions(float) { auto geometry = world()->geometry(); auto pos = position(); auto body = collisionBody(); @@ -804,7 +802,6 @@ void MovementController::updateForceRegions(float dt) { } void MovementController::updateLiquidPercentage() { - auto geometry = world()->geometry(); auto pos = position(); auto body = collisionBody(); RectF boundBox = body.boundBox(); diff --git a/source/game/StarPlayer.cpp b/source/game/StarPlayer.cpp index ebadf20..adf3334 100644 --- a/source/game/StarPlayer.cpp +++ b/source/game/StarPlayer.cpp @@ -2186,7 +2186,7 @@ Maybe<String> Player::inspectionLogName() const { return identifier; } -Maybe<String> Player::inspectionDescription(String const& species) const { +Maybe<String> Player::inspectionDescription(String const&) const { return m_description; } diff --git a/source/game/StarPlayerInventory.cpp b/source/game/StarPlayerInventory.cpp index d7ba6a1..b6c4af3 100644 --- a/source/game/StarPlayerInventory.cpp +++ b/source/game/StarPlayerInventory.cpp @@ -444,7 +444,7 @@ void PlayerInventory::condenseBagStacks(String const& bagType) {\ bag->condenseStacks(); - m_customBar.forEach([&](auto const& index, CustomBarLink& link) { + m_customBar.forEach([&](auto const&, CustomBarLink& link) { if (link.first) { if (auto bs = link.first->ptr<BagSlot>()) { if (bs->first == bagType && !bag->at(bs->second)) diff --git a/source/game/StarStatCollection.cpp b/source/game/StarStatCollection.cpp index 6ded538..a12256f 100644 --- a/source/game/StarStatCollection.cpp +++ b/source/game/StarStatCollection.cpp @@ -157,7 +157,7 @@ void StatCollection::tickMaster(float dt) { m_stats.update(dt); } -void StatCollection::tickSlave(float dt) { +void StatCollection::tickSlave(float) { m_stats.update(0.0f); } diff --git a/source/game/StarSystemWorldServerThread.cpp b/source/game/StarSystemWorldServerThread.cpp index 3ec26a7..281ae3e 100644 --- a/source/game/StarSystemWorldServerThread.cpp +++ b/source/game/StarSystemWorldServerThread.cpp @@ -7,7 +7,7 @@ namespace Star { SystemWorldServerThread::SystemWorldServerThread(Vec3I const& location, SystemWorldServerPtr systemWorld, String storageFile) : Thread(strf("SystemWorldServer: {}", location)) , m_systemLocation(location) - , m_systemWorld(move(systemWorld)) + , m_systemWorld(std::move(systemWorld)) , m_storageFile(storageFile) { } diff --git a/source/game/StarToolUser.cpp b/source/game/StarToolUser.cpp index c9bd068..4951128 100644 --- a/source/game/StarToolUser.cpp +++ b/source/game/StarToolUser.cpp @@ -524,7 +524,7 @@ List<PhysicsForceRegion> ToolUser::forceRegions() const { return ds; } -void ToolUser::render(RenderCallback* renderCallback, bool inToolRange, bool shifting, EntityRenderLayer renderLayer) { +void ToolUser::render(RenderCallback* renderCallback, bool, bool shifting, EntityRenderLayer renderLayer) { if (m_suppress.get()) { for (auto item : {m_primaryHandItem.get(), m_altHandItem.get()}) { if (auto activeItem = as<ActiveItem>(item)) { diff --git a/source/game/StarUniverseClient.cpp b/source/game/StarUniverseClient.cpp index 6f089d4..72dd6a0 100644 --- a/source/game/StarUniverseClient.cpp +++ b/source/game/StarUniverseClient.cpp @@ -487,7 +487,7 @@ void UniverseClient::stopLua() { m_scriptContexts.clear(); } -bool UniverseClient::reloadPlayer(Json const& data, Uuid const& uuid, bool resetInterfaces, bool showIndicator) { +bool UniverseClient::reloadPlayer(Json const& data, Uuid const&, bool resetInterfaces, bool showIndicator) { auto player = mainPlayer(); bool playerInWorld = player->inWorld(); auto world = as<WorldClient>(player->world()); diff --git a/source/game/StarWorldClient.cpp b/source/game/StarWorldClient.cpp index b4244b0..ca99165 100644 --- a/source/game/StarWorldClient.cpp +++ b/source/game/StarWorldClient.cpp @@ -1369,7 +1369,6 @@ void WorldClient::collectLiquid(List<Vec2I> const& tilePositions, LiquidId liqui return; float bucketSize = Root::singleton().assets()->json("/items/defaultParameters.config:liquidItems.bucketSize").toFloat(); - unsigned drainedUnits = 0; float nextUnit = bucketSize; List<Vec2I> maybeDrainTiles; @@ -1386,7 +1385,7 @@ void WorldClient::collectLiquid(List<Vec2I> const& tilePositions, LiquidId liqui liquid.take(nextUnit); nextUnit = bucketSize; - for (auto& previousTile : maybeDrainTiles) + for (size_t i = 0; i < maybeDrainTiles.size(); ++i) m_predictedTiles[pos].liquid.emplace(EmptyLiquidId, 0.0f); maybeDrainTiles.clear(); diff --git a/source/game/StarWorldImpl.hpp b/source/game/StarWorldImpl.hpp index 1a4a509..828ed1e 100644 --- a/source/game/StarWorldImpl.hpp +++ b/source/game/StarWorldImpl.hpp @@ -92,7 +92,7 @@ namespace WorldImpl { } template <typename TileSectorArray> - CollisionKind tileCollisionKind(shared_ptr<TileSectorArray> const& tileSectorArray, EntityMapPtr const& entityMap, + CollisionKind tileCollisionKind(shared_ptr<TileSectorArray> const& tileSectorArray, EntityMapPtr const&, Vec2I const& pos) { return tileSectorArray->tile(pos).collision; } @@ -226,7 +226,7 @@ namespace WorldImpl { if (!isRealMaterial(material)) return false; - auto isAdjacentToConnectable = [&](Vec2I const& pos, unsigned distance, bool foreground) { + auto isAdjacentToConnectable = [&](Vec2I const& pos, int distance, bool foreground) { if (pos.y() - distance < 0) return true; @@ -237,7 +237,7 @@ namespace WorldImpl { for (int x = pos.x() - distance; x != maxX; ++x) { tPos[0] = x; if (tPos != pos) { - auto& tile = getTile(tPos); + const auto& tile = getTile(tPos); if (isConnectableMaterial(foreground ? tile.foreground : tile.background)) return true; } @@ -286,7 +286,7 @@ namespace WorldImpl { if (!materialDatabase->canPlaceInLayer(material, layer)) return false; - auto& tile = getTile(pos); + const auto& tile = getTile(pos); if (layer == TileLayer::Background) { if (tile.background != EmptyMaterialId && tile.background != ObjectPlatformMaterialId) return false; diff --git a/source/game/StarWorldLayout.cpp b/source/game/StarWorldLayout.cpp index 58eca08..fd73190 100644 --- a/source/game/StarWorldLayout.cpp +++ b/source/game/StarWorldLayout.cpp @@ -629,7 +629,7 @@ void WorldLayout::expandBiomeRegion(Vec2I const& position, int newWidth) { pair<size_t, size_t> WorldLayout::findLayerAndCell(int x, int y) const { // find the target layer - size_t targetLayerIndex; + size_t targetLayerIndex{}; for (size_t i = 0; i < m_layers.size(); ++i) { if (m_layers[i].yStart < y) targetLayerIndex = i; diff --git a/source/game/StarWorldParameters.cpp b/source/game/StarWorldParameters.cpp index 97f0611..01d497a 100644 --- a/source/game/StarWorldParameters.cpp +++ b/source/game/StarWorldParameters.cpp @@ -25,19 +25,6 @@ EnumMap<WorldEdgeForceRegionType> const WorldEdgeForceRegionTypeNames{ {WorldEdgeForceRegionType::Bottom, "Bottom"}, {WorldEdgeForceRegionType::TopAndBottom, "TopAndBottom"}}; -VisitableWorldParameters::VisitableWorldParameters() { - threatLevel = 0; - gravity = 0; - airless = false; - disableDeathDrops = false; - terraformed = false; - worldEdgeForceRegions = WorldEdgeForceRegionType::None; -} - -VisitableWorldParameters::VisitableWorldParameters(VisitableWorldParameters const& visitableWorldParameters) { - *this = visitableWorldParameters; -} - VisitableWorldParameters::VisitableWorldParameters(Json const& store) { typeName = store.getString("typeName", ""); threatLevel = store.getFloat("threatLevel"); @@ -54,8 +41,6 @@ VisitableWorldParameters::VisitableWorldParameters(Json const& store) { worldEdgeForceRegions = WorldEdgeForceRegionTypeNames.getLeft(store.getString("worldEdgeForceRegions", "None")); } -VisitableWorldParameters::~VisitableWorldParameters() {} - Json VisitableWorldParameters::store() const { return JsonObject{{"typeName", typeName}, {"threatLevel", threatLevel}, @@ -104,15 +89,6 @@ void VisitableWorldParameters::write(DataStream& ds) const { ds << worldEdgeForceRegions; } -TerrestrialWorldParameters::TerrestrialWorldParameters() { - blendSize = 0; - dayLength = 0; -} - -TerrestrialWorldParameters::TerrestrialWorldParameters(TerrestrialWorldParameters const& terrestrialWorldParameters) { - *this = terrestrialWorldParameters; -} - TerrestrialWorldParameters::TerrestrialWorldParameters(Json const& store) : VisitableWorldParameters(store) { auto loadTerrestrialRegion = [](Json const& config) { return TerrestrialRegion{config.getString("biome"), @@ -122,19 +98,19 @@ TerrestrialWorldParameters::TerrestrialWorldParameters(Json const& store) : Visi config.getString("fgOreSelector"), config.getString("bgOreSelector"), config.getString("subBlockSelector"), - (LiquidId)config.getUInt("caveLiquid"), + static_cast<LiquidId>(config.getUInt("caveLiquid")), config.getFloat("caveLiquidSeedDensity"), - (LiquidId)config.getUInt("oceanLiquid"), - (int)config.getInt("oceanLiquidLevel"), - (bool)config.getBool("encloseLiquids"), - (bool)config.getBool("fillMicrodungeons")}; + static_cast<LiquidId>(config.getUInt("oceanLiquid")), + static_cast<int>(config.getInt("oceanLiquidLevel")), + config.getBool("encloseLiquids"), + config.getBool("fillMicrodungeons")}; }; auto loadTerrestrialLayer = [loadTerrestrialRegion](Json const& config) { - return TerrestrialLayer{(int)config.getInt("layerMinHeight"), - (int)config.getInt("layerBaseHeight"), + return TerrestrialLayer{static_cast<int>(config.getInt("layerMinHeight")), + static_cast<int>(config.getInt("layerBaseHeight")), jsonToStringList(config.get("dungeons")), - (int)config.getInt("dungeonXVariance"), + static_cast<int>(config.getInt("dungeonXVariance")), loadTerrestrialRegion(config.get("primaryRegion")), loadTerrestrialRegion(config.get("primarySubRegion")), config.getArray("secondaryRegions").transformed(loadTerrestrialRegion), @@ -333,14 +309,11 @@ void TerrestrialWorldParameters::write(DataStream& ds) const { AsteroidsWorldParameters::AsteroidsWorldParameters() { airless = true; - asteroidTopLevel = 0; - asteroidBottomLevel = 0; - blendSize = 0; } AsteroidsWorldParameters::AsteroidsWorldParameters(Json const& store) : VisitableWorldParameters(store) { - asteroidTopLevel = store.getInt("asteroidTopLevel"); - asteroidBottomLevel = store.getInt("asteroidBottomLevel"); + asteroidTopLevel = static_cast<int>(store.getInt("asteroidTopLevel")); + asteroidBottomLevel = static_cast<int>(store.getInt("asteroidBottomLevel")); blendSize = store.getFloat("blendSize"); asteroidBiome = store.getString("asteroidBiome"); ambientLightLevel = jsonToColor(store.get("ambientLightLevel")); @@ -376,12 +349,10 @@ void AsteroidsWorldParameters::write(DataStream& ds) const { ds << ambientLightLevel; } -FloatingDungeonWorldParameters::FloatingDungeonWorldParameters() {} - FloatingDungeonWorldParameters::FloatingDungeonWorldParameters(Json const& store) : VisitableWorldParameters(store) { - dungeonBaseHeight = store.getInt("dungeonBaseHeight"); - dungeonSurfaceHeight = store.getInt("dungeonSurfaceHeight"); - dungeonUndergroundLevel = store.getInt("dungeonUndergroundLevel"); + dungeonBaseHeight = static_cast<int>(store.getInt("dungeonBaseHeight")); + dungeonSurfaceHeight = static_cast<int>(store.getInt("dungeonSurfaceHeight")); + dungeonUndergroundLevel = static_cast<int>(store.getInt("dungeonUndergroundLevel")); primaryDungeon = store.getString("primaryDungeon"); biome = store.optString("biome"); ambientLightLevel = jsonToColor(store.get("ambientLightLevel")); @@ -438,7 +409,7 @@ void FloatingDungeonWorldParameters::write(DataStream& ds) const { Json diskStoreVisitableWorldParameters(VisitableWorldParametersConstPtr const& parameters) { if (!parameters) - return Json(); + return {}; return parameters->store().setAll({{"type", WorldParametersTypeNames.getRight(parameters->type())}}); } @@ -459,7 +430,7 @@ VisitableWorldParametersPtr diskLoadVisitableWorldParameters(Json const& store) ByteArray netStoreVisitableWorldParameters(VisitableWorldParametersConstPtr const& parameters) { if (!parameters) - return ByteArray(); + return {}; DataStreamBuffer ds; ds.write(parameters->type()); @@ -509,7 +480,7 @@ TerrestrialWorldParametersPtr generateTerrestrialWorldParameters(String const& t auto dayLengthRange = jsonToVec2F(config.get("dayLengthRange")); auto threatLevelRange = jsonToVec2F(config.query("threatRange")); - float threatLevel = staticRandomDouble(seed, "ThreatLevel") * (threatLevelRange[1] - threatLevelRange[0]) + threatLevelRange[0]; + auto threatLevel = static_cast<float>(staticRandomDouble(seed, "ThreatLevel") * (threatLevelRange[1] - threatLevelRange[0]) + threatLevelRange[0]); auto surfaceBiomeSeed = staticRandomU64(seed, "SurfaceBiomeSeed"); auto readRegion = [liquidsDatabase, threatLevel, seed](Json const& regionConfig, String const& layerName, int layerBaseHeight) { @@ -539,7 +510,7 @@ TerrestrialWorldParametersPtr generateTerrestrialWorldParameters(String const& t if (auto oceanLiquid = staticRandomValueFrom(regionConfig.getArray("oceanLiquid", {}), seed, "oceanLiquid", layerName.utf8Ptr()).optString()) { region.oceanLiquid = liquidsDatabase->liquidId(*oceanLiquid); - region.oceanLiquidLevel = regionConfig.getInt("oceanLevelOffset", 0) + layerBaseHeight; + region.oceanLiquidLevel = static_cast<int>(regionConfig.getInt("oceanLevelOffset", 0) + layerBaseHeight); } else { region.oceanLiquid = EmptyLiquidId; region.oceanLiquidLevel = 0; @@ -561,30 +532,32 @@ TerrestrialWorldParametersPtr generateTerrestrialWorldParameters(String const& t TerrestrialWorldParameters::TerrestrialLayer layer; - layer.layerMinHeight = layerConfig.getFloat("layerLevel"); - layer.layerBaseHeight = layerConfig.getFloat("baseHeight"); + layer.layerMinHeight = static_cast<int>(layerConfig.getFloat("layerLevel")); + layer.layerBaseHeight = static_cast<int>(layerConfig.getFloat("baseHeight")); auto primaryRegionList = layerConfig.getArray("primaryRegion"); auto primaryRegionConfigName = staticRandomFrom(primaryRegionList, seed, layerName.utf8Ptr(), "PrimaryRegionSelection").toString(); Json primaryRegionConfig = jsonMerge(regionDefaults, regionTypes.get(primaryRegionConfigName)); layer.primaryRegion = readRegion(primaryRegionConfig, layerName, layer.layerBaseHeight); - auto subRegionList = primaryRegionConfig.getArray("subRegion"); - Json subRegionConfig; - if (subRegionList.size() > 0) { - String subRegionName = staticRandomFrom(subRegionList, seed, layerName, primaryRegionConfigName).toString(); - subRegionConfig = jsonMerge(regionDefaults, regionTypes.get(subRegionName)); - } else { - subRegionConfig = primaryRegionConfig; + { + auto subRegionList = primaryRegionConfig.getArray("subRegion"); + Json subRegionConfig; + if (!subRegionList.empty()) { + String subRegionName = staticRandomFrom(subRegionList, seed, layerName, primaryRegionConfigName).toString(); + subRegionConfig = jsonMerge(regionDefaults, regionTypes.get(subRegionName)); + } else { + subRegionConfig = primaryRegionConfig; + } + layer.primarySubRegion = readRegion(subRegionConfig, layerName, layer.layerBaseHeight); } - layer.primarySubRegion = readRegion(subRegionConfig, layerName, layer.layerBaseHeight); Vec2U secondaryRegionCountRange = jsonToVec2U(layerConfig.get("secondaryRegionCount")); - int secondaryRegionCount = staticRandomI32Range(secondaryRegionCountRange[0], secondaryRegionCountRange[1], seed, layerName, "SecondaryRegionCount"); + int secondaryRegionCount = staticRandomI32Range(static_cast<int>(secondaryRegionCountRange[0]), static_cast<int>(secondaryRegionCountRange[1]), seed, layerName, "SecondaryRegionCount"); auto secondaryRegionList = layerConfig.getArray("secondaryRegions"); - if (secondaryRegionList.size() > 0) { + if (!secondaryRegionList.empty()) { staticRandomShuffle(secondaryRegionList, seed, layerName, "SecondaryRegionShuffle"); - for (auto regionName : secondaryRegionList) { + for (const auto& regionName : secondaryRegionList) { if (secondaryRegionCount <= 0) break; Json secondaryRegionConfig = jsonMerge(regionDefaults, regionTypes.get(regionName.toString())); @@ -592,7 +565,7 @@ TerrestrialWorldParametersPtr generateTerrestrialWorldParameters(String const& t auto subRegionList = secondaryRegionConfig.getArray("subRegion"); Json subRegionConfig; - if (subRegionList.size() > 0) { + if (!subRegionList.empty()) { String subRegionName = staticRandomFrom(subRegionList, seed, layerName, regionName.toString()).toString(); subRegionConfig = jsonMerge(regionDefaults, regionTypes.get(subRegionName)); } else { @@ -611,7 +584,7 @@ TerrestrialWorldParametersPtr generateTerrestrialWorldParameters(String const& t Vec2U dungeonCountRange = layerConfig.opt("dungeonCountRange").apply(jsonToVec2U).value(); unsigned dungeonCount = staticRandomU32Range(dungeonCountRange[0], dungeonCountRange[1], seed, layerName, "DungeonCount"); layer.dungeons.appendAll(dungeonPool.selectUniques(dungeonCount, staticRandomHash64(seed, layerName, "DungeonChoice"))); - layer.dungeonXVariance = layerConfig.getInt("dungeonXVariance", 0); + layer.dungeonXVariance = static_cast<int>(layerConfig.getInt("dungeonXVariance", 0)); return layer; }; @@ -674,7 +647,7 @@ AsteroidsWorldParametersPtr generateAsteroidsWorldParameters(uint64_t seed) { auto gravityRange = jsonToVec2F(asteroidsConfig.get("gravityRange")); auto threatLevelRange = jsonToVec2F(asteroidsConfig.get("threatRange")); - parameters->threatLevel = staticRandomDouble(seed, "ThreatLevel") * (threatLevelRange[1] - threatLevelRange[0]) + threatLevelRange[0]; + parameters->threatLevel = static_cast<float>(staticRandomDouble(seed, "ThreatLevel") * (threatLevelRange[1] - threatLevelRange[0]) + threatLevelRange[0]); parameters->typeName = "asteroids"; parameters->worldSize = jsonToVec2U(asteroidsConfig.get("worldSize")); parameters->gravity = staticRandomFloatRange(gravityRange[0], gravityRange[1], seed, "WorldGravity"); @@ -685,8 +658,8 @@ AsteroidsWorldParametersPtr generateAsteroidsWorldParameters(uint64_t seed) { parameters->disableDeathDrops = asteroidsConfig.getBool("disableDeathDrops", false); parameters->worldEdgeForceRegions = WorldEdgeForceRegionTypeNames.getLeft(asteroidsConfig.getString("worldEdgeForceRegions", "TopAndBottom")); - parameters->asteroidTopLevel = asteroidsConfig.getInt("asteroidsTop"); - parameters->asteroidBottomLevel = asteroidsConfig.getInt("asteroidsBottom"); + parameters->asteroidTopLevel = static_cast<int>(asteroidsConfig.getInt("asteroidsTop")); + parameters->asteroidBottomLevel = static_cast<int>(asteroidsConfig.getInt("asteroidsBottom")); parameters->blendSize = asteroidsConfig.getFloat("blendSize"); parameters->asteroidBiome = biome; parameters->ambientLightLevel = jsonToColor(asteroidsConfig.get("ambientLightLevel")); @@ -716,9 +689,9 @@ FloatingDungeonWorldParametersPtr generateFloatingDungeonWorldParameters(String parameters->disableDeathDrops = worldConfig.getBool("disableDeathDrops", false); parameters->worldEdgeForceRegions = WorldEdgeForceRegionTypeNames.getLeft(worldConfig.getString("worldEdgeForceRegions", "Top")); - parameters->dungeonBaseHeight = worldConfig.getInt("dungeonBaseHeight"); - parameters->dungeonSurfaceHeight = worldConfig.getInt("dungeonSurfaceHeight", parameters->dungeonBaseHeight); - parameters->dungeonUndergroundLevel = worldConfig.getInt("dungeonUndergroundLevel", 0); + parameters->dungeonBaseHeight = static_cast<int>(worldConfig.getInt("dungeonBaseHeight")); + parameters->dungeonSurfaceHeight = static_cast<int>(worldConfig.getInt("dungeonSurfaceHeight", parameters->dungeonBaseHeight)); + parameters->dungeonUndergroundLevel = static_cast<int>(worldConfig.getInt("dungeonUndergroundLevel", 0)); parameters->primaryDungeon = worldConfig.getString("primaryDungeon"); parameters->biome = worldConfig.optString("biome"); parameters->ambientLightLevel = jsonToColor(worldConfig.get("ambientLightLevel")); diff --git a/source/game/StarWorldParameters.hpp b/source/game/StarWorldParameters.hpp index 0e54929..386e98e 100644 --- a/source/game/StarWorldParameters.hpp +++ b/source/game/StarWorldParameters.hpp @@ -36,11 +36,11 @@ STAR_STRUCT(AsteroidsWorldParameters); STAR_STRUCT(FloatingDungeonWorldParameters); struct VisitableWorldParameters { - VisitableWorldParameters(); - VisitableWorldParameters(VisitableWorldParameters const& visitableWorldParameters); - VisitableWorldParameters(Json const& store); + VisitableWorldParameters() = default; + VisitableWorldParameters(VisitableWorldParameters const& visitableWorldParameters) = default; + explicit VisitableWorldParameters(Json const& store); - virtual ~VisitableWorldParameters(); + virtual ~VisitableWorldParameters() = default; virtual WorldParametersType type() const = 0; @@ -50,18 +50,18 @@ struct VisitableWorldParameters { virtual void write(DataStream& ds) const; String typeName; - float threatLevel; + float threatLevel{}; Vec2U worldSize; - float gravity; - bool airless; + float gravity{}; + bool airless{false}; WeatherPool weatherPool; StringList environmentStatusEffects; Maybe<StringList> overrideTech; Maybe<List<Directives>> globalDirectives; BeamUpRule beamUpRule; - bool disableDeathDrops; - bool terraformed; - WorldEdgeForceRegionType worldEdgeForceRegions; + bool disableDeathDrops{false}; + bool terraformed{false}; + WorldEdgeForceRegionType worldEdgeForceRegions{WorldEdgeForceRegionType::None}; }; struct TerrestrialWorldParameters : VisitableWorldParameters { @@ -75,14 +75,14 @@ struct TerrestrialWorldParameters : VisitableWorldParameters { String bgOreSelector; String subBlockSelector; - LiquidId caveLiquid; - float caveLiquidSeedDensity; + LiquidId caveLiquid{}; + float caveLiquidSeedDensity{}; - LiquidId oceanLiquid; - int oceanLiquidLevel; + LiquidId oceanLiquid{}; + int oceanLiquidLevel{}; - bool encloseLiquids; - bool fillMicrodungeons; + bool encloseLiquids{false}; + bool fillMicrodungeons{false}; }; struct TerrestrialLayer { @@ -102,67 +102,67 @@ struct TerrestrialWorldParameters : VisitableWorldParameters { Vec2F subRegionSizeRange; }; - TerrestrialWorldParameters(); - TerrestrialWorldParameters(TerrestrialWorldParameters const& terrestrialWorldParameters); - TerrestrialWorldParameters(Json const& store); + TerrestrialWorldParameters() = default; + TerrestrialWorldParameters(TerrestrialWorldParameters const& terrestrialWorldParameters) = default; + explicit TerrestrialWorldParameters(Json const& store); TerrestrialWorldParameters &operator=(TerrestrialWorldParameters const& terrestrialWorldParameters); - virtual WorldParametersType type() const override; + WorldParametersType type() const override; - virtual Json store() const override; + Json store() const override; - virtual void read(DataStream& ds) override; - virtual void write(DataStream& ds) const override; + void read(DataStream& ds) override; + void write(DataStream& ds) const override; String primaryBiome; - LiquidId primarySurfaceLiquid; + LiquidId primarySurfaceLiquid{}; String sizeName; - float hueShift; + float hueShift{}; SkyColoring skyColoring; - float dayLength; + float dayLength{}; Json blockNoiseConfig; Json blendNoiseConfig; - float blendSize; + float blendSize{}; - TerrestrialLayer spaceLayer; - TerrestrialLayer atmosphereLayer; - TerrestrialLayer surfaceLayer; - TerrestrialLayer subsurfaceLayer; + TerrestrialLayer spaceLayer{}; + TerrestrialLayer atmosphereLayer{}; + TerrestrialLayer surfaceLayer{}; + TerrestrialLayer subsurfaceLayer{}; List<TerrestrialLayer> undergroundLayers; - TerrestrialLayer coreLayer; + TerrestrialLayer coreLayer{}; }; struct AsteroidsWorldParameters : VisitableWorldParameters { AsteroidsWorldParameters(); - AsteroidsWorldParameters(Json const& store); + explicit AsteroidsWorldParameters(Json const& store); - virtual WorldParametersType type() const override; + WorldParametersType type() const override; - virtual Json store() const override; + Json store() const override; - virtual void read(DataStream& ds) override; - virtual void write(DataStream& ds) const override; + void read(DataStream& ds) override; + void write(DataStream& ds) const override; - int asteroidTopLevel; - int asteroidBottomLevel; - float blendSize; + int asteroidTopLevel{}; + int asteroidBottomLevel{}; + float blendSize{}; String asteroidBiome; Color ambientLightLevel; }; struct FloatingDungeonWorldParameters : VisitableWorldParameters { - FloatingDungeonWorldParameters(); - FloatingDungeonWorldParameters(Json const& store); + FloatingDungeonWorldParameters() = default; + explicit FloatingDungeonWorldParameters(Json const& store); - virtual WorldParametersType type() const override; + WorldParametersType type() const override; - virtual Json store() const override; + Json store() const override; - virtual void read(DataStream& ds) override; - virtual void write(DataStream& ds) const override; + void read(DataStream& ds) override; + void write(DataStream& ds) const override; int dungeonBaseHeight; int dungeonSurfaceHeight; diff --git a/source/game/StarWorldServer.cpp b/source/game/StarWorldServer.cpp index f8acc18..7837221 100644 --- a/source/game/StarWorldServer.cpp +++ b/source/game/StarWorldServer.cpp @@ -535,7 +535,8 @@ List<PacketPtr> WorldServer::getOutgoingPackets(ConnectionId clientId) { Maybe<Json> WorldServer::receiveMessage(ConnectionId fromConnection, String const& message, JsonArray const& args) { Maybe<Json> result; for (auto& p : m_scriptContexts) { - if (result = p.second->handleMessage(message, fromConnection == ServerConnectionId, args)) + result = p.second->handleMessage(message, fromConnection == ServerConnectionId, args); + if (result) break; } return result; diff --git a/source/game/StarWorldStorage.cpp b/source/game/StarWorldStorage.cpp index 08cf55c..7cd890a 100644 --- a/source/game/StarWorldStorage.cpp +++ b/source/game/StarWorldStorage.cpp @@ -256,7 +256,6 @@ void WorldStorage::generateQueue(Maybe<size_t> sectorGenerationLevelLimit, funct }); } - unsigned total = 0; while (!m_generationQueue.empty()) { if (sectorGenerationLevelLimit && *sectorGenerationLevelLimit == 0) break; @@ -266,7 +265,6 @@ void WorldStorage::generateQueue(Maybe<size_t> sectorGenerationLevelLimit, funct m_generationQueue.removeFirst(); if (sectorGenerationLevelLimit) *sectorGenerationLevelLimit -= p.second; - total += p.second; } } catch (std::exception const& e) { m_db.rollback(); diff --git a/source/game/interfaces/StarEntity.cpp b/source/game/interfaces/StarEntity.cpp index 801a0d8..cbc968f 100644 --- a/source/game/interfaces/StarEntity.cpp +++ b/source/game/interfaces/StarEntity.cpp @@ -111,7 +111,7 @@ Maybe<Json> Entity::receiveMessage(ConnectionId, String const&, JsonArray const& return {}; } -void Entity::update(float dt, uint64_t) {} +void Entity::update(float, uint64_t) {} void Entity::render(RenderCallback*) {} diff --git a/source/game/interfaces/StarToolUserItem.cpp b/source/game/interfaces/StarToolUserItem.cpp index 104b513..8191cec 100644 --- a/source/game/interfaces/StarToolUserItem.cpp +++ b/source/game/interfaces/StarToolUserItem.cpp @@ -14,7 +14,7 @@ void ToolUserItem::uninit() { m_hand = {}; } -void ToolUserItem::update(float dt, FireMode, bool, HashSet<MoveControlType> const&) {} +void ToolUserItem::update(float, FireMode, bool, HashSet<MoveControlType> const&) {} bool ToolUserItem::initialized() const { return (bool)m_owner; diff --git a/source/game/items/StarArmors.cpp b/source/game/items/StarArmors.cpp index 85580d9..01ff4c4 100644 --- a/source/game/items/StarArmors.cpp +++ b/source/game/items/StarArmors.cpp @@ -43,11 +43,11 @@ List<Drawable> ArmorItem::drawables() const { return drawables; } -float ArmorItem::getAngle(float aimAngle) { +float ArmorItem::getAngle(float) { return -25.0f * Constants::deg2rad; } -void ArmorItem::fire(FireMode mode, bool shifting, bool edgeTriggered) {} +void ArmorItem::fire(FireMode, bool, bool) {} void ArmorItem::fireTriggered() {} List<String> const& ArmorItem::colorOptions() { diff --git a/source/game/items/StarInspectionTool.cpp b/source/game/items/StarInspectionTool.cpp index a9b12c4..6ab6789 100644 --- a/source/game/items/StarInspectionTool.cpp +++ b/source/game/items/StarInspectionTool.cpp @@ -31,7 +31,7 @@ ItemPtr InspectionTool::clone() const { return make_shared<InspectionTool>(*this); } -void InspectionTool::update(float dt, FireMode fireMode, bool, HashSet<MoveControlType> const&) { +void InspectionTool::update(float, FireMode fireMode, bool, HashSet<MoveControlType> const&) { m_currentAngle = world()->geometry().diff(owner()->aimPosition(), owner()->position()).angle(); m_currentPosition = owner()->position() + owner()->handPosition(hand(), m_lightPosition - m_handPosition); SpatialLogger::logPoint("world", m_currentPosition, {0, 0, 255, 255}); diff --git a/source/game/items/StarInstrumentItem.cpp b/source/game/items/StarInstrumentItem.cpp index f93bca7..1a31934 100644 --- a/source/game/items/StarInstrumentItem.cpp +++ b/source/game/items/StarInstrumentItem.cpp @@ -42,7 +42,7 @@ StringSet InstrumentItem::effectSources() const { return m_inactiveEffectSources; } -void InstrumentItem::update(float dt, FireMode, bool, HashSet<MoveControlType> const&) { +void InstrumentItem::update(float, FireMode, bool, HashSet<MoveControlType> const&) { if (entityMode() == EntityMode::Master) { if (active()) { m_activeCooldown--; diff --git a/source/game/items/StarMaterialItem.cpp b/source/game/items/StarMaterialItem.cpp index c533add..42f90fc 100644 --- a/source/game/items/StarMaterialItem.cpp +++ b/source/game/items/StarMaterialItem.cpp @@ -119,7 +119,7 @@ void MaterialItem::update(float dt, FireMode fireMode, bool shifting, HashSet<Mo } } -void MaterialItem::render(RenderCallback* renderCallback, EntityRenderLayer renderLayer) { +void MaterialItem::render(RenderCallback* renderCallback, EntityRenderLayer) { if (m_collisionOverride != TileCollisionOverride::None) { float pulseLevel = 1.f - 0.3f * 0.5f * ((float)sin(2 * Constants::pi * 4.0 * Time::monotonicTime()) + 1.f); Color color = Color::rgba(owner()->favoriteColor()).mix(Color::White); @@ -190,8 +190,8 @@ void MaterialItem::fire(FireMode mode, bool shifting, bool edgeTriggered) { : Root::singleton().materialDatabase()->materialCollisionKind(m_material); size_t total = 0; - for (int i = 0; i != steps; ++i) { - auto placementOrigin = aimPosition + diff * (1.0f - ((float)i / steps)); + for (unsigned i = 0; i != steps; ++i) { + auto placementOrigin = aimPosition + diff * (1.0f - (static_cast<float>(i) / steps)); if (!owner()->inToolRange(placementOrigin)) continue; @@ -218,7 +218,7 @@ void MaterialItem::fire(FireMode mode, bool shifting, bool edgeTriggered) { m_lastAimPosition = aimPosition; } -void MaterialItem::endFire(FireMode mode, bool shifting) { +void MaterialItem::endFire(FireMode, bool) { m_lastAimPosition.reset(); } diff --git a/source/game/items/StarUnlockItem.cpp b/source/game/items/StarUnlockItem.cpp index ee067fb..ad61eae 100644 --- a/source/game/items/StarUnlockItem.cpp +++ b/source/game/items/StarUnlockItem.cpp @@ -24,7 +24,7 @@ List<Drawable> UnlockItem::drawables() const { return m_drawables; } -List<Drawable> UnlockItem::preview(PlayerPtr const& viewer) const { +List<Drawable> UnlockItem::preview(PlayerPtr const&) const { return iconDrawables(); } diff --git a/source/game/scripting/StarLuaGameConverters.cpp b/source/game/scripting/StarLuaGameConverters.cpp index 64415e6..15dd11d 100644 --- a/source/game/scripting/StarLuaGameConverters.cpp +++ b/source/game/scripting/StarLuaGameConverters.cpp @@ -470,7 +470,7 @@ LuaValue LuaConverter<Drawable>::from(LuaEngine& engine, Drawable const& v) { return table; } -Maybe<Drawable> LuaConverter<Drawable>::to(LuaEngine& engine, LuaValue const& v) { +Maybe<Drawable> LuaConverter<Drawable>::to(LuaEngine&, LuaValue const& v) { if (auto table = v.ptr<LuaTable>()) { Maybe<Drawable> result; result.emplace(); |