[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