[FFmpeg-devel] [PATCH] Optimize QTRLE encoding

[Malcolm Bechard]

On Fri, Feb 15, 2013 at 3:12 PM, Alexis Ballier <alexis.ballier at gmail.com>wrote:

> looks good to me, thanks
>
> just one nitpick:
>
> @@ -164,6 +218,7 @@ static void qtrle_encode_line(QtrleEncContext *s,
> const AVFrame *p, int line, ui
>
>          total_repeat_cost = s->length_table[i + repeatcount] + 1 +
> s->pixel_size;
>
> +
>          /* skip code is free for the first pixel, it costs one byte
> for repeat and bulk copy
>           * so let's make it aware */
>          if (i == 0) {
>
>
> this hunk appears in your diff, I think some people would prefer if it
> were removed ^^
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

Ok, here's an update with that removed.

Malcolm
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Improve-QTRLE-encoding-performance-no-change-to-outp.patch
Type: application/octet-stream
Size: 5325 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130215/491e44ab/attachment.obj>