From 92dc0f09375051fb8ea29779aeb55ed5d1bdf580 Mon Sep 17 00:00:00 2001 From: Pascal Massimino Date: Tue, 2 Oct 2018 09:32:54 +0200 Subject: [PATCH] clean-up MakeInputImageCopy() use pointer increments. Change-Id: I269412d41a58ab9ffd7fc0f3d479fe73a3d07b9e --- src/enc/vp8l_enc.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) diff --git a/src/enc/vp8l_enc.c b/src/enc/vp8l_enc.c index a89184eb..b7411bf4 100644 --- a/src/enc/vp8l_enc.c +++ b/src/enc/vp8l_enc.c @@ -1248,14 +1248,20 @@ static WebPEncodingError MakeInputImageCopy(VP8LEncoder* const enc) { const WebPPicture* const picture = enc->pic_; const int width = picture->width; const int height = picture->height; - int y; + err = AllocateTransformBuffer(enc, width, height); if (err != VP8_ENC_OK) return err; if (enc->argb_content_ == kEncoderARGB) return VP8_ENC_OK; - for (y = 0; y < height; ++y) { - memcpy(enc->argb_ + y * width, - picture->argb + y * picture->argb_stride, - width * sizeof(*enc->argb_)); + + { + uint32_t* dst = enc->argb_; + const uint32_t* src = picture->argb; + int y; + for (y = 0; y < height; ++y) { + memcpy(dst, src, width * sizeof(*dst)); + dst += width; + src += picture->argb_stride; + } } enc->argb_content_ = kEncoderARGB; assert(enc->current_width_ == width);