commit 63ae1cfb10d0d14722df59cba0012f8a4370c090
author robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Fri Aug 17 13:53:05 2012 +0000
committer robertphillips@google.com <robertphillips@google.com@2bbb7eff-a529-9590-31e7-b0007b416f81> Fri Aug 17 13:53:05 2012 +0000
tree a07217b4a1b4df64d8c54228882ee876173837ed
parent 1b3ce47c7b6c14c84d7aaee249b33f9d3b473050

Make SkDeque::back faster & inline

http://codereview.appspot.com/6462073/



git-svn-id: http://skia.googlecode.com/svn/trunk@5149 2bbb7eff-a529-9590-31e7-b0007b416f81

src/core/SkClipStack.cpp

diff --git a/src/core/SkClipStack.cpp b/src/core/SkClipStack.cpp
index 8857244..add46e4 100644
--- a/src/core/SkClipStack.cpp
+++ b/src/core/SkClipStack.cpp
@@ -565,6 +565,7 @@
 
     int32_t genID = GetNextGenID();
 
+    // Use reverse iterator instead of back because Rect path may need previous
     SkDeque::Iter iter(fDeque, SkDeque::Iter::kBack_IterStart);
     Rec* rec = (Rec*) iter.prev();
 
@@ -647,10 +648,9 @@
 }
 
 void SkClipStack::clipEmpty() {
-    
-    SkDeque::Iter iter(fDeque, SkDeque::Iter::kBack_IterStart);
-    Rec* rec = (Rec*) iter.prev();
-    
+
+    Rec* rec = (Rec*) fDeque.back();
+
     if (rec && rec->canBeIntersectedInPlace(fSaveCount, SkRegion::kIntersect_Op)) {
         switch (rec->fState) {
             case Rec::kEmpty_State: