Build fix for [-Werror=maybe-uninitialized].

src/core/SkPathRef.h

 
 /*
  * Copyright 2012 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkPathRef_DEFINED
 #define SkPathRef_DEFINED
@@ skipping 452 matching lines @@
     SkPoint* growForVerb(SkPath::Verb verb) {
         this->validate();
         int pCnt;
         switch (verb) {
             case SkPath::kMove_Verb:
                 pCnt = 1;
                 break;
             case SkPath::kLine_Verb:
                 pCnt = 1;
                 break;
+            case SkPath::kQuad_Verb:
+                // fall through
             case SkPath::kConic_Verb:
-            case SkPath::kQuad_Verb:
                 pCnt = 2;
                 break;
             case SkPath::kCubic_Verb:
                 pCnt = 3;
                 break;
+            case SkPath::kClose_Verb:
+                pCnt = 0;
+                break;
             case SkPath::kDone_Verb:
                 SkASSERT(!"growForVerb called for kDone");
                 // fall through
-            case SkPath::kClose_Verb:
+            default:
+                SkASSERT(!"default is not reached");
                 pCnt = 0;

[dshwang, 2013/07/03 07:50:26]

How about this change.
now case statements are sorted by enum order.
pCnt is initialized only one time.

         }
         size_t space = sizeof(uint8_t) + pCnt * sizeof (SkPoint);
         this->makeSpace(space);
         this->fVerbs[~fVerbCnt] = verb;
         SkPoint* ret = fPoints + fPointCnt;
         fVerbCnt += 1;
         fPointCnt += pCnt;
         fFreeSpace -= space;
         this->validate();
         return ret;
@@ skipping 104 matching lines @@
 #if SK_DEBUG_PATH_REF
     SkTDArray<SkPath*> fOwners;
 #endif
 
     typedef SkRefCnt INHERITED;
 };
 
 SK_DEFINE_INST_COUNT(SkPathRef);
 
 #endif