Avoid counting verbs twice in SkPath::isEmpty()

src/core/SkPath.cpp

 
 /*
  * Copyright 2006 The Android Open Source Project
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 
 #include "SkBuffer.h"
@@ skipping 362 matching lines @@
     this->resetFields();
 }
 
 bool SkPath::isEmpty() const {
     SkDEBUGCODE(this->validate();)
     return 0 == fPathRef->countVerbs();
 }
 
 bool SkPath::isLine(SkPoint line[2]) const {
     int verbCount = fPathRef->countVerbs();
-    int ptCount = fPathRef->countVerbs();
 
-    if (2 == verbCount && 2 == ptCount) {
-        if (kMove_Verb == fPathRef->atVerb(0) &&
-            kLine_Verb == fPathRef->atVerb(1)) {
+    if (2 == verbCount) {
+        SkASSERT(kMove_Verb == fPathRef->atVerb(0));
+        if (kLine_Verb == fPathRef->atVerb(1)) {
+            SkASSERT(2 == fPathRef->countPoints());
             if (line) {
                 const SkPoint* pts = fPathRef->points();
                 line[0] = pts[0];
                 line[1] = pts[1];
             }
             return true;
         }
     }
     return false;
 }
@@ skipping 2597 matching lines @@
     switch (this->getFillType()) {
         case SkPath::kEvenOdd_FillType:
         case SkPath::kInverseEvenOdd_FillType:
             w &= 1;
             break;
         default:
             break;
     }
     return SkToBool(w);
 }