Pass the destination pointer to next

src/codec/SkMaskSwizzler.h

Index: src/codec/SkMaskSwizzler.h
diff --git a/src/codec/SkMaskSwizzler.h b/src/codec/SkMaskSwizzler.h
index 69103023c63e06c7df65427a685382b146d48a04..b2d04d9618bc0be5c1d31f73d0cbfe46d7ab746c 100644
--- a/src/codec/SkMaskSwizzler.h
+++ b/src/codec/SkMaskSwizzler.h
@@ -25,16 +25,15 @@ public:
      *
      */
     static SkMaskSwizzler* CreateMaskSwizzler(const SkImageInfo& imageInfo,
-                                              void* dst, size_t dstRowBytes,
                                               SkMasks* masks,
                                               uint32_t bitsPerPixel);
 
     /*
      *
-     * Swizzle the row with the specified y value
+     * Swizzle a row
      *
      */
-    SkSwizzler::ResultAlpha next(const uint8_t* SK_RESTRICT src, int y);
+    SkSwizzler::ResultAlpha next(const uint8_t* SK_RESTRICT src, void* dst);

[scroggo, 2015/07/27 20:42:06]

As I explain in more detail in SkSwizzler.cpp, I think these parameters should
be swapped.

[msarett, 2015/07/27 21:11:33]

Done.

 
 private:
 
@@ -52,13 +51,10 @@ private:
      * Constructor for mask swizzler
      *
      */
-    SkMaskSwizzler(const SkImageInfo& info, void* dst, size_t dstRowBytes,
-            SkMasks* masks, RowProc proc);
+    SkMaskSwizzler(const SkImageInfo& info, SkMasks* masks, RowProc proc);
 
     // Fields
     const SkImageInfo& fDstInfo;
-    void*              fDst;
-    size_t             fDstRowBytes;
     SkMasks*           fMasks;       // unowned
     const RowProc      fRowProc;
 };