commit 9449616020a58043a49f0342679bf095918792a6
author herb <herb@google.com> Thu Dec 10 14:17:41 2015 -0800
committer Commit bot <commit-bot@chromium.org> Thu Dec 10 14:17:41 2015 -0800
tree 78b7be3d49b1e4c0c79f318a11c93b1686144d72
parent 419c37269f7d0786e9058c93003061ed8037a402

Fix UB function problems for AA merger.

BUG=skia:4634

Review URL: https://codereview.chromium.org/1518863002

src/core/SkAAClip.cpp

diff --git a/src/core/SkAAClip.cpp b/src/core/SkAAClip.cpp
index bbd1f55..0de6d35 100644
--- a/src/core/SkAAClip.cpp
+++ b/src/core/SkAAClip.cpp
@@ -2056,9 +2056,10 @@
                        SkMulDiv255Round(b, alpha));
 }
 
-template <typename T> void mergeT(const T* SK_RESTRICT src, int srcN,
-                                 const uint8_t* SK_RESTRICT row, int rowN,
-                                 T* SK_RESTRICT dst) {
+template <typename T>
+void mergeT(const void* inSrc, int srcN, const uint8_t* SK_RESTRICT row, int rowN, void* inDst) {
+    const T* SK_RESTRICT src = static_cast<const T*>(inSrc);
+    T* SK_RESTRICT       dst = static_cast<T*>(inDst);
     for (;;) {
         SkASSERT(rowN > 0);
         SkASSERT(srcN > 0);
@@ -2094,14 +2095,10 @@
             SkDEBUGFAIL("unsupported");
             return nullptr;
         case SkMask::kA8_Format:
-        case SkMask::k3D_Format: {
-            void (*proc8)(const uint8_t*, int, const uint8_t*, int, uint8_t*) = mergeT;
-            return (MergeAAProc)proc8;
-        }
-        case SkMask::kLCD16_Format: {
-            void (*proc16)(const uint16_t*, int, const uint8_t*, int, uint16_t*) = mergeT;
-            return (MergeAAProc)proc16;
-        }
+        case SkMask::k3D_Format:
+            return mergeT<uint8_t> ;
+        case SkMask::kLCD16_Format:
+            return mergeT<uint16_t>;
         default:
             SkDEBUGFAIL("unsupported");
             return nullptr;