add support for capped hairlines

src/core/SkPathRef.cpp

 /*
  * Copyright 2013 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #include "SkBuffer.h"
 #include "SkOncePtr.h"
 #include "SkPath.h"
@@ skipping 555 matching lines @@
         case SkPath::kClose_Verb:
             break;
         case SkPath::kDone_Verb:
             SkASSERT(fVerbs == fVerbStop);
             break;
     }
     fPts = srcPts;
     return (uint8_t) verb;
 }
 
+uint8_t SkPathRef::Iter::peek() const {
+    const uint8_t* next = fVerbs - 1;

[reed1, 2015/12/08 16:59:02]

I wonder if this should just SkASSERT if we read past the end. It seems like a
client would never need to call pack *after* receiving a kDone_Verb. 

Might also inline it if its gets that simple. (or not, just a thought).

[caryclark, 2015/12/08 19:59:39]

I'll try the assert. I thought that if fVerbs is pointing to the last verb, next
is off the end and it would be reasonable to check.

+    return next <= fVerbStop ? (uint8_t) SkPath::kDone_Verb : *next;
+}
+
 #ifdef SK_DEBUG
 void SkPathRef::validate() const {
     this->INHERITED::validate();
     SkASSERT(static_cast<ptrdiff_t>(fFreeSpace) >= 0);
     SkASSERT(reinterpret_cast<intptr_t>(fVerbs) - reinterpret_cast<intptr_t>(fPoints) >= 0);
     SkASSERT((nullptr == fPoints) == (nullptr == fVerbs));
     SkASSERT(!(nullptr == fPoints && 0 != fFreeSpace));
     SkASSERT(!(nullptr == fPoints && 0 != fFreeSpace));
     SkASSERT(!(nullptr == fPoints && fPointCnt));
     SkASSERT(!(nullptr == fVerbs && fVerbCnt));
@@ skipping 53 matching lines @@
                 break;
             default:
                 SkDEBUGFAIL("Unknown Verb");
                 break;
         }
     }
     SkASSERT(mask == fSegmentMask);
 #endif // SK_DEBUG_PATH
 }
 #endif