lör 2019-08-03 klockan 01:49 +0200 skrev Michael Niedermayer:
> - uint32_t x, y, src_x, src_y;
> + uint32_t x, y, src_y;
> + int width = hnm->width;
>
> for (y = 0; y < hnm->height; y++) {
> + uint8_t *dst = hnm->processed + y * width;
> + const uint8_t *src = hnm->current;
> src_y = y - (y % 2);
> - src_x = src_y * hnm->width + (y % 2);
> - for (x = 0; x < hnm->width; x++) {
> - hnm->processed[(y * hnm->width) + x] = hnm-
> >current[src_x];
> - src_x += 2;
> + src += src_y * width + (y % 2);
> + for (x = 0; x < width; x++) {
> + dst[x] = *src;
> + src += 2;
Looks OK. Maybe telling the compiler that src and dst don't alias would
be worthwhile?
/Tomas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
