[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