Analytic AntiAlias for Convex Shapes

src/core/SkAntiRun.h

 /*
  * 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.
  */
 
 
 #ifndef SkAntiRun_DEFINED
 #define SkAntiRun_DEFINED
@@ skipping 61 matching lines @@
             x = 0;
             SkDEBUGCODE(this->validate();)
         }
 
         if (middleCount) {
             SkAlphaRuns::Break(runs, alpha, x, middleCount);
             alpha += x;
             runs += x;
             x = 0;
             do {
-                alpha[0] = SkToU8(alpha[0] + maxValue);
+                alpha[0] = SkToU8(SkTMin<int>(0xFF, alpha[0] + maxValue));

[reed1, 2016/08/18 20:31:00]

is this needed today, or just with your CL? If today, lets land this sooner.

[liyuqian, 2016/08/22 15:30:48]

I don't think that the current supersampling algorithm will trigger this because
it's descretized (0 - 16) without many arithmetic operations.

Anyway, I found that I actually triggered this because of some bugs in my code.
I've fixed them and I no longer need this check.

                 int n = runs[0];
                 SkASSERT(n <= middleCount);
                 alpha += n;
                 runs += n;
                 middleCount -= n;
             } while (middleCount > 0);
             SkDEBUGCODE(this->validate();)
             lastAlpha = alpha;
         }
 
@@ skipping 88 matching lines @@
             x -= n;
         }
     }
 
 private:
     SkDEBUGCODE(int fWidth;)
     SkDEBUGCODE(void validate() const;)
 };
 
 #endif