sera/0ad
1
0
Fork 0
forked from 0ad/0ad
Code Pull Requests Activity

Fix most Clang 10 Warnings

Browse Source 
Refs: #5550, #5294

Remove dead code from f71be3c764
Fix buggy code from ff50b0b74c
Comments by: @wraitii, @vladislavbelov
Differential Revision: https://code.wildfiregames.com/D3177
This was SVN commit r24487.
This commit is contained in:
Stan 2020-12-31 14:25:37 +00:00
parent a395a0ab66
commit 4942cabab5
15 changed files with 88 additions and 81 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
source/renderer/ModelRenderer.cpp
View File

@ -314,11 +314,14 @@ struct SMRCompareSortByDistItem
}
};
struct SMRMaterialBucketKey
class SMRMaterialBucketKey
{
public:
SMRMaterialBucketKey(CStrIntern effect, const CShaderDefines& defines)
: effect(effect), defines(defines) { }
SMRMaterialBucketKey(const SMRMaterialBucketKey& entity) = default;
CStrIntern effect;
CShaderDefines defines;

4
source/renderer/PatchRData.cpp
View File

@ -1131,7 +1131,7 @@ void CPatchRData::RenderStreams(const std::vector<CPatchRData*>& patches, const
ENSURE(!(streamflags & ~(STREAM_POS|STREAM_POSTOUV0|STREAM_POSTOUV1)));
// Render each batch
for (const std::pair<CVertexBuffer*, StreamIndexBufferBatches>& streamBatch : batches)
for (const std::pair<CVertexBuffer* const, StreamIndexBufferBatches>& streamBatch : batches)
{
GLsizei stride = sizeof(SBaseVertex);
SBaseVertex *base = (SBaseVertex *)streamBatch.first->Bind();
@ -1144,7 +1144,7 @@ void CPatchRData::RenderStreams(const std::vector<CPatchRData*>& patches, const
shader->AssertPointersBound();
for (const std::pair<CVertexBuffer*, StreamBatchElements>& batchIndexBuffer : streamBatch.second)
for (const std::pair<CVertexBuffer* const, StreamBatchElements>& batchIndexBuffer : streamBatch.second)
{
batchIndexBuffer.first->Bind();

4
source/simulation2/components/CCmpCinemaManager.cpp
View File

@ -1,4 +1,4 @@
/* Copyright (C) 2017 Wildfire Games.
/* Copyright (C) 2020 Wildfire Games.
* This file is part of 0 A.D.
*
* 0 A.D. is free software: you can redistribute it and/or modify
@ -65,7 +65,7 @@ public:
serializer.Bool("MapRevealed", m_MapRevealed);
serializer.NumberU32_Unbounded("NumberOfPaths", m_Paths.size());
for (const std::pair<CStrW, CCinemaPath>& it : m_Paths)
for (const std::pair<const CStrW, CCinemaPath>& it : m_Paths)
SerializePath(it.second, serializer);
serializer.NumberU32_Unbounded("NumberOfQueuedPaths", m_PathQueue.size());

2
source/simulation2/components/CCmpPathfinder.cpp
View File

@ -262,7 +262,7 @@ void CCmpPathfinder::GetPassabilityClasses(std::map<std::string, pass_class_t>&
void CCmpPathfinder::GetPassabilityClasses(std::map<std::string, pass_class_t>& nonPathfindingPassClasses, std::map<std::string, pass_class_t>& pathfindingPassClasses) const
{
for (const std::pair<std::string, pass_class_t>& pair : m_PassClassMasks)
for (const std::pair<const std::string, pass_class_t>& pair : m_PassClassMasks)
{
if ((GetPassabilityFromMask(pair.second)->m_Obstructions == PathfinderPassability::PATHFINDING))
pathfindingPassClasses[pair.first] = pair.second;

5
source/simulation2/components/CCmpRangeManager.cpp
View File

@ -300,13 +300,16 @@ struct SerializeHelper<EntityData>
* It must only be passed entities that are in 'entities'
* and are currently in the world.
*/
struct EntityDistanceOrdering
class EntityDistanceOrdering
{
public:
EntityDistanceOrdering(const EntityMap<EntityData>& entities, const CFixedVector2D& source) :
m_EntityData(entities), m_Source(source)
{
}
EntityDistanceOrdering(const EntityDistanceOrdering& entity) = default;
bool operator()(entity_id_t a, entity_id_t b) const
{
const EntityData& da = m_EntityData.find(a)->second;

12
source/simulation2/components/CCmpTemplateManager.cpp
View File

@ -58,9 +58,9 @@ public:
virtual void Serialize(ISerializer& serialize)
{
std::map<CStr, std::vector<entity_id_t>> templateMap;
std::map<std::string, std::vector<entity_id_t>> templateMap;
for (const std::pair<entity_id_t, std::string>& templateEnt : m_LatestTemplates)
for (const std::pair<const entity_id_t, std::string>& templateEnt : m_LatestTemplates)
if (!ENTITY_IS_LOCAL(templateEnt.first))
templateMap[templateEnt.second].push_back(templateEnt.first);
@ -71,9 +71,9 @@ public:
{
Init(paramNode);
std::map<CStr, std::vector<entity_id_t>> templateMap;
std::map<std::string, std::vector<entity_id_t>> templateMap;
Serializer(deserialize, "templates", templateMap);
for (const std::pair<CStr, std::vector<entity_id_t>>& mapEl : templateMap)
for (const std::pair<const std::string, std::vector<entity_id_t>>& mapEl : templateMap)
for (entity_id_t id : mapEl.second)
m_LatestTemplates[id] = mapEl.first;
}
@ -218,7 +218,7 @@ std::vector<std::string> CCmpTemplateManager::FindAllTemplates(bool includeActor
std::vector<std::string> CCmpTemplateManager::FindUsedTemplates() const
{
std::vector<std::string> usedTemplates;
for (const std::pair<entity_id_t, std::string>& p : m_LatestTemplates)
for (const std::pair<const entity_id_t, std::string>& p : m_LatestTemplates)
if (std::find(usedTemplates.begin(), usedTemplates.end(), p.second) == usedTemplates.end())
usedTemplates.push_back(p.second);
return usedTemplates;
@ -230,7 +230,7 @@ std::vector<std::string> CCmpTemplateManager::FindUsedTemplates() const
std::vector<entity_id_t> CCmpTemplateManager::GetEntitiesUsingTemplate(const std::string& templateName) const
{
std::vector<entity_id_t> entities;
for (const std::pair<entity_id_t, std::string>& p : m_LatestTemplates)
for (const std::pair<const entity_id_t, std::string>& p : m_LatestTemplates)
if (p.second == templateName)
entities.push_back(p.first);

4
source/simulation2/components/CCmpTerritoryManager.cpp
View File

@ -459,14 +459,14 @@ void CCmpTerritoryManager::CalculateTerritories()
// store the root influences to mark territory as connected
std::vector<entity_id_t> rootInfluenceEntities;
for (const std::pair<player_id_t, std::vector<entity_id_t> >& pair : influenceEntities)
for (const std::pair<const player_id_t, std::vector<entity_id_t>>& pair : influenceEntities)
{
// entityGrid stores the weight for a single entity, and is reset per entity
Grid<u32> entityGrid(tilesW, tilesH);
// playerGrid stores the combined weight of all entities for this player
Grid<u32> playerGrid(tilesW, tilesH);
u8 owner = (u8)pair.first;
u8 owner = static_cast<u8>(pair.first);
const std::vector<entity_id_t>& ents = pair.second;
// With 2^16 entities, we're safe against overflows as the weight is also limited to 2^16
ENSURE(ents.size() < 1 << 16);

6
source/simulation2/helpers/HierarchicalPathfinder.cpp
View File

@ -467,7 +467,7 @@ void HierarchicalPathfinder::Update(Grid<NavcellData>* grid, const Grid<u8>& dir
if (!dirtinessGrid.any_set_in_square(i0, j0, i1, j1))
continue;
for (const std::pair<std::string, pass_class_t>& passClassMask : m_PassClassMasks)
for (const std::pair<const std::string, pass_class_t>& passClassMask : m_PassClassMasks)
{
pass_class_t passClass = passClassMask.second;
Chunk& a = m_Chunks[passClass].at(ci + cj*m_ChunksW);
@ -627,7 +627,8 @@ void HierarchicalPathfinder::UpdateGlobalRegions(const std::map<pass_class_t, st
{
// Use FindReachableRegions because we cannot be sure, even if we find a non-dirty chunk nearby,
// that we weren't the only bridge connecting that chunk to the rest of the global region.
for (const std::pair<pass_class_t, std::vector<RegionID> >& regionsInNeed : needNewGlobalRegionMap)
for (const std::pair<const pass_class_t, std::vector<RegionID>>& regionsInNeed : needNewGlobalRegionMap)
{
for (const RegionID& reg : regionsInNeed.second)
{
std::map<RegionID, GlobalRegionID>& globalRegions = m_GlobalRegions[regionsInNeed.first];
@ -643,6 +644,7 @@ void HierarchicalPathfinder::UpdateGlobalRegions(const std::map<pass_class_t, st
for (const RegionID& regionId : reachable)
globalRegions[regionId] = ID;
}
}
}
HierarchicalPathfinder::RegionID HierarchicalPathfinder::Get(u16 i, u16 j, pass_class_t passClass) const

2
source/soundmanager/SoundManager.cpp
View File

@ -277,7 +277,7 @@ CSoundManager::~CSoundManager()
}
AL_CHECK;
for (const std::pair<std::wstring, CSoundGroup*>& p : m_SoundGroups)
for (const std::pair<const std::wstring, CSoundGroup*>& p : m_SoundGroups)
delete p.second;
m_SoundGroups.clear();

4
source/third_party/jsonspirit/json_spirit_value.h vendored
View File

@ -32,7 +32,7 @@ namespace json_spirit
{
enum Value_type{ obj_type, array_type, str_type, bool_type, int_type, real_type, null_type };
static std::string value_type_to_string( Value_type vtype );
static inline std::string value_type_to_string(const Value_type vtype );
struct Null{};
@ -584,7 +584,7 @@ namespace json_spirit
return internal_::get_value( *this, internal_::Type_to_type< T >() );
}
static std::string value_type_to_string( const Value_type vtype )
static inline std::string value_type_to_string( const Value_type vtype )
{
switch( vtype )
{

18
source/tools/atlas/AtlasUI/ScenarioEditor/Sections/Map/Map.cpp
View File

@ -239,7 +239,7 @@ void MapSettingsControl::ReadFromEngine()
m_MapSettingsVictoryConditions.insert(std::string(victoryCondition));
// Clear Checkboxes before loading data. We don't update victory condition just yet because it might reenable some of the checkboxes.
for (const std::pair<long, AtObj>& vc : m_VictoryConditions)
for (const std::pair<const long, AtObj>& vc : m_VictoryConditions)
{
wxCheckBox* checkBox = wxDynamicCast(FindWindow(vc.first), wxCheckBox);
if (!checkBox)
@ -249,7 +249,7 @@ void MapSettingsControl::ReadFromEngine()
checkBox->Enable(true);
}
for (const std::pair<long, AtObj>& vc : m_VictoryConditions)
for (const std::pair<const long, AtObj>& vc : m_VictoryConditions)
{
std::string escapedTitle = wxString::FromUTF8(vc.second["Data"]["Title"]).Lower().ToStdString();
std::replace(escapedTitle.begin(), escapedTitle.end(), ' ', '_');
@ -298,20 +298,20 @@ void MapSettingsControl::OnVictoryConditionChanged(long controlId)
AtObj victoryCondition;
wxCheckBox* modifiedCheckbox = wxDynamicCast(FindWindow(controlId), wxCheckBox);
for (const std::pair<long, AtObj>& vc : m_VictoryConditions)
for (const std::pair<const long, AtObj>& vc : m_VictoryConditions)
{
if(vc.first != controlId)
if (vc.first != controlId)
continue;
victoryCondition = vc.second;
break;
}
if(modifiedCheckbox->GetValue())
if (modifiedCheckbox->GetValue())
{
for (AtIter victoryConditionPair = victoryCondition["Data"]["ChangeOnChecked"]; victoryConditionPair.defined(); ++victoryConditionPair)
{
for (const std::pair<long, AtObj>& vc : m_VictoryConditions)
for (const std::pair<const long, AtObj>& vc : m_VictoryConditions)
{
std::string escapedTitle = wxString::FromUTF8(vc.second["Data"]["Title"]).Lower().ToStdString();
std::replace(escapedTitle.begin(), escapedTitle.end(), ' ', '_');
@ -325,7 +325,7 @@ void MapSettingsControl::OnVictoryConditionChanged(long controlId)
}
}
for (const std::pair<long, AtObj>& vc : m_VictoryConditions)
for (const std::pair<const long, AtObj>& vc : m_VictoryConditions)
{
if (vc.first == controlId)
continue;
@ -333,7 +333,7 @@ void MapSettingsControl::OnVictoryConditionChanged(long controlId)
wxCheckBox* otherCheckbox = wxDynamicCast(FindWindow(vc.first), wxCheckBox);
otherCheckbox->Enable(true);
for (const std::pair<long, AtObj>& vc2 : m_VictoryConditions)
for (const std::pair<const long, AtObj>& vc2 : m_VictoryConditions)
{
for (AtIter victoryConditionTitle = vc2.second["Data"]["DisabledWhenChecked"]; victoryConditionTitle.defined(); ++victoryConditionTitle)
{
@ -371,7 +371,7 @@ AtObj MapSettingsControl::UpdateSettingsObject()
else \
m_MapSettingsVictoryConditions.erase(name);
for (const std::pair<long, AtObj>& vc : m_VictoryConditions)
for (const std::pair<const long, AtObj>& vc : m_VictoryConditions)
{
std::string escapedTitle = wxString::FromUTF8(vc.second["Data"]["Title"]).Lower().ToStdString();
std::replace(escapedTitle.begin(), escapedTitle.end(), ' ', '_');

4
source/tools/atlas/GameInterface/Handlers/CinemaHandler.cpp
View File

@ -1,4 +1,4 @@
/* Copyright (C) 2017 Wildfire Games.
/* Copyright (C) 2020 Wildfire Games.
* This file is part of 0 A.D.
*
* 0 A.D. is free software: you can redistribute it and/or modify
@ -491,7 +491,7 @@ QUERYHANDLER(PickPathNode)
CVector2D cursor;
msg->pos->GetScreenSpace(cursor.X, cursor.Y);
for (const std::pair<CStrW, CCinemaPath>& p : cmpCinemaManager->GetPaths())
for (const std::pair<const CStrW, CCinemaPath>& p : cmpCinemaManager->GetPaths())
{
const CCinemaPath& path = p.second;
if (isPathNodePicked(path, cursor, node, false) || isPathNodePicked(path.GetTargetSpline(), cursor, node, true))

3
source/tools/atlas/GameInterface/Handlers/MapHandlers.cpp
View File

@ -518,10 +518,9 @@ BEGIN_COMMAND(ResizeMap)
const CFixedVector3D offset = CFixedVector3D(fixed::FromInt(offsetX), fixed::FromInt(0), fixed::FromInt(offsetZ));
const CSimulation2::InterfaceListUnordered& ents = sim.GetEntitiesWithInterfaceUnordered(IID_Selectable);
for (const std::pair<entity_id_t, IComponent*>& ent : ents)
for (const std::pair<const entity_id_t, IComponent*>& ent : ents)
{
const entity_id_t entityId = ent.first;
CmpPtr<ICmpPosition> cmpPosition(sim, entityId);
if (cmpPosition && cmpPosition->IsInWorld() && Within(cmpPosition->GetPosition(), mapCenterX, mapCenterZ, radiusInTerrainUnits))

8
source/tools/atlas/GameInterface/Handlers/ObjectHandlers.cpp
View File

@ -767,7 +767,7 @@ BEGIN_COMMAND(MoveObjects)
void SetPos(const std::map<entity_id_t, CVector3D>& map)
{
for (const std::pair<entity_id_t, CVector3D>& p : map)
for (const std::pair<const entity_id_t, CVector3D>& p : map)
{
CmpPtr<ICmpPosition> cmpPosition(*g_Game->GetSimulation2(), p.first);
if (!cmpPosition)
@ -859,7 +859,7 @@ BEGIN_COMMAND(RotateObjectsFromCenterPoint)
void SetPos(const std::map<entity_id_t, CVector3D>& position, const std::map<entity_id_t, float>& angle)
{
for (const std::pair<entity_id_t, CVector3D>& p : position)
for (const std::pair<const entity_id_t, CVector3D>& p : position)
{
CmpPtr<ICmpPosition> cmpPosition(*g_Game->GetSimulation2(), p.first);
if (!cmpPosition)
@ -873,7 +873,7 @@ BEGIN_COMMAND(RotateObjectsFromCenterPoint)
}
for (const std::pair<entity_id_t, CVector3D>& p: position)
for (const std::pair<const entity_id_t, CVector3D>& p : position)
CheckObstructionAndUpdateVisual(p.first);
}
@ -963,7 +963,7 @@ BEGIN_COMMAND(RotateObject)
void SetAngle(const std::map<entity_id_t, float>& angles)
{
for (const std::pair<entity_id_t, float>& p : angles)
for (const std::pair<const entity_id_t, float>& p : angles)
{
CmpPtr<ICmpPosition> cmpPosition(*g_Game->GetSimulation2(), p.first);
if (!cmpPosition)

2
source/tools/atlas/GameInterface/View.cpp
View File

@ -179,7 +179,7 @@ AtlasViewGame::AtlasViewGame()
AtlasViewGame::~AtlasViewGame()
{
for (const std::pair<std::wstring, SimState*>& p : m_SavedStates)
for (const std::pair<const std::wstring, SimState*>& p : m_SavedStates)
delete p.second;
}