diff options
author | Jacques Lucke <jacques@blender.org> | 2020-07-03 15:59:27 +0300 |
---|---|---|
committer | Jacques Lucke <jacques@blender.org> | 2020-07-03 15:59:27 +0300 |
commit | 2633683b52054d15006aea1314f62f73a8505b42 (patch) | |
tree | a97b2916170b53f04fc9a898f4eedb527f14f84e /source/blender/freestyle | |
parent | 93da09d717ff4502975c506c574faf0c07f010b4 (diff) |
Clang-tidy: enable readability-container-size-empty warning
Reviewers: sergey
Differential Revision: https://developer.blender.org/D8197
Diffstat (limited to 'source/blender/freestyle')
7 files changed, 10 insertions, 10 deletions
diff --git a/source/blender/freestyle/intern/application/Controller.cpp b/source/blender/freestyle/intern/application/Controller.cpp index 253d62ea3dc..dfe78eaf10c 100644 --- a/source/blender/freestyle/intern/application/Controller.cpp +++ b/source/blender/freestyle/intern/application/Controller.cpp @@ -469,7 +469,7 @@ void Controller::DeleteViewMap(bool freeCache) void Controller::ComputeViewMap() { - if (!_ListOfModels.size()) { + if (_ListOfModels.empty()) { return; } diff --git a/source/blender/freestyle/intern/blender_interface/BlenderFileLoader.cpp b/source/blender/freestyle/intern/blender_interface/BlenderFileLoader.cpp index 996906826e9..0af57e5d728 100644 --- a/source/blender/freestyle/intern/blender_interface/BlenderFileLoader.cpp +++ b/source/blender/freestyle/intern/blender_interface/BlenderFileLoader.cpp @@ -746,7 +746,7 @@ void BlenderFileLoader::insertShapeNode(Object *ob, Mesh *me, int id) detriList.push_back(detri); } - if (detriList.size() > 0) { + if (!detriList.empty()) { vector<detri_t>::iterator v; for (v = detriList.begin(); v != detriList.end(); v++) { detri_t detri = (*v); diff --git a/source/blender/freestyle/intern/geometry/Grid.cpp b/source/blender/freestyle/intern/geometry/Grid.cpp index e3e25796de1..46e91197764 100644 --- a/source/blender/freestyle/intern/geometry/Grid.cpp +++ b/source/blender/freestyle/intern/geometry/Grid.cpp @@ -93,7 +93,7 @@ bool firstIntersectionGridVisitor::stop() ///////////////// void Grid::clear() { - if (_occluders.size() != 0) { + if (!_occluders.empty()) { for (OccludersSet::iterator it = _occluders.begin(); it != _occluders.end(); it++) { delete (*it); } @@ -154,7 +154,7 @@ void Grid::configure(const Vec3r &orig, const Vec3r &size, unsigned nb) void Grid::insertOccluder(Polygon3r *occluder) { const vector<Vec3r> vertices = occluder->getVertices(); - if (vertices.size() == 0) { + if (vertices.empty()) { return; } diff --git a/source/blender/freestyle/intern/scene_graph/NodeShape.cpp b/source/blender/freestyle/intern/scene_graph/NodeShape.cpp index e55b2fb625f..cf7a609b347 100644 --- a/source/blender/freestyle/intern/scene_graph/NodeShape.cpp +++ b/source/blender/freestyle/intern/scene_graph/NodeShape.cpp @@ -27,7 +27,7 @@ NodeShape::~NodeShape() { vector<Rep *>::iterator rep; - if (0 != _Shapes.size()) { + if (!_Shapes.empty()) { for (rep = _Shapes.begin(); rep != _Shapes.end(); ++rep) { int refCount = (*rep)->destroy(); if (0 == refCount) { diff --git a/source/blender/freestyle/intern/view_map/ViewMapBuilder.cpp b/source/blender/freestyle/intern/view_map/ViewMapBuilder.cpp index c3c43ce6722..b7de3a5b319 100644 --- a/source/blender/freestyle/intern/view_map/ViewMapBuilder.cpp +++ b/source/blender/freestyle/intern/view_map/ViewMapBuilder.cpp @@ -633,7 +633,7 @@ static void computeCumulativeVisibility(ViewMap *ioViewMap, wFaces.clear(); } - if (iRenderMonitor && vedges.size()) { + if (iRenderMonitor && !vedges.empty()) { stringstream ss; ss << "Freestyle: Visibility computations " << (100 * cnt / vedges.size()) << "%"; iRenderMonitor->setInfo(ss.str()); diff --git a/source/blender/freestyle/intern/view_map/ViewMapTesselator.cpp b/source/blender/freestyle/intern/view_map/ViewMapTesselator.cpp index 8357231a73d..ef79384e2af 100644 --- a/source/blender/freestyle/intern/view_map/ViewMapTesselator.cpp +++ b/source/blender/freestyle/intern/view_map/ViewMapTesselator.cpp @@ -25,7 +25,7 @@ namespace Freestyle { NodeGroup *ViewMapTesselator::Tesselate(ViewMap *iViewMap) { - if (0 == iViewMap->ViewEdges().size()) { + if (iViewMap->ViewEdges().empty()) { return NULL; } diff --git a/source/blender/freestyle/intern/winged_edge/WingedEdgeBuilder.cpp b/source/blender/freestyle/intern/winged_edge/WingedEdgeBuilder.cpp index 620601df5e7..63f65988fc9 100644 --- a/source/blender/freestyle/intern/winged_edge/WingedEdgeBuilder.cpp +++ b/source/blender/freestyle/intern/winged_edge/WingedEdgeBuilder.cpp @@ -200,7 +200,7 @@ bool WingedEdgeBuilder::buildWShape(WShape &shape, IndexedFaceSet &ifs) delete[] new_vertices; delete[] new_normals; - if (shape.GetFaceList().size() == 0) { // this may happen due to degenerate triangles + if (shape.GetFaceList().empty()) { // this may happen due to degenerate triangles return false; } @@ -219,8 +219,8 @@ bool WingedEdgeBuilder::buildWShape(WShape &shape, IndexedFaceSet &ifs) if ((*wv)->isBoundary()) { continue; } - if ((*wv)->GetEdges().size() == - 0) { // This means that the WVertex has no incoming edges... (12-Sep-2011 T.K.) + if ((*wv)->GetEdges().empty()) { + // This means that the WVertex has no incoming edges... (12-Sep-2011 T.K.) continue; } normalsSet.clear(); |