Merge lp:~mc-return/unity/unity.merge-minor-possible-speed-improvement into lp:unity

[MC Return]

Minor possible performance improvement by replacement of 'rendering_stack.size() > 0' with '!rendering_stack.empty()' in unity-shared/GraphicsUtils.cpp.
xxxx.size() can take linear time while xxxx.empty() is guaranteed to take constant time.

[lp:~mc-return/unity/unity.merge-minor-possible-speed-improvement]

2881. By MC Return on 2012-11-08

Merged latest lp:unity

[Thomi Richards (thomir)]

Looks good to me. I'm happy to merge this without tests.

review: Approve

[Brandon Schaefer (brandontschaefer)]

Looks good to me.

review: Approve

=== modified file 'unity-shared/GraphicsUtils.cpp'
--- unity-shared/GraphicsUtils.cpp	2012-10-31 16:33:00 +0000
+++ unity-shared/GraphicsUtils.cpp	2012-11-08 10:24:18 +0000
@@ -53,10 +53,10 @@
 
 void PopOffscreenRenderTarget()
 {
-  g_assert(rendering_stack.size() > 0);
+  g_assert(!rendering_stack.empty());
 
   rendering_stack.pop();
-  if (rendering_stack.size() > 0)
+  if (!rendering_stack.empty())
   {
     nux::ObjectPtr<nux::IOpenGLBaseTexture>& texture = rendering_stack.top();
     PushOffscreenRenderTarget_(texture);
@@ -68,4 +68,4 @@
 }
 
 }
-}
\ No newline at end of file
+}