From 5d56a901a8b5f103cc2511941e70f6daa0bd9a3a Mon Sep 17 00:00:00 2001 From: Campbell Barton Date: Thu, 20 Sep 2012 00:55:32 +0000 Subject: code cleanup: c++ - use empty() rather than size(), use prefix operators for non-primitive types - recommended for non primitive types. --- source/gameengine/Rasterizer/RAS_MeshObject.cpp | 2 +- source/gameengine/SceneGraph/SG_Tree.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) (limited to 'source/gameengine') diff --git a/source/gameengine/Rasterizer/RAS_MeshObject.cpp b/source/gameengine/Rasterizer/RAS_MeshObject.cpp index 25afefbcc5a..c50aa28e9fc 100644 --- a/source/gameengine/Rasterizer/RAS_MeshObject.cpp +++ b/source/gameengine/Rasterizer/RAS_MeshObject.cpp @@ -173,7 +173,7 @@ const STR_String& RAS_MeshObject::GetMaterialName(unsigned int matid) RAS_MeshMaterial* RAS_MeshObject::GetMeshMaterial(unsigned int matid) { - if (m_materials.size() > 0 && (matid < m_materials.size())) + if ((m_materials.empty() == false) && (matid < m_materials.size())) { list::iterator it = m_materials.begin(); while (matid--) ++it; diff --git a/source/gameengine/SceneGraph/SG_Tree.cpp b/source/gameengine/SceneGraph/SG_Tree.cpp index 0f8264985de..99f68ef625a 100644 --- a/source/gameengine/SceneGraph/SG_Tree.cpp +++ b/source/gameengine/SceneGraph/SG_Tree.cpp @@ -236,7 +236,7 @@ void SG_TreeFactory::Add(SG_Tree* tree) SG_Tree* SG_TreeFactory::MakeTreeDown(SG_BBox &bbox) { - if (m_objects.size() == 0) + if (m_objects.empty()) return NULL; if (m_objects.size() == 1) return *m_objects.begin(); -- cgit v1.2.3