use empty() rather than size() for determining the emptyness of the transform stack
authorJonathan Marshall <jmarshall@never.you.mind>
Sun, 10 Feb 2013 23:02:58 +0000 (12:02 +1300)
committerJonathan Marshall <jmarshall@never.you.mind>
Sun, 10 Feb 2013 23:03:17 +0000 (12:03 +1300)
xbmc/guilib/GraphicContext.h

index 6fa7ab4..0ce3087 100644 (file)
@@ -176,8 +176,8 @@ public:
   }
   inline TransformMatrix AddTransform(const TransformMatrix &matrix)
   {
-    ASSERT(m_groupTransform.size());
-    TransformMatrix absoluteMatrix = m_groupTransform.size() ? m_groupTransform.top() * matrix : matrix;
+    ASSERT(!m_groupTransform.empty());
+    TransformMatrix absoluteMatrix = m_groupTransform.empty() ? matrix : m_groupTransform.top() * matrix;
     m_groupTransform.push(absoluteMatrix);
     UpdateFinalTransform(absoluteMatrix);
     return absoluteMatrix;
@@ -186,16 +186,16 @@ public:
   {
     // TODO: We only need to add it to the group transform as other transforms may be added on top of this one later on
     //       Once all transforms are cached then this can be removed and UpdateFinalTransform can be called directly
-    ASSERT(m_groupTransform.size());
+    ASSERT(!m_groupTransform.empty());
     m_groupTransform.push(matrix);
     UpdateFinalTransform(m_groupTransform.top());
   }
   inline void RemoveTransform()
   {
-    ASSERT(m_groupTransform.size());
-    if (m_groupTransform.size())
+    ASSERT(!m_groupTransform.empty());
+    if (!m_groupTransform.empty())
       m_groupTransform.pop();
-    if (m_groupTransform.size())
+    if (!m_groupTransform.empty())
       UpdateFinalTransform(m_groupTransform.top());
     else
       UpdateFinalTransform(TransformMatrix());