[Ffmpeg-cvslog] CVS: ffmpeg/libavcodec mpegvideo.h, 1.237, 1.238 xvid_rc.c, 1.2, 1.3

[Michael Niedermayer CVS]

Update of /cvsroot/ffmpeg/ffmpeg/libavcodec
In directory mail:/var2/tmp/cvs-serv22324

Modified Files:
	mpegvideo.h xvid_rc.c 
Log Message:
various fixes for xvid_rc


Index: mpegvideo.h
===================================================================
RCS file: /cvsroot/ffmpeg/ffmpeg/libavcodec/mpegvideo.h,v
retrieving revision 1.237
retrieving revision 1.238
diff -u -d -r1.237 -r1.238
--- mpegvideo.h	5 Feb 2006 13:35:17 -0000	1.237
+++ mpegvideo.h	19 Mar 2006 11:49:37 -0000	1.238
@@ -123,6 +123,7 @@
 
     void *non_lavc_opaque;        ///< context for non lavc rc code (for example xvid)
     float dry_run_qscale;         ///< for xvid rc
+    int last_picture_number;      ///< for xvid rc
 }RateControlContext;
 
 /**

Index: xvid_rc.c
===================================================================
RCS file: /cvsroot/ffmpeg/ffmpeg/libavcodec/xvid_rc.c,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -d -r1.2 -r1.3
--- xvid_rc.c	18 Mar 2006 02:46:39 -0000	1.2
+++ xvid_rc.c	19 Mar 2006 11:49:37 -0000	1.3
@@ -44,7 +44,7 @@
         fd = mkstemp(tmp_name);
     }
 
-    for(i=0; i<s->rc_context.num_entries - s->max_b_frames; i++){
+    for(i=0; i<s->rc_context.num_entries; i++){
         static const char *frame_types = " ipbs";
         char tmp[256];
         RateControlEntry *rce;
@@ -53,8 +53,9 @@
 
         snprintf(tmp, sizeof(tmp), "%c %d %d %d %d %d %d\n",
             frame_types[rce->pict_type], (int)lrintf(rce->qscale / FF_QP2LAMBDA), rce->i_count, s->mb_num - rce->i_count - rce->skip_count,
-            rce->skip_count, (rce->i_tex_bits + rce->p_tex_bits + rce->mv_bits + rce->misc_bits+7)/8, (rce->header_bits+7)/8);
+            rce->skip_count, (rce->i_tex_bits + rce->p_tex_bits + rce->misc_bits+7)/8, (rce->header_bits+rce->mv_bits+7)/8);
 
+//av_log(NULL, AV_LOG_ERROR, "%s\n", tmp);
         write(fd, tmp, strlen(tmp));
     }
 
@@ -74,7 +75,11 @@
     xvid_plg_create.fincr= s->avctx->time_base.num;
     xvid_plg_create.param= &xvid_2pass2;
 
-    return xvid_plugin_2pass2(NULL, XVID_PLG_CREATE, &xvid_plg_create, &s->rc_context.non_lavc_opaque);
+    if(xvid_plugin_2pass2(NULL, XVID_PLG_CREATE, &xvid_plg_create, &s->rc_context.non_lavc_opaque)<0){
+        av_log(NULL, AV_LOG_ERROR, "xvid_plugin_2pass2 failed\n");
+        return -1;
+    }
+    return 0;
 }
 
 float ff_xvid_rate_estimate_qscale(MpegEncContext *s, int dry_run){
@@ -94,8 +99,8 @@
     xvid_plg_data.max_quant[0]= s->avctx->qmax;
     xvid_plg_data.max_quant[1]= s->avctx->qmax;
     xvid_plg_data.max_quant[2]= s->avctx->qmax; //FIXME i/b factor & offset
-    xvid_plg_data.bquant_offset = 100 * s->avctx->b_quant_offset;
-    xvid_plg_data.bquant_ratio = 100 * s->avctx->b_quant_factor;
+    xvid_plg_data.bquant_offset = 0; //  100 * s->avctx->b_quant_offset;
+    xvid_plg_data.bquant_ratio = 100; // * s->avctx->b_quant_factor;
 
 #if 0
     xvid_plg_data.stats.hlength= X
@@ -105,7 +110,7 @@
         if(s->picture_number){
             xvid_plg_data.length=
             xvid_plg_data.stats.length= (s->frame_bits + 7)/8;
-            xvid_plg_data.frame_num= s->picture_number - 1;
+            xvid_plg_data.frame_num= s->rc_context.last_picture_number;
             xvid_plg_data.quant= s->qscale;
 
             xvid_plg_data.type= s->last_pict_type;
@@ -114,6 +119,7 @@
                 return -1;
             }
         }
+        s->rc_context.last_picture_number=
         xvid_plg_data.frame_num= s->picture_number;
         xvid_plg_data.quant= 0;
         if(xvid_plugin_2pass2(s->rc_context.non_lavc_opaque, XVID_PLG_BEFORE, &xvid_plg_data, NULL)){
@@ -126,7 +132,10 @@
     if(!dry_run)
         s->rc_context.dry_run_qscale= 0;
 
-    return xvid_plg_data.quant * FF_QP2LAMBDA;
+    if(s->pict_type == B_TYPE) //FIXME this is not exactly identical to xvid
+        return xvid_plg_data.quant * FF_QP2LAMBDA * s->avctx->b_quant_factor + s->avctx->b_quant_offset;
+    else
+        return xvid_plg_data.quant * FF_QP2LAMBDA;
 }
 
 void ff_xvid_rate_control_uninit(MpegEncContext *s){