SkScaledCodec class

src/codec/SkSwizzler.h

 /*
  * Copyright 2015 Google Inc.
  *
  * Use of this source code is governed by a BSD-style license that can be
  * found in the LICENSE file.
  */
 
 #ifndef SkSwizzler_DEFINED
 #define SkSwizzler_DEFINED
 
@@ skipping 97 matching lines @@
      *
      */
     static int BytesPerPixel(SrcConfig sc) {
         SkASSERT(SkIsAlign8(BitsPerPixel(sc)));
         return BitsPerPixel(sc) >> 3;
     }
 
     /**
      *  Create a new SkSwizzler.
      *  @param SrcConfig Description of the format of the source.
-     *  @param SkImageInfo dimensions() describe both the src and the dst.
-     *              Other fields describe the dst.
+     *  @param dstInfo describes the destination.
      *  @param ZeroInitialized Whether dst is zero-initialized. The
                                implementation may choose to skip writing zeroes
      *                         if set to kYes_ZeroInitialized.
+     *  @param srcWidth is the width of the source. Used to calculate the width samplesize. 
+     *  Width sampling is supported by the swizzler, by skipping pixels when swizzling the row.

[scroggo, 2015/07/31 19:31:58]

Maybe this should also explain why we chose to support width sampling, but not
height? Something like "sampling in Y can be done by a client with a scanline
decoder, but sampling in X allows the swizzler to skip swizzling pixels and
reading from and writing to memory."

nit: this should be indented. (I do not have a strong preference for how much,
and I am not sure whether our guides tell us.)

[emmaleer, 2015/07/31 20:43:10]

Acknowledged.

+     *  Height sampling is not supported by the swizzler, but is implemented in SkScaledCodec.
      *  @return A new SkSwizzler or NULL on failure.
      */
     static SkSwizzler* CreateSwizzler(SrcConfig, const SkPMColor* ctable,
-                                      const SkImageInfo&, SkCodec::ZeroInitialized);
-
+                                      const SkImageInfo& dstInfo, SkCodec::ZeroInitialized, 
+                                      int srcWidth);
     /**
      * Fill the remainder of the destination with a single color
      *
      * @param dstStartRow
      * The destination row to fill from.
      *
      * @param numRows
      * The number of rows to fill.
      *
      * @param colorOrIndex
@@ skipping 52 matching lines @@
                                    const SkPMColor ctable[]);
 
     const RowProc       fRowProc;
     const SkPMColor*    fColorTable;      // Unowned pointer
     const int           fDeltaSrc;        // if bitsPerPixel % 8 == 0
                                           //     deltaSrc is bytesPerPixel
                                           // else
                                           //     deltaSrc is bitsPerPixel
     const SkImageInfo   fDstInfo;
     int                 fCurrY;
+    const int           fX0;              // first X coord to sample
+    const int           fSampleX;         // step between X samples
 
-    SkSwizzler(RowProc proc, const SkPMColor* ctable, int deltaSrc,
-               const SkImageInfo& info);
-
+    SkSwizzler(RowProc proc, const SkPMColor* ctable, int deltaSrc, const SkImageInfo& info, 
+               int sampleX);
 };
 #endif // SkSwizzler_DEFINED