[FFmpeg-devel] [PATCH] Optimize QTRLE encoding

Alexis Ballier alexis.ballier at gmail.com
Fri Feb 15 21:12:32 CET 2013


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 ^^


More information about the ffmpeg-devel mailing list