diff options
author | Campbell Barton <ideasman42@gmail.com> | 2012-09-20 04:55:32 +0400 |
---|---|---|
committer | Campbell Barton <ideasman42@gmail.com> | 2012-09-20 04:55:32 +0400 |
commit | 5d56a901a8b5f103cc2511941e70f6daa0bd9a3a (patch) | |
tree | 92aa2de0a14e14b273558b801c0bc8ed7a2f0077 /source/gameengine | |
parent | 52059278920e161a89b8817131d31857bf4d7973 (diff) |
code cleanup: c++ - use empty() rather than size(), use prefix operators for non-primitive types - recommended for non primitive types.
Diffstat (limited to 'source/gameengine')
-rw-r--r-- | source/gameengine/Rasterizer/RAS_MeshObject.cpp | 2 | ||||
-rw-r--r-- | source/gameengine/SceneGraph/SG_Tree.cpp | 2 |
2 files changed, 2 insertions, 2 deletions
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<RAS_MeshMaterial>::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(); |