Swizzler changes Index8 and 565

src/codec/SkCodec_libbmp.h

Index: src/codec/SkCodec_libbmp.h
diff --git a/src/codec/SkCodec_libbmp.h b/src/codec/SkCodec_libbmp.h
index 650259ad16cea6b8191097c6a80455768458defd..4ff366c059235c9b94a8253466e6194507ad708f 100644
--- a/src/codec/SkCodec_libbmp.h
+++ b/src/codec/SkCodec_libbmp.h
@@ -120,7 +120,7 @@ private:
      * Set an RLE pixel using the color table
      *
      */
-    void setRLEPixel(SkPMColor* dst, size_t dstRowBytes,
+    void setRLEPixel(void* dst, size_t dstRowBytes,

[scroggo, 2015/04/02 19:20:31]

Why did these change?

[msarett, 2015/04/03 18:01:32]

I wanted to setRLEPixels for kN32 and for kIndex8, so it made more sense to pass
a void* than a SkPMColor*.  After the fact, I removed the capability to
setRLEPixels for kIndex8 because of the fact that some pixels in a decoded RLE
may be left transparent, and we do not have a transparent index to create a
valid kIndex8 representation.

However, I left the change to void* because I think it is cleaner to pass a
generic destination to setRLEPixels and then to cast the pointer in setRLEPixels
to the color type indicated by dstInfo.

[scroggo, 2015/04/06 14:55:11]

sgtm

                      const SkImageInfo& dstInfo, uint32_t x, uint32_t y,
                      uint8_t index);
     /*
@@ -128,7 +128,7 @@ private:
      * Set an RLE24 pixel from R, G, B values
      *
      */
-    void setRLE24Pixel(SkPMColor* dst, size_t dstRowBytes,
+    void setRLE24Pixel(void* dst, size_t dstRowBytes,
                        const SkImageInfo& dstInfo, uint32_t x, uint32_t y,
                        uint8_t red, uint8_t green, uint8_t blue);
 
@@ -138,7 +138,7 @@ private:
      *
      */
     Result decodeRLE(const SkImageInfo& dstInfo, void* dst,
-                     size_t dstRowBytes);
+                     size_t dstRowBytes, const Options& opts);
 
     /*
      *
@@ -180,7 +180,7 @@ private:
     const uint16_t                      fBitsPerPixel;
     const BitmapInputFormat             fInputFormat;
     SkAutoTDelete<SkMasks>              fMasks;          // owned
-    SkAutoTDelete<SkColorTable>         fColorTable;     // owned
+    SkPMColor*                          fColorTable;     // unowned
     uint32_t                            fNumColors;
     const uint32_t                      fBytesPerColor;
     const uint32_t                      fOffset;