Diff of /xvidcore/src/plugins/plugin_2pass2.c

-revision 1.1.2.8, Thu May 22 16:36:07 2003 UTC
+revision 1.8, Wed Jul  1 09:25:38 2009 UTC
 Line 1
  /******************************************************************************
   *
   * XviD Bit Rate Controller Library
-  * - VBR 2 pass bitrate controler implementation -
+  *  - VBR 2 pass bitrate controller implementation -
   *
   * Copyright (C)      2002 Foxer <email?>
   *                    2002 Dirk Knop <dknop@gwdg.de>
 Line 29
   *
   *****************************************************************************/
+ #define BQUANT_PRESCALE
+ #undef COMPENSATE_FORMULA
+ /* forces second pass not to be bigger than first */
+ #undef PASS_SMALLER
+ /* automatically alters overflow controls (strength and improvement/degradation)
+         to fight most common problems without user's knowladge */
+ #define SMART_OVERFLOW_SETTING
  #include <stdio.h>
  #include <math.h>
+ #include <limits.h>
- #define RAD2DEG 57.295779513082320876798154814105
- #define DEG2RAD 0.017453292519943295769236907684886
  #include "../xvid.h"
  #include "../image/image.h"
+ /*****************************************************************************
+  * Some default settings
+  ****************************************************************************/
+ #define DEFAULT_KEYFRAME_BOOST 0
+ #define DEFAULT_OVERFLOW_CONTROL_STRENGTH 10
+ #define DEFAULT_CURVE_COMPRESSION_HIGH 0
+ #define DEFAULT_CURVE_COMPRESSION_LOW 0
+ #define DEFAULT_MAX_OVERFLOW_IMPROVEMENT 10
+ #define DEFAULT_MAX_OVERFLOW_DEGRADATION 10
+ /* Keyframe settings */
+ #define DEFAULT_KFREDUCTION 20
+ #define DEFAULT_KFTHRESHOLD 1
+ /*****************************************************************************
+  * Some default constants (can be tuned)
+  ****************************************************************************/
+ /* Specify the invariant part of the headers bits (header+MV)
+  * as  hlength/cst */
+ #define INVARIANT_HEADER_PART_IVOP 1 /* factor 1.0f   */
+ #define INVARIANT_HEADER_PART_PVOP 2 /* factor 0.5f   */
+ #define INVARIANT_HEADER_PART_BVOP 8 /* factor 0.125f */
+ /*****************************************************************************
+  * Structures
+  ****************************************************************************/
+ /* Statistics */
  typedef struct {
      int type;               /* first pass type */
      int quant;              /* first pass quant */
          int blks[3];                    /* k,m,y blks */
      int length;             /* first pass length */
+         int invariant;          /* what we assume as being invariant between the two passes, it's a sub part of header + MV bits */
      int scaled_length;      /* scaled length */
-     int desired_length;     /* desired length; calcuated during encoding */
+         int desired_length;     /* desired length; calculated during encoding */
+         int error;
      int zone_mode;   /* XVID_ZONE_xxx */
      double weight;
- } stat_t;
+ } twopass_stat_t;
- /* context struct */
+ /* Context struct */
  typedef struct
  {
      xvid_plugin_2pass2_t param;
-     /* constant statistical data */
+         /*----------------------------------
+          * constant statistical data
+          *--------------------------------*/
+         /* Number of frames of the sequence */
          int num_frames;
+         /* Number of Intra frames of the sequence */
      int num_keyframes;
-     uint64_t target;    /* target filesize */
-     int count[3];   /* count of each frame types */
+         /* Target filesize to reach */
-     uint64_t tot_length[3];  /* total length of each frame types */
+         uint64_t target;
-     double avg_length[3];   /* avg */
-     int min_length[3];  /* min frame length of each frame types */
-     uint64_t tot_scaled_length[3];  /* total scaled length of each frame type */
-     int max_length;     /* max frame size */
-     /* zone statistical data */
-     double avg_weight;  /* average weight */
-     int64_t tot_quant;   /* total length used by XVID_ZONE_QUANT zones */
-     double curve_comp_scale;
-     double movie_curve;
-         double alt_curve_low;
-         double alt_curve_high;
-         double alt_curve_low_diff;
-         double alt_curve_high_diff;
-     double alt_curve_curve_bias_bonus;
-         double alt_curve_mid_qual;
-         double alt_curve_qual_dev;
-     /* dynamic */
+         /* Count of each frame types */
+         int count[3];
+         /* Total length of each frame types (1st pass) */
+         uint64_t tot_length[3];
+         uint64_t tot_invariant[3];
+         /* Average length of each frame types (used first for 1st pass data and
+          * then for scaled averages */
+         double avg_length[3];
+         /* Minimum frame length allowed for each frame type */
+         int min_length[3];
+         /* Total bytes per frame type once the curve has been scaled
+          * NB: advanced parameters do not change this value. This field
+          *     represents the total scaled w/o any advanced settings */
+         uint64_t tot_scaled_length[3];
+         /* Maximum observed frame size observed during the first pass, the RC
+          * will try tp force all frame sizes in the second pass to be under that
+          * limit */
+         int max_length;
+         /*----------------------------------
+          * Zones statistical data
+          *--------------------------------*/
+         /* Total length used by XVID_ZONE_QUANT zones */
+         uint64_t tot_quant;
+         uint64_t tot_quant_invariant;
+         /* Holds the total amount of frame bytes, zone weighted (only scalable
+          * part of frame bytes) */
+         uint64_t tot_weighted;
+         /*----------------------------------
+          * Advanced settings helper ratios
+          *--------------------------------*/
+         /* This the ratio that has to be applied to all p/b frames in order
+          * to reserve/retrieve bits for/from keyframe boosting and consecutive
+          * keyframe penalty */
+         double pb_iboost_tax_ratio;
+         /* This the ratio to apply to all b/p frames in order to respect the
+          * assymetric curve compression while respecting a target filesize
+          * NB: The assymetric delta gain has to be computed before this ratio
+          *     is applied, and then the delta is added to the scaled size */
+         double assymetric_tax_ratio;
+         /*----------------------------------
+          * Data from the stats file kept
+          * into RAM for easy access
+          *--------------------------------*/
+         /* Array of keyframe locations
+          * eg: rc->keyframe_locations[100] returns the frame number of the 100th
+          *     keyframe */
      int * keyframe_locations;
-     stat_t * stats;
-     double pquant_error[32];
+         /* Index of the last keyframe used in the keyframe_location */
-     double bquant_error[32];
+         int KF_idx;
-     int quant_count[32];
+         /* Array of all 1st pass data file -- see the twopass_stat_t structure
+          * definition for more details */
+         twopass_stat_t * stats;
+         /*----------------------------------
+          * Hysteresis helpers
+          *--------------------------------*/
+         /* This field holds the int2float conversion errors of each quant per
+          * frame type, this allow the RC to keep track of rouding error and thus
+          * increase or decrease the chosen quant according to this residue */
+         double quant_error[3][32];
+         /* This fields stores the count of each quant usage per frame type
+          * No real role but for debugging */
+         int quant_count[3][32];
+         /* Last valid quantizer used per frame type, it allows quantizer
+          * increament/decreament limitation in order to avoid big image quality
+          * "jumps" */
      int last_quant[3];
-     double curve_comp_error;
+         /*----------------------------------
-     int overflow;
+          * Overflow control
-     int KFoverflow;
+          *--------------------------------*/
-     int KFoverflow_partial;
-     int KF_idx;
+         /* Current overflow that has to be distributed to p/b frames */
+         double overflow;
+         /* Total overflow for keyframes -- not distributed directly */
+         double KFoverflow;
+         /* Amount of keyframe overflow to introduce to the global p/b frame
+          * overflow counter at each encoded frame */
+         double KFoverflow_partial;
+         /* Unknown ???
+          * ToDo: description */
      double fq_error;
- } rc_2pass2_t;
+         int min_quant; /* internal minimal quant, prevents wrong quants from being used */
+         /*----------------------------------
- #define BUF_SZ 1024
+          * Debug
- #define MAX_COLS    5
+          *--------------------------------*/
+         double desired_total;
+         double real_total;
+ } rc_2pass2_t;
- /* open stats file, and count num frames */
+ /*****************************************************************************
+  * Sub plugin functions prototypes
+  ****************************************************************************/
+ static int rc_2pass2_create(xvid_plg_create_t * create, rc_2pass2_t ** handle);
+ static int rc_2pass2_before(rc_2pass2_t * rc, xvid_plg_data_t * data);
+ static int rc_2pass2_after(rc_2pass2_t * rc, xvid_plg_data_t * data);
+ static int rc_2pass2_destroy(rc_2pass2_t * rc, xvid_plg_destroy_t * destroy);
+ /*****************************************************************************
+  * Plugin definition
+  ****************************************************************************/
- static int det_stats_length(rc_2pass2_t * rc, char * filename)
+ int
+ xvid_plugin_2pass2(void * handle, int opt, void * param1, void * param2)
  {
-     FILE * f;
+         switch(opt) {
-     int n, ignore;
+         case XVID_PLG_INFO :
-     char type;
+         case XVID_PLG_FRAME :
-     rc->num_frames = 0;
-     rc->num_keyframes = 0;
-     if ((f = fopen(filename, "rt")) == NULL)
          return 0;
-     while((n = fscanf(f, "%c %d %d %d %d %d %d\n",
+         case XVID_PLG_CREATE :
-         &type, &ignore, &ignore, &ignore, &ignore, &ignore, &ignore)) != EOF) {
+                 return rc_2pass2_create((xvid_plg_create_t*)param1, param2);
-         if (type == 'i') {
-             rc->num_frames++;
-             rc->num_keyframes++;
-         }else if (type == 'p' || type == 'b' || type == 's') {
-             rc->num_frames++;
-         }
-     }
-     fclose(f);
+         case XVID_PLG_DESTROY :
+                 return rc_2pass2_destroy((rc_2pass2_t*)handle, (xvid_plg_destroy_t*)param1);
+         case XVID_PLG_BEFORE :
+                 return rc_2pass2_before((rc_2pass2_t*)handle, (xvid_plg_data_t*)param1);
-     return 1;
+         case XVID_PLG_AFTER :
+                 return rc_2pass2_after((rc_2pass2_t*)handle, (xvid_plg_data_t*)param1);
  }
+         return XVID_ERR_FAIL;
+ }
+ /*****************************************************************************
+  * Sub plugin functions definitions
+  ****************************************************************************/
+ /* First a few local helping function prototypes */
+ static  int statsfile_count_frames(rc_2pass2_t * rc, char * filename);
+ static  int statsfile_load(rc_2pass2_t *rc, char * filename);
+ static void zone_process(rc_2pass2_t *rc, const xvid_plg_create_t * create);
+ static void first_pass_stats_prepare_data(rc_2pass2_t * rc);
+ static void first_pass_scale_curve_internal(rc_2pass2_t *rc);
+ static void scaled_curve_apply_advanced_parameters(rc_2pass2_t * rc);
+ static  int check_curve_for_vbv_compliancy(rc_2pass2_t * rc, const float fps);
+ static  int scale_curve_for_vbv_compliancy(rc_2pass2_t * rc, const float fps);
+ #if 0
+ static void stats_print(rc_2pass2_t * rc);
+ #endif
- /* open stats file(s) and read into rc->stats array */
+ /*----------------------------------------------------------------------------
+  *--------------------------------------------------------------------------*/
- static int load_stats(rc_2pass2_t *rc, char * filename)
+ static int
+ rc_2pass2_create(xvid_plg_create_t * create, rc_2pass2_t **handle)
  {
-     FILE * f;
+         xvid_plugin_2pass2_t * param = (xvid_plugin_2pass2_t *)create->param;
-     int i, not_scaled;
+         rc_2pass2_t * rc;
+         int i;
-     if ((f = fopen(filename, "rt"))==NULL)
+         rc = malloc(sizeof(rc_2pass2_t));
-         return 0;
+         if (rc == NULL)
+                 return XVID_ERR_MEMORY;
-     i = 0;
+         /* v1.0.x */
-         not_scaled = 0;
+         rc->param.version = param->version;
-     while(i < rc->num_frames) {
+         rc->param.bitrate = param->bitrate;
-         stat_t * s = &rc->stats[i];
+         rc->param.filename = param->filename;
-         int n;
+         rc->param.keyframe_boost = param->keyframe_boost;
-         char type;
+         rc->param.curve_compression_high = param->curve_compression_high;
+         rc->param.curve_compression_low = param->curve_compression_low;
+         rc->param.overflow_control_strength = param->overflow_control_strength;
+         rc->param.max_overflow_improvement = param->max_overflow_improvement;
+         rc->param.max_overflow_degradation = param->max_overflow_degradation;
+         rc->param.kfreduction = param->kfreduction;
+         rc->param.kfthreshold = param->kfthreshold;
+         rc->param.container_frame_overhead = param->container_frame_overhead;
+         if (XVID_VERSION_MINOR(param->version) >= 1) {
+                 rc->param.vbv_size = param->vbv_size;
+                 rc->param.vbv_initial = param->vbv_initial;
+                 rc->param.vbv_maxrate = param->vbv_maxrate;
+                 rc->param.vbv_peakrate = param->vbv_peakrate;
+         }else{
+                 rc->param.vbv_size =
+                 rc->param.vbv_initial =
+                 rc->param.vbv_maxrate =
+                 rc->param.vbv_peakrate = 0;
+         }
+         /* Initialize all defaults */
+ #define _INIT(a, b) if((a) <= 0) (a) = (b)
+         /* Let's set our defaults if needed */
+         _INIT(rc->param.keyframe_boost, DEFAULT_KEYFRAME_BOOST);
+         _INIT(rc->param.overflow_control_strength, DEFAULT_OVERFLOW_CONTROL_STRENGTH);
+         _INIT(rc->param.curve_compression_high, DEFAULT_CURVE_COMPRESSION_HIGH);
+         _INIT(rc->param.curve_compression_low, DEFAULT_CURVE_COMPRESSION_LOW);
+         _INIT(rc->param.max_overflow_improvement, DEFAULT_MAX_OVERFLOW_IMPROVEMENT);
+         _INIT(rc->param.max_overflow_degradation,  DEFAULT_MAX_OVERFLOW_DEGRADATION);
+         /* Keyframe settings */
+         _INIT(rc->param.kfreduction, DEFAULT_KFREDUCTION);
+         _INIT(rc->param.kfthreshold, DEFAULT_KFTHRESHOLD);
+ #undef _INIT
-                 s->scaled_length = 0;
+         /* Initialize some stuff to zero */
-         n = fscanf(f, "%c %d %d %d %d %d %d\n", &type, &s->quant, &s->blks[0], &s->blks[1], &s->blks[2], &s->length, &s->scaled_length);
+         for(i=0; i<3; i++) {
-         if (n == EOF) break;
+                 int j;
-                 if (n < 7) {
+                 for (j=0; j<32; j++) {
-                         not_scaled = 1;
+                         rc->quant_error[i][j] = 0;
+                         rc->quant_count[i][j] = 0;
                  }
-         if (type == 'i') {
-             s->type = XVID_TYPE_IVOP;
-         }else if (type == 'p' || type == 's') {
-             s->type = XVID_TYPE_PVOP;
-         }else if (type == 'b') {
-             s->type = XVID_TYPE_BVOP;
-         }else{  /* unknown type */
-             DPRINTF(XVID_DEBUG_RC, "unknown stats frame type; assuming pvop\n");
-             s->type = XVID_TYPE_PVOP;
          }
-         i++;
+         for (i=0; i<3; i++) rc->last_quant[i] = 0;
-     }
-     rc->num_frames = i;
+         rc->fq_error = 0;
+         rc->min_quant = 1;
-         fclose(f);
+         /* Count frames (and intra frames) in the stats file, store the result into
+          * the rc structure */
+         if (statsfile_count_frames(rc, param->filename) == -1) {
+                 DPRINTF(XVID_DEBUG_RC,"[xvid rc] -- ERROR: fopen %s failed\n", param->filename);
+                 free(rc);
+                 return(XVID_ERR_FAIL);
+         }
+         /* Allocate the stats' memory */
+         if ((rc->stats = malloc(rc->num_frames * sizeof(twopass_stat_t))) == NULL) {
+                 free(rc);
+                 return(XVID_ERR_MEMORY);
+         }
-     return 1;
+         /* Allocate keyframes location's memory
+          * PS: see comment in pre_process0 for the +1 location requirement */
+         rc->keyframe_locations = malloc((rc->num_keyframes + 1) * sizeof(int));
+         if (rc->keyframe_locations == NULL) {
+                 free(rc->stats);
+                 free(rc);
+                 return(XVID_ERR_MEMORY);
  }
+         /* Load the first pass stats */
+         if (statsfile_load(rc, param->filename) == -1) {
+                 DPRINTF(XVID_DEBUG_RC,"[xvid rc] -- ERROR: fopen %s failed\n", param->filename);
+                 free(rc->keyframe_locations);
+                 free(rc->stats);
+                 free(rc);
+                 return XVID_ERR_FAIL;
+         }
+         /* Compute the target filesize */
+         if (rc->param.bitrate<0) {
+                 /* if negative, bitrate equals the target (in kbytes) */
+                 rc->target = ((uint64_t)(-rc->param.bitrate)) * 1024;
+         } else if (rc->num_frames  < create->fbase/create->fincr) {
+                 /* Source sequence is less than 1s long, we do as if it was 1s long */
+                 rc->target = rc->param.bitrate / 8;
+         } else {
+                 /* Target filesize = bitrate/8 * numframes / framerate */
+                 rc->target =
+                         ((uint64_t)rc->param.bitrate * (uint64_t)rc->num_frames * \
+                          (uint64_t)create->fincr) / \
+                         ((uint64_t)create->fbase * 8);
+         }
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Frame rate: %d/%d (%ffps)\n",
+                         create->fbase, create->fincr,
+                         (double)create->fbase/(double)create->fincr);
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Number of frames: %d\n", rc->num_frames);
+         if(rc->param.bitrate>=0)
+                 DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Target bitrate: %ld\n", rc->param.bitrate);
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Target filesize: %lld\n", rc->target);
+         /* Compensate the average frame overhead caused by the container */
+         rc->target -= rc->num_frames*rc->param.container_frame_overhead;
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Container Frame overhead: %d\n", rc->param.container_frame_overhead);
+         if(rc->param.container_frame_overhead)
+                 DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- New target filesize after container compensation: %lld\n", rc->target);
- #if 0
+         /* When bitrate is not given it means it has been scaled by an external
- static void print_stats(rc_2pass2_t * rc)
+          * application */
- {
+         if (rc->param.bitrate) {
-     int i;
+                 /* Apply zone settings
+                  * - set rc->tot_quant which represents the total num of bytes spent in
+                  *   fixed quant zones
+                  * - set rc->tot_weighted which represents the total amount of bytes
+                  *   spent in normal or weighted zones in first pass (normal zones can
+                  *   be considered weight=1)
+                  * - set rc->tot_quant_invariant which represents the total num of bytes
+                  *   spent in fixed quant zones for headers */
+                 zone_process(rc, create);
+         } else {
+                 /* External scaling -- zones are ignored */
      for (i = 0; i < rc->num_frames; i++) {
-         stat_t * s = &rc->stats[i];
+                         rc->stats[i].zone_mode = XVID_ZONE_WEIGHT;
-         DPRINTF(XVID_DEBUG_RC, "%i %i %i %i\n", s->type, s->quant, s->length, s->scaled_length);
+                         rc->stats[i].weight = 1.0;
      }
+                 rc->tot_quant = 0;
  }
- #endif
- /* pre-process the statistics data
+         /* Gathers some information about first pass stats:
-     - for each type, count, tot_length, min_length, max_length
+          *  - finds the minimum frame length for each frame type during 1st pass.
-     - set keyframes_locations
+          *     rc->min_size[]
+          *  - determines the maximum frame length observed (no frame type distinction).
+          *     rc->max_size
+          *  - count how many times each frame type has been used.
+          *     rc->count[]
+          *  - total bytes used per frame type
+          *     rc->tot_length[]
+          *  - total bytes considered invariant between the 2 passes
+          *  - store keyframe location
+          *     rc->keyframe_locations[]
  */
+         first_pass_stats_prepare_data(rc);
- void pre_process0(rc_2pass2_t * rc)
+         /* If we have a user bitrate, it means it's an internal curve scaling */
- {
+         if (rc->param.bitrate) {
-     int i,j;
+                 /* Perform internal curve scaling */
+                 first_pass_scale_curve_internal(rc);
-     for (i=0; i<3; i++) {
-         rc->count[i]=0;
-         rc->tot_length[i] = 0;
-         rc->last_quant[i] = 0;
      }
-     for (i=j=0; i<rc->num_frames; i++) {
+         /* Apply advanced curve options, and compute some parameters in order to
-         stat_t * s = &rc->stats[i];
+          * shape the curve in the BEFORE/AFTER pair of functions */
+         scaled_curve_apply_advanced_parameters(rc);
+         /* Check curve for VBV compliancy and rescale if necessary */
+ #ifdef VBV_FORCE
+         if (rc->param.vbv_size==0) {
+                 rc->param.vbv_size      =  3145728;
+                 rc->param.vbv_initial   =  2359296;
+                 rc->param.vbv_maxrate   =  4854000;
+                 rc->param.vbv_peakrate  =  8000000;
+         }
+ #endif
-         rc->count[s->type-1]++;
+         /* vbv_size==0 switches VBV check off */
-         rc->tot_length[s->type-1] += s->length;
+         if (rc->param.vbv_size > 0)  {
+                 const double fps = (double)create->fbase/(double)create->fincr;
+                 int status = check_curve_for_vbv_compliancy(rc, fps);
-         if (i == 0 || s->length < rc->min_length[s->type-1]) {
+                 if (status) {
-             rc->min_length[s->type-1] = s->length;
+                         DPRINTF(XVID_DEBUG_RC, "[xvid rc] Underflow detected - Scaling Curve for compliancy.\n");
          }
-         if (i == 0 || s->length > rc->max_length) {
+                 status = scale_curve_for_vbv_compliancy(rc, fps);
-             rc->max_length = s->length;
-         }
-         if (s->type == XVID_TYPE_IVOP) {
+                 if (status == 0) {
-             rc->keyframe_locations[j] = i;
+                         DPRINTF(XVID_DEBUG_RC, "[xvid rc] VBV compliant curve scaling done.\n");
-             j++;
+                 } else {
+                         DPRINTF(XVID_DEBUG_RC, "[xvid rc] VBV compliant curve scaling impossible.\n");
          }
      }
+         *handle = rc;
-         /*
+         return(0);
-          * The "per sequence" overflow system considers a natural sequence to be
-          * formed by all frames between two iframes, so if we want to make sure
-          * the system does not go nuts during last sequence, we force the last
-          * frame to appear in the keyframe locations array.
-          */
-     rc->keyframe_locations[j] = i;
  }
+ /*----------------------------------------------------------------------------
+  *--------------------------------------------------------------------------*/
- /* calculate zone weight "center" */
+ static int
+ rc_2pass2_destroy(rc_2pass2_t * rc, xvid_plg_destroy_t * destroy)
- static void zone_process(rc_2pass2_t *rc, const xvid_plg_create_t * create)
  {
-     int i,j;
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- target_total:%lld desired_total:%.2f (%.2f%%) actual_total:%.2f (%.2f%%)\n",
-     int n = 0;
+                         rc->target,
+                         rc->desired_total,
-     rc->avg_weight = 0.0;
+*rc->desired_total/(double)rc->target,
-     rc->tot_quant = 0;
+                         rc->real_total,
+*rc->real_total/(double)rc->target);
+         free(rc->keyframe_locations);
-     if (create->num_zones == 0) {
+         free(rc->stats);
-         for (j = 0; j < rc->num_frames; j++) {
+         free(rc);
-             rc->stats[j].zone_mode = XVID_ZONE_WEIGHT;
+         return(0);
-             rc->stats[j].weight = 1.0;
-         }
-         rc->avg_weight += rc->num_frames * 1.0;
-         n += rc->num_frames;
      }
+ /*----------------------------------------------------------------------------
+  *--------------------------------------------------------------------------*/
-     for(i=0; i < create->num_zones; i++) {
+ static int
+ rc_2pass2_before(rc_2pass2_t * rc, xvid_plg_data_t * data)
+ {
+         twopass_stat_t * s = &rc->stats[data->frame_num];
+         double dbytes;
+         double scaled_quant;
+         double overflow;
+         int capped_to_max_framesize = 0;
-         int next = (i+1<create->num_zones) ? create->zones[i+1].frame : rc->num_frames;
+         /* This function is quite long but easy to understand. In order to simplify
+          * the code path (a bit), we treat 3 cases that can return immediatly. */
-         if (i==0 && create->zones[i].frame > 0) {
+         /* First case: Another plugin has already set a quantizer */
-             for (j = 0; j < create->zones[i].frame && j < rc->num_frames; j++) {
+         if (data->quant > 0)
-                 rc->stats[j].zone_mode = XVID_ZONE_WEIGHT;
+                 return(0);
-                 rc->stats[j].weight = 1.0;
-             }
-             rc->avg_weight += create->zones[i].frame * 1.0;
-             n += create->zones[i].frame;
-         }
-         if (create->zones[i].mode == XVID_ZONE_WEIGHT) {
+         /* Second case: insufficent stats data
-             for (j = create->zones[i].frame; j < next && j < rc->num_frames; j++ ) {
+          * We can't guess much what we should do, let core decide all alone */
-                 rc->stats[j].zone_mode = XVID_ZONE_WEIGHT;
+         if (data->frame_num >= rc->num_frames) {
-                 rc->stats[j].weight = (double)create->zones[i].increment / (double)create->zones[i].base;
+                 DPRINTF(XVID_DEBUG_RC,"[xvid rc] -- stats file too short (now processing frame %d)",
-             }
+                         data->frame_num);
-             next -= create->zones[i].frame;
+                 return(0);
-             rc->avg_weight += (double)(next * create->zones[i].increment) / (double)create->zones[i].base;
-             n += next;
-         }else{  // XVID_ZONE_QUANT
-             for (j = create->zones[i].frame; j < next && j < rc->num_frames; j++ ) {
-                 rc->stats[j].zone_mode = XVID_ZONE_QUANT;
-                 rc->stats[j].weight = (double)create->zones[i].increment / (double)create->zones[i].base;
-                 rc->tot_quant += rc->stats[j].length;
-             }
-         }
      }
-     rc->avg_weight = n>0 ? rc->avg_weight/n : 1.0;
-     DPRINTF(XVID_DEBUG_RC, "center_weight: %f (for %i frames);   fixed_bytes: %i\n", rc->avg_weight, n, rc->tot_quant);
+         /* Third case: We are in a Quant zone
- }
+          * Quant zones must just ensure we use the same settings as first pass
+          * So set the quantizer and the type */
+         if (s->zone_mode == XVID_ZONE_QUANT) {
+                 /* Quant stuff */
+                 rc->fq_error += s->weight;
+                 data->quant = (int)rc->fq_error;
+                 rc->fq_error -= data->quant;
+                 /* The type stuff */
+                 data->type = s->type;
- /* scale the curve */
+                 /* The only required data for AFTER step is this one for the overflow
+                  * control */
+                 s->desired_length = s->length;
- static void internal_scale(rc_2pass2_t *rc)
+                 return(0);
- {
+         }
-         int64_t target  = rc->target - rc->tot_quant;
-         int64_t pass1_length = rc->tot_length[0] + rc->tot_length[1] + rc->tot_length[2] - rc->tot_quant;
-         int min_size[3];
-         double scaler;
-         int i;
-         /* perform an initial scale pass.
+         /*************************************************************************/
-            if a frame size is scaled underneath our hardcoded minimums, then we force the
+         /*************************************************************************/
-            frame size to the minimum, and deduct the original & scaled frmae length from the
+         /*************************************************************************/
-            original and target total lengths */
-         min_size[0] = ((rc->stats[0].blks[0]*22) + 240) / 8;
+         /*-------------------------------------------------------------------------
-         min_size[1] = (rc->stats[0].blks[0] + 88) / 8;
+          * Frame bit allocation first part
-         min_size[2] = 8;
+          *
+          * First steps apply user settings, just like it is done in the theoritical
+          * scaled_curve_apply_advanced_parameters
+          *-----------------------------------------------------------------------*/
-         scaler = (double)target / (double)pass1_length;
+         /* Set desired to what we are wanting to obtain for this frame */
+         dbytes = (double)s->scaled_length;
-         if (target <= 0 || pass1_length <= 0 || target >= pass1_length) {
+         /* IFrame user settings*/
-                 DPRINTF(XVID_DEBUG_RC, "undersize warning\n");
+         if (s->type == XVID_TYPE_IVOP) {
-         scaler = 1.0;
+                 /* Keyframe boosting -- All keyframes benefit from it */
-         }
+                 dbytes += dbytes*rc->param.keyframe_boost / 100;
-     DPRINTF(XVID_DEBUG_RC, "target=%i, tot_length=%i, scaler=%f\n", (int)target, (int)pass1_length, scaler);
+ #if 0 /* ToDo: decide how to apply kfthresholding */
+ #endif
+         } else {
-         for (i=0; i<rc->num_frames; i++) {
+                 /* P/S/B frames must reserve some bits for iframe boosting */
-                 stat_t * s = &rc->stats[i];
+                 dbytes *= rc->pb_iboost_tax_ratio;
-                 int len;
-         if (s->zone_mode == XVID_ZONE_QUANT) {
+                 /* Apply assymetric curve compression */
-             s->scaled_length = s->length;
+                 if (rc->param.curve_compression_high || rc->param.curve_compression_low) {
-         }else {
+                         double assymetric_delta;
-                     len = (int)((double)s->length * scaler * s->weight / rc->avg_weight);
-                     if (len < min_size[s->type-1]) {            /* force frame size */
+                         /* Compute the assymetric delta, this is computed before applying
-                             s->scaled_length = min_size[s->type-1];
+                          * the tax, as done in the pre_process function */
-                             target -= s->scaled_length;
+                         if (dbytes > rc->avg_length[s->type-1])
-                             pass1_length -= s->length;
+                                 assymetric_delta = (rc->avg_length[s->type-1] - dbytes) * rc->param.curve_compression_high / 100.0;
-                     }else{
+                         else
-                             s->scaled_length = 0;
+                                 assymetric_delta = (rc->avg_length[s->type-1] - dbytes) * rc->param.curve_compression_low  / 100.0;
-                     }
+                         /* Now we must apply the assymetric tax, else our curve compression
+                          * would not give a theoritical target size equal to what it is
+                          * expected */
+                         dbytes *= rc->assymetric_tax_ratio;
+                         /* Now we can add the assymetric delta */
+                         dbytes += assymetric_delta;
          }
          }
-     scaler = (double)target / (double)pass1_length;
+         /* That is what we would like to have -- Don't put that chunk after
-     if (target <= 0 || pass1_length <= 0 || target >= pass1_length) {
+          * overflow control, otherwise, overflow is counted twice and you obtain
-                 DPRINTF(XVID_DEBUG_RC,"undersize warning\n");
+          * half sized bitrate sequences */
-                 scaler = 1.0;
+         s->desired_length  = (int)dbytes;
-         }
+         rc->desired_total += dbytes;
+         /*------------------------------------------------------------------------
+          * Frame bit allocation: overflow control part.
+          *
+          * Unlike the theoritical scaled_curve_apply_advanced_parameters, here
+          * it's real encoding and we need to make sure we don't go so far from
+          * what is our ideal scaled curve.
+          *-----------------------------------------------------------------------*/
+         /* Compute the overflow we should compensate */
+         if (s->type != XVID_TYPE_IVOP || rc->overflow > 0) {
+                 double frametype_factor;
+                 double framesize_factor;
-         DPRINTF(XVID_DEBUG_RC, "target=%i, tot_length=%i, scaler=%f\n", (int)target, (int)pass1_length, scaler);
+                 /* Take only the desired part of overflow */
+                 overflow = rc->overflow;
-         for (i=0; i<rc->num_frames; i++) {
+                 /* Factor that will take care to decrease the overflow applied
-                 stat_t * s = &rc->stats[i];
+                  * according to the importance of this frame type in term of
+                  * overall size */
+                 frametype_factor  = rc->count[XVID_TYPE_IVOP-1]*rc->avg_length[XVID_TYPE_IVOP-1];
+                 frametype_factor += rc->count[XVID_TYPE_PVOP-1]*rc->avg_length[XVID_TYPE_PVOP-1];
+                 frametype_factor += rc->count[XVID_TYPE_BVOP-1]*rc->avg_length[XVID_TYPE_BVOP-1];
+                 frametype_factor /= rc->count[s->type-1]*rc->avg_length[s->type-1];
+                 frametype_factor  = 1/frametype_factor;
+                 /* Factor that will take care not to compensate too much for this frame
+                  * size */
+                 framesize_factor  = dbytes;
+                 framesize_factor /= rc->avg_length[s->type-1];
-                 if (s->scaled_length==0) {      /* ignore frame with forced frame sizes */
+                 /* Treat only the overflow part concerned by this frame type and size */
-                         s->scaled_length = (int)((double)s->length * scaler * s->weight / rc->avg_weight);
+                 overflow *= frametype_factor;
-                 }
+ #if 0
+                 /* Leave this one alone, as it impacts badly on quality */
+                 overflow *= framesize_factor;
+ #endif
+                 /* Apply the overflow strength imposed by the user */
+                 overflow *= (rc->param.overflow_control_strength/100.0f);
+         } else {
+                 /* no negative overflow applied in IFrames because:
+                  *  - their role is important as they're references for P/BFrames.
+                  *  - there aren't much in typical sequences, so if an IFrame overflows too
+                  *    much, this overflow may impact the next IFrame too much and generate
+                  *    a sequence of poor quality frames */
+                 overflow = 0;
          }
+         /* Make sure we are not trying to compensate more overflow than we even have */
+         if (fabs(overflow) > fabs(rc->overflow))
+                 overflow = rc->overflow;
+         /* Make sure the overflow doesn't make the frame size to get out of the range
+          * [-max_degradation..+max_improvment] */
+         if (overflow > dbytes*rc->param.max_overflow_improvement / 100) {
+                 if(overflow <= dbytes)
+                         dbytes += dbytes * rc->param.max_overflow_improvement / 100;
+                 else
+                         dbytes += overflow * rc->param.max_overflow_improvement / 100;
+         } else if (overflow < - dbytes * rc->param.max_overflow_degradation / 100) {
+                 dbytes -= dbytes * rc->param.max_overflow_degradation / 100;
+         } else {
+                 dbytes += overflow;
  }
+         /*-------------------------------------------------------------------------
+          * Frame bit allocation last part:
+          *
+          * Cap frame length so we don't reach neither bigger frame sizes than first
+          * pass nor smaller than the allowed minimum.
+          *-----------------------------------------------------------------------*/
+ #ifdef PASS_SMALLER
+         if (dbytes > s->length) {
+                 dbytes = s->length;
+         }
+ #endif
+         /* Prevent stupid desired sizes under logical values */
+         if (dbytes < rc->min_length[s->type-1]) {
+                 dbytes = rc->min_length[s->type-1];
+         }
+         /*------------------------------------------------------------------------
+          * Desired frame length <-> quantizer mapping
+          *-----------------------------------------------------------------------*/
- void pre_process1(rc_2pass2_t * rc)
+ #ifdef BQUANT_PRESCALE
- {
+         /* For bframes we prescale the quantizer to avoid too high quant scaling */
-     int i;
+         if(s->type == XVID_TYPE_BVOP) {
-     double total1, total2;
-     uint64_t ivop_boost_total;
-     ivop_boost_total = 0;
+                 twopass_stat_t *b_ref = s;
-     rc->curve_comp_error = 0;
-     for (i=0; i<3; i++) {
+                 /* Find the reference frame */
-         rc->tot_scaled_length[i] = 0;
+                 while(b_ref != &rc->stats[0] && b_ref->type == XVID_TYPE_BVOP)
+                         b_ref--;
+                 /* Compute the original quant */
+                 s->quant  = 2*(100*s->quant - data->bquant_offset);
+                 s->quant += data->bquant_ratio - 1; /* to avoid rounding issues */
+                 s->quant  = s->quant/data->bquant_ratio - b_ref->quant;
      }
+ #endif
-     for (i=0; i<rc->num_frames; i++) {
+         /* Don't laugh at this very 'simple' quant<->size relationship, it
-         stat_t * s = &rc->stats[i];
+          * proves to be acurate enough for our algorithm */
+         scaled_quant = (double)s->quant*(double)s->length/(double)dbytes;
+ #ifdef COMPENSATE_FORMULA
+         /* We know xvidcore will apply the bframe formula again, so we compensate
+          * it right now to make sure we would not apply it twice */
+         if(s->type == XVID_TYPE_BVOP) {
-         rc->tot_scaled_length[s->type-1] += s->scaled_length;
+                 twopass_stat_t *b_ref = s;
-         if (s->type == XVID_TYPE_IVOP) {
+                 /* Find the reference frame */
-             ivop_boost_total += s->scaled_length * rc->param.keyframe_boost / 100;
+                 while(b_ref != &rc->stats[0] && b_ref->type == XVID_TYPE_BVOP)
-         }
+                         b_ref--;
+                 /* Compute the quant it would be if the core did not apply the bframe
+                  * formula */
+                 scaled_quant  = 100*scaled_quant - data->bquant_offset;
+                 scaled_quant += data->bquant_ratio - 1; /* to avoid rouding issues */
+                 scaled_quant /= data->bquant_ratio;
      }
+ #endif
-     rc->movie_curve = ((double)(rc->tot_scaled_length[XVID_TYPE_PVOP-1] + rc->tot_scaled_length[XVID_TYPE_BVOP-1] + ivop_boost_total) /
+         /* Quantizer has been scaled using floating point operations/results, we
-                                         (rc->tot_scaled_length[XVID_TYPE_PVOP-1] + rc->tot_scaled_length[XVID_TYPE_BVOP-1]));
+          * must cast it to integer */
+         data->quant = (int)scaled_quant;
-     for(i=0; i<3; i++) {
+         /* Let's clip the computed quantizer, if needed */
-         if (rc->count[i] == 0 || rc->movie_curve == 0) {
+         if (data->quant < 1) {
-             rc->avg_length[i] = 1;
+                 data->quant = 1;
+         } else if (data->quant > 31) {
+                 data->quant = 31;
          }else{
-             rc->avg_length[i] = rc->tot_scaled_length[i] / rc->count[i] / rc->movie_curve;
-         }
-     }
-     /* alt curve stuff here */
+                 /* The frame quantizer has not been clipped, this appears to be a good
+                  * computed quantizer, do not loose quantizer decimal part that we
+                  * accumulate for later reuse when its sum represents a complete
+                  * unit. */
+                 rc->quant_error[s->type-1][data->quant] += scaled_quant - (double)data->quant;
-     if (rc->param.use_alt_curve) {
+                 if (rc->quant_error[s->type-1][data->quant] >= 1.0) {
-         const double avg_pvop = rc->avg_length[XVID_TYPE_PVOP-1];
+                         rc->quant_error[s->type-1][data->quant] -= 1.0;
-         const uint64_t tot_pvop = rc->tot_length[XVID_TYPE_PVOP-1];
+                         data->quant++;
-         const uint64_t tot_bvop = rc->tot_length[XVID_TYPE_BVOP-1];
+                 } else if (rc->quant_error[s->type-1][data->quant] <= -1.0) {
-         const uint64_t tot_scaled_pvop = rc->tot_scaled_length[XVID_TYPE_PVOP-1];
+                         rc->quant_error[s->type-1][data->quant] += 1.0;
-         const uint64_t tot_scaled_bvop = rc->tot_scaled_length[XVID_TYPE_BVOP-1];
+                         data->quant--;
+                 }
+         }
-                 rc->alt_curve_low = avg_pvop - avg_pvop * (double)rc->param.alt_curve_low_dist / 100.0;
+         /* Now we have a computed quant that is in the right quante range, with a
-                 rc->alt_curve_low_diff = avg_pvop - rc->alt_curve_low;
+          * possible +1 correction due to cumulated error. We can now safely clip
-                 rc->alt_curve_high = avg_pvop + avg_pvop * (double)rc->param.alt_curve_high_dist / 100.0;
+          * the quantizer again with user's quant ranges. "Safely" means the Rate
-                 rc->alt_curve_high_diff = rc->alt_curve_high - avg_pvop;
+          * Control could learn more about this quantizer, this knowledge is useful
+          * for future frames even if it this quantizer won't be really used atm,
+          * that's why we don't perform this clipping earlier. */
+         if (data->quant < data->min_quant[s->type-1]) {
+                 data->quant = data->min_quant[s->type-1];
+         } else if (data->quant > data->max_quant[s->type-1]) {
+                 data->quant = data->max_quant[s->type-1];
+         }
-         if (rc->param.alt_curve_use_auto) {
+         if (data->quant < rc->min_quant) data->quant = rc->min_quant;
-             if (tot_bvop + tot_pvop > tot_scaled_bvop + tot_scaled_pvop) {
-                                 rc->param.alt_curve_min_rel_qual = (int)(100.0 - (100.0 - 100.0 /
-                                         ((double)(tot_pvop + tot_bvop) / (double)(tot_scaled_pvop + tot_scaled_bvop))) * (double)rc->param.alt_curve_auto_str / 100.0);
-                                 if (rc->param.alt_curve_min_rel_qual < 20)
+         /* To avoid big quality jumps from frame to frame, we apply a "security"
-                                         rc->param.alt_curve_min_rel_qual = 20;
+          * rule that makes |last_quant - new_quant| <= 2. This rule only applies
-             }else{
+          * to predicted frames (P and B) */
-                                 rc->param.alt_curve_min_rel_qual = 100;
+         if (s->type != XVID_TYPE_IVOP && rc->last_quant[s->type-1] && capped_to_max_framesize == 0) {
-             }
-         }
-                 rc->alt_curve_mid_qual = (1.0 + (double)rc->param.alt_curve_min_rel_qual / 100.0) / 2.0;
-                 rc->alt_curve_qual_dev = 1.0 - rc->alt_curve_mid_qual;
-         if (rc->param.alt_curve_low_dist > 100) {
+                 if (data->quant > rc->last_quant[s->type-1] + 2) {
-                         switch(rc->param.alt_curve_type) {
+                         data->quant = rc->last_quant[s->type-1] + 2;
-             case XVID_CURVE_SINE: // Sine Curve (high aggressiveness)
+                         DPRINTF(XVID_DEBUG_RC,
-                                 rc->alt_curve_qual_dev *= 2.0 / (1.0 + sin(DEG2RAD * (avg_pvop * 90.0 / rc->alt_curve_low_diff)));
+                                         "[xvid rc] -- frame %d p/b-frame quantizer prevented from rising too steeply\n",
-                                 rc->alt_curve_mid_qual = 1.0 - rc->alt_curve_qual_dev * sin(DEG2RAD * (avg_pvop * 90.0 / rc->alt_curve_low_diff));
+                                         data->frame_num);
-                                 break;
-                         case XVID_CURVE_LINEAR: // Linear (medium aggressiveness)
-                                 rc->alt_curve_qual_dev *= 2.0 / (1.0 + avg_pvop / rc->alt_curve_low_diff);
-                                 rc->alt_curve_mid_qual = 1.0 - rc->alt_curve_qual_dev * avg_pvop / rc->alt_curve_low_diff;
-                                 break;
-                         case XVID_CURVE_COSINE: // Cosine Curve (low aggressiveness)
-                                 rc->alt_curve_qual_dev *= 2.0 / (1.0 + (1.0 - cos(DEG2RAD * (avg_pvop * 90.0 / rc->alt_curve_low_diff))));
-                                 rc->alt_curve_mid_qual = 1.0 - rc->alt_curve_qual_dev * (1.0 - cos(DEG2RAD * (avg_pvop * 90.0 / rc->alt_curve_low_diff)));
                          }
+                 if (data->quant < rc->last_quant[s->type-1] - 2) {
+                         data->quant = rc->last_quant[s->type-1] - 2;
+                         DPRINTF(XVID_DEBUG_RC,
+                                         "[xvid rc] -- frame:%d p/b-frame quantizer prevented from falling too steeply\n",
+                                         data->frame_num);
                  }
      }
-     /* --- */
+         /* We don't want to pollute the RC histerisis when our computed quant has
+          * been computed from a capped frame size */
+         if (capped_to_max_framesize == 0)
+                 rc->last_quant[s->type-1] = data->quant;
-     total1=total2=0;
+         /* Don't forget to force 1st pass frame type ;-) */
-     for (i=0; i<rc->num_frames; i++) {
+         data->type = s->type;
-         stat_t * s = &rc->stats[i];
-         if (s->type != XVID_TYPE_IVOP) {
+         return 0;
-             double dbytes,dbytes2;
+ }
-             dbytes = s->scaled_length / rc->movie_curve;
+ /*----------------------------------------------------------------------------
-             dbytes2 = 0; /* XXX: warning */
+  *--------------------------------------------------------------------------*/
-             total1 += dbytes;
-             if (s->type == XVID_TYPE_BVOP)
-                 dbytes *= rc->avg_length[XVID_TYPE_PVOP-1] / rc->avg_length[XVID_TYPE_BVOP-1];
-             if (rc->param.use_alt_curve) {
+ static int
-                 if (dbytes > rc->avg_length[XVID_TYPE_PVOP-1]) {
+ rc_2pass2_after(rc_2pass2_t * rc, xvid_plg_data_t * data)
+ {
+         const char frame_type[4] = { 'i', 'p', 'b', 's'};
+         twopass_stat_t * s = &rc->stats[data->frame_num];
-                     if (dbytes >= rc->alt_curve_high) {
+         /* Insufficent stats data */
-                                                 dbytes2 = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev);
+         if (data->frame_num >= rc->num_frames)
-                     }else{
+                 return 0;
-                                                 switch(rc->param.alt_curve_type) {
-                         case XVID_CURVE_SINE :
-                                                     dbytes2 = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * sin(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_high_diff)));
-                                                         break;
-                         case XVID_CURVE_LINEAR :
-                                                     dbytes2 = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) / rc->alt_curve_high_diff);
-                                                         break;
-                                                 case XVID_CURVE_COSINE :
-                                                     dbytes2 = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (1.0 - cos(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_high_diff))));
-                                                 }
-                                         }
-                 }else{
-                     if (dbytes <= rc->alt_curve_low) {
-                                                 dbytes2 = dbytes;
-                     }else{
-                                                 switch(rc->param.alt_curve_type) {
-                                                 case XVID_CURVE_SINE :
-                                                     dbytes2 = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * sin(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_low_diff)));
-                                                         break;
-                                                 case XVID_CURVE_LINEAR :
-                                                     dbytes2 = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) / rc->alt_curve_low_diff);
-                                                         break;
-                                                 case XVID_CURVE_COSINE :
-                                                     dbytes2 = dbytes * (rc->alt_curve_mid_qual + rc->alt_curve_qual_dev * (1.0 - cos(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_low_diff))));
-                                                 }
-                                         }
-                 }
+         /* Update the quantizer counter */
+         rc->quant_count[s->type-1][data->quant]++;
+         /* Update the frame type overflow */
+         if (data->type == XVID_TYPE_IVOP) {
+                 int kfdiff = 0;
-             }else{
+                 if(rc->KF_idx != rc->num_frames -1) {
-                 if (dbytes > rc->avg_length[XVID_TYPE_PVOP-1]) {
+                         kfdiff  = rc->keyframe_locations[rc->KF_idx+1];
-                     dbytes2=((double)dbytes + (rc->avg_length[XVID_TYPE_PVOP-1] - dbytes) * rc->param.curve_compression_high / 100.0);
+                         kfdiff -= rc->keyframe_locations[rc->KF_idx];
-                 }else{
-                                 dbytes2 = ((double)dbytes + (rc->avg_length[XVID_TYPE_PVOP-1] - dbytes) * rc->param.curve_compression_low / 100.0);
-                 }
              }
-             if (s->type == XVID_TYPE_BVOP) {
+                 /* Flush Keyframe overflow accumulator */
-                             dbytes2 *= rc->avg_length[XVID_TYPE_BVOP-1] / rc->avg_length[XVID_TYPE_PVOP-1];
+                 rc->overflow += rc->KFoverflow;
-                             if (dbytes2 < rc->min_length[XVID_TYPE_BVOP-1])
-                                     dbytes2 = rc->min_length[XVID_TYPE_BVOP-1];
+                 /* Store the frame overflow to the keyframe accumulator */
+                 rc->KFoverflow = s->desired_length - data->length;
+                 if (kfdiff > 1) {
+                         /* Non-consecutive keyframes case:
+                          * We can then divide this total keyframe overflow into equal parts
+                          * that we will distribute into regular overflow at each frame
+                          * between the sequence bounded by two IFrames */
+                         rc->KFoverflow_partial = rc->KFoverflow / (kfdiff - 1);
              }else{
-                             if (dbytes2 < rc->min_length[XVID_TYPE_PVOP-1])
+                         /* Consecutive keyframes case:
-                                     dbytes2 = rc->min_length[XVID_TYPE_PVOP-1];
+                          * Flush immediatly the keyframe overflow and reset keyframe
-             }
+                          * overflow */
-             total2 += dbytes2;
+                         rc->overflow += rc->KFoverflow;
+                         rc->KFoverflow = 0;
+                         rc->KFoverflow_partial = 0;
          }
+                 rc->KF_idx++;
+         } else {
+                 /* Accumulate the frame overflow */
+                 rc->overflow += s->desired_length - data->length;
+                 /* Distribute part of the keyframe overflow */
+                 rc->overflow += rc->KFoverflow_partial;
+                 /* Don't forget to substract that same amount from the total keyframe
+                  * overflow */
+                 rc->KFoverflow -= rc->KFoverflow_partial;
      }
-     rc->curve_comp_scale = total1 / total2;
+         s->error = s->desired_length - data->length;
+         rc->real_total += data->length;
-     if (!rc->param.use_alt_curve) {
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- frame:%d type:%c quant:%d stats:%d scaled:%d desired:%d actual:%d error:%d overflow:%.2f\n",
-         DPRINTF(XVID_DEBUG_RC, "middle frame size for asymmetric curve compression: %i\n",
+                         data->frame_num,
-             (int)(rc->avg_length[XVID_TYPE_PVOP-1] * rc->curve_comp_scale));
+                         frame_type[data->type-1],
+                         data->quant,
+                         s->length,
+                         s->scaled_length,
+                         s->desired_length,
+                         s->desired_length - s->error,
+                         -s->error,
+                         rc->overflow);
+         return(0);
      }
-     if (rc->param.use_alt_curve) {
+ /*****************************************************************************
-         int bonus_bias = rc->param.alt_curve_bonus_bias;
+  * Helper functions definition
-         int oldquant = 1;
+  ****************************************************************************/
-             if (rc->param.alt_curve_use_auto_bonus_bias)
+ /* Default buffer size for reading lines */
-                     bonus_bias = rc->param.alt_curve_min_rel_qual;
+ #define BUF_SZ   1024
+ /* Helper functions for reading/parsing the stats file */
+ static char *skipspaces(char *string);
+ static int iscomment(char *string);
+ static char *readline(FILE *f);
+ /* This function counts the number of frame entries in the stats file
+  * It also counts the number of I Frames */
+ static int
+ statsfile_count_frames(rc_2pass2_t * rc, char * filename)
+ {
+         FILE * f;
+         char *line;
+         int lines;
-             rc->alt_curve_curve_bias_bonus = (total1 - total2) * (double)bonus_bias / 100.0 / (double)(rc->num_frames /* - credits_frames */ - rc->num_keyframes);
+         rc->num_frames = 0;
-             rc->curve_comp_scale = ((total1 - total2) * (1.0 - (double)bonus_bias / 100.0) + total2) / total2;
+         rc->num_keyframes = 0;
+         if ((f = fopen(filename, "rb")) == NULL)
+                 return(-1);
-         /* special info for alt curve:  bias bonus and quantizer thresholds */
+         lines = 0;
+         while ((line = readline(f)) != NULL) {
-                 DPRINTF(XVID_DEBUG_RC, "avg scaled framesize:%i\n", (int)rc->avg_length[XVID_TYPE_PVOP-1]);
+                 char *ptr;
-                 DPRINTF(XVID_DEBUG_RC, "bias bonus:%i bytes\n", (int)rc->alt_curve_curve_bias_bonus);
+                 char type;
+                 int fields;
-                 for (i=1; i <= (int)(rc->alt_curve_high*2)+1; i++) {
+                 lines++;
-             double curve_temp, dbytes;
-             int newquant;
-             dbytes = i;
+                 /* We skip spaces */
-                         if (dbytes > rc->avg_length[XVID_TYPE_PVOP-1]) {
+                 ptr = skipspaces(line);
-                 if (dbytes >= rc->alt_curve_high) {
-                                         curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev);
+                 /* Skip coment lines or empty lines */
-                 }else{
+                 if(iscomment(ptr) || *ptr == '\0') {
-                                         switch(rc->param.alt_curve_type)
+                         free(line);
-                                         {
+                         continue;
-                                         case XVID_CURVE_SINE :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * sin(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_high_diff)));
-                                                 break;
-                                         case XVID_CURVE_LINEAR :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) / rc->alt_curve_high_diff);
-                                                 break;
-                                         case XVID_CURVE_COSINE :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (1.0 - cos(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_high_diff))));
-                                         }
                                  }
-                         }else{
-                 if (dbytes <= rc->alt_curve_low) {
+                 /* Read the stat line from buffer */
-                                         curve_temp = dbytes;
+                 fields = sscanf(ptr, "%c", &type);
-                 }else{
-                                         switch(rc->param.alt_curve_type)
+                 /* Valid stats files have at least 7 fields */
-                                         {
+                 if (fields == 1) {
-                                         case XVID_CURVE_SINE :
+                         switch(type) {
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * sin(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_low_diff)));
+                         case 'i':
-                                                 break;
+                         case 'I':
-                                         case XVID_CURVE_LINEAR :
+                                 rc->num_keyframes++;
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) / rc->alt_curve_low_diff);
+                         case 'p':
+                         case 'P':
+                         case 'b':
+                         case 'B':
+                         case 's':
+                         case 'S':
+                                 rc->num_frames++;
                                                  break;
-                                         case XVID_CURVE_COSINE :
+                         default:
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual + rc->alt_curve_qual_dev * (1.0 - cos(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_low_diff))));
+                                 DPRINTF(XVID_DEBUG_RC,
+                                                 "[xvid rc] -- WARNING: L%d unknown frame type used (%c).\n",
+                                                 lines, type);
                                          }
+                 } else {
+                                 DPRINTF(XVID_DEBUG_RC,
+                                                 "[xvid rc] -- WARNING: L%d misses some stat fields (%d).\n",
+                                                 lines, 7-fields);
                                  }
+                 /* Free the line buffer */
+                 free(line);
                          }
-                         if (rc->movie_curve > 1.0)
+         /* We are done with the file */
-                                 dbytes *= rc->movie_curve;
+         fclose(f);
-                         newquant = (int)(dbytes * 2.0 / (curve_temp * rc->curve_comp_scale + rc->alt_curve_curve_bias_bonus));
+         if (!rc->num_keyframes) return (-1); /* No keyframes? Then something is wrong */
-                         if (newquant > 1) {
+         else return(0);
-                                 if (newquant != oldquant) {
-                     int percent = (int)((i - rc->avg_length[XVID_TYPE_PVOP-1]) * 100.0 / rc->avg_length[XVID_TYPE_PVOP-1]);
-                                         oldquant = newquant;
-                                         DPRINTF(XVID_DEBUG_RC, "quant:%i threshold at %i : %i percent\n", newquant, i, percent);
-                                 }
-                         }
                  }
+ /* open stats file(s) and read into rc->stats array */
+ static int
+ statsfile_load(rc_2pass2_t *rc, char * filename)
+ {
+         FILE * f;
+         int processed_entries;
+         /* Opens the file */
+         if ((f = fopen(filename, "rb"))==NULL)
+                 return(-1);
+         processed_entries = 0;
+         while(processed_entries < rc->num_frames) {
+                 char type;
+                 int fields;
+                 twopass_stat_t * s = &rc->stats[processed_entries];
+                 char *line, *ptr;
+                 /* Read the line from the file */
+                 if((line = readline(f)) == NULL)
+                         break;
+                 /* We skip spaces */
+                 ptr = skipspaces(line);
+                 /* Skip comment lines or empty lines */
+                 if(iscomment(ptr) || *ptr == '\0') {
+                         free(line);
+                         continue;
      }
-     rc->overflow = 0;
+                 /* Reset this field that is optional */
-     rc->KFoverflow = 0;
+                 s->scaled_length = 0;
-     rc->KFoverflow_partial = 0;
-     rc->KF_idx = 1;
+                 /* Convert the fields */
+                 fields = sscanf(ptr,
+                                                 "%c %d %d %d %d %d %d %d\n",
+                                                 &type,
+                                                 &s->quant,
+                                                 &s->blks[0], &s->blks[1], &s->blks[2],
+                                                 &s->length, &s->invariant /* not really yet */,
+                                                 &s->scaled_length);
+                 /* Free line buffer, we don't need it anymore */
+                 free(line);
+                 /* Fail silently, this has probably been warned in
+                  * statsfile_count_frames */
+                 if(fields != 7 && fields != 8)
+                         continue;
+                 /* Convert frame type and compute the invariant length part */
+                 switch(type) {
+                 case 'i':
+                 case 'I':
+                         s->type = XVID_TYPE_IVOP;
+                         s->invariant /= INVARIANT_HEADER_PART_IVOP;
+                         break;
+                 case 'p':
+                 case 'P':
+                 case 's':
+                 case 'S':
+                         s->type = XVID_TYPE_PVOP;
+                         s->invariant /= INVARIANT_HEADER_PART_PVOP;
+                         break;
+                 case 'b':
+                 case 'B':
+                         s->type = XVID_TYPE_BVOP;
+                         s->invariant /= INVARIANT_HEADER_PART_BVOP;
+                         break;
+                 default:
+                         /* Same as before, fail silently */
+                         continue;
  }
+                 /* Ok it seems it's been processed correctly */
+                 processed_entries++;
+         }
+         /* Close the file */
+         fclose(f);
+         return(0);
+ }
- static int rc_2pass2_create(xvid_plg_create_t * create, rc_2pass2_t ** handle)
+ /* pre-process the statistics data
+  * - for each type, count, tot_length, min_length, max_length
+  * - set keyframes_locations, tot_prescaled */
+ static void
+ first_pass_stats_prepare_data(rc_2pass2_t * rc)
  {
-     xvid_plugin_2pass2_t * param = (xvid_plugin_2pass2_t *)create->param;
+         int i,j;
-     rc_2pass2_t * rc;
-     int i;
-     rc = malloc(sizeof(rc_2pass2_t));
+         /* *rc fields initialization
-     if (rc == NULL)
+          * NB: INT_MAX and INT_MIN are used in order to be immediately replaced
-         return XVID_ERR_MEMORY;
+          *     with real values of the 1pass */
+         for (i=0; i<3; i++) {
+                 rc->count[i]=0;
+                 rc->tot_length[i] = 0;
+                 rc->tot_invariant[i] = 0;
+                 rc->min_length[i] = INT_MAX;
+         }
-     rc->param = *param;
+         rc->max_length = INT_MIN;
+         rc->tot_weighted = 0;
-     if (rc->param.keyframe_boost <= 0) rc->param.keyframe_boost = 0;
+         /* Loop through all frames and find/compute all the stuff this function
-     if (rc->param.payback_method <= 0) rc->param.payback_method = XVID_PAYBACK_PROP;
+          * is supposed to do */
-     if (rc->param.bitrate_payback_delay <= 0) rc->param.bitrate_payback_delay = 250;
+         for (i=j=0; i<rc->num_frames; i++) {
-     if (rc->param.curve_compression_high <= 0) rc->param.curve_compression_high = 0;
+                 twopass_stat_t * s = &rc->stats[i];
-     if (rc->param.curve_compression_low <= 0) rc->param.curve_compression_low = 0;
-     if (rc->param.max_overflow_improvement <= 0) rc->param.max_overflow_improvement = 60;
-     if (rc->param.max_overflow_degradation <= 0) rc->param.max_overflow_degradation = 60;
-     if (rc->param.use_alt_curve <= 0) rc->param.use_alt_curve = 0;
-     if (rc->param.alt_curve_high_dist <= 0) rc->param.alt_curve_high_dist = 500;
-     if (rc->param.alt_curve_low_dist <= 0) rc->param.alt_curve_low_dist = 90;
-     if (rc->param.alt_curve_use_auto <= 0) rc->param.alt_curve_use_auto = 1;
-     if (rc->param.alt_curve_auto_str <= 0) rc->param.alt_curve_auto_str = 30;
-     if (rc->param.alt_curve_type <= 0) rc->param.alt_curve_type = XVID_CURVE_LINEAR;
-     if (rc->param.alt_curve_min_rel_qual <= 0) rc->param.alt_curve_min_rel_qual = 50;
-     if (rc->param.alt_curve_use_auto_bonus_bias <= 0) rc->param.alt_curve_use_auto_bonus_bias = 1;
-     if (rc->param.alt_curve_bonus_bias <= 0) rc->param.alt_curve_bonus_bias = 50;
-     if (rc->param.kftreshold <= 0) rc->param.kftreshold = 10;
-     if (rc->param.kfreduction <= 0) rc->param.kfreduction = 20;
-     if (rc->param.min_key_interval <= 0) rc->param.min_key_interval = 300;
-     if (!det_stats_length(rc, param->filename)){
+                 rc->count[s->type-1]++;
-         DPRINTF(XVID_DEBUG_RC,"fopen %s failed\n", param->filename);
+                 rc->tot_length[s->type-1] += s->length;
-         free(rc);
+                 rc->tot_invariant[s->type-1] += s->invariant;
-         return XVID_ERR_FAIL;
+                 if (s->zone_mode != XVID_ZONE_QUANT)
+                         rc->tot_weighted += (int)(s->weight*(s->length - s->invariant));
+                 if (s->length < rc->min_length[s->type-1]) {
+                         rc->min_length[s->type-1] = s->length;
      }
-     if ((rc->stats = malloc(rc->num_frames * sizeof(stat_t))) == NULL) {
+                 if (s->length > rc->max_length) {
-         free(rc);
+                         rc->max_length = s->length;
-         return XVID_ERR_MEMORY;
      }
-     /*
+                 if (s->type == XVID_TYPE_IVOP) {
-          * We need an extra location because we do as if the last frame were an
+                         rc->keyframe_locations[j] = i;
-          * IFrame. This is needed because our code consider that frames between
+                         j++;
-          * 2 IFrames form a natural sequence. So we store last frame as a
+                 }
-          * keyframe location.
-          */
-     if ((rc->keyframe_locations = malloc((rc->num_keyframes + 1) * sizeof(int))) == NULL) {
-         free(rc->stats);
-         free(rc);
-         return XVID_ERR_MEMORY;
      }
-     if (!load_stats(rc, param->filename)) {
+         /* NB:
-         DPRINTF(XVID_DEBUG_RC,"fopen %s failed\n", param->filename);
+          * The "per sequence" overflow system considers a natural sequence to be
-         free(rc->keyframe_locations);
+          * formed by all frames between two iframes, so if we want to make sure
-         free(rc->stats);
+          * the system does not go nuts during last sequence, we force the last
-         free(rc);
+          * frame to appear in the keyframe locations array. */
-         return XVID_ERR_FAIL;
+         rc->keyframe_locations[j] = i;
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Min 1st pass IFrame length: %d\n", rc->min_length[0]);
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Min 1st pass PFrame length: %d\n", rc->min_length[1]);
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Min 1st pass BFrame length: %d\n", rc->min_length[2]);
      }
-     /* pre-process our stats */
+ /* calculate zone weight "center" */
+ static void
+ zone_process(rc_2pass2_t *rc, const xvid_plg_create_t * create)
+ {
+         int i,j;
+         int n = 0;
-         if (rc->num_frames  < create->fbase/create->fincr) {
+         rc->tot_quant = 0;
-                 rc->target = rc->param.bitrate / 8;     /* one second */
+         rc->tot_quant_invariant = 0;
-         }else{
-                 rc->target = (rc->param.bitrate * rc->num_frames * create->fincr) / (create->fbase * 8);
+         if (create->num_zones == 0) {
+                 for (j = 0; j < rc->num_frames; j++) {
+                         rc->stats[j].zone_mode = XVID_ZONE_WEIGHT;
+                         rc->stats[j].weight = 1.0;
+                 }
+                 n += rc->num_frames;
          }
-     DPRINTF(XVID_DEBUG_RC, "rc->target : %i\n", rc->target);
- #if 0
+         for(i=0; i < create->num_zones; i++) {
-         rc->target -= rc->num_frames*24;        /* avi file header */
- #endif
+                 int next = (i+1<create->num_zones) ? create->zones[i+1].frame : rc->num_frames;
-         pre_process0(rc);
+                 /* Zero weight make no sense */
+                 if (create->zones[i].increment == 0) create->zones[i].increment = 1;
+                 /* And obviously an undetermined infinite makes even less sense */
+                 if (create->zones[i].base == 0) create->zones[i].base = 1;
-         if (rc->param.bitrate) {
+                 if (i==0 && create->zones[i].frame > 0) {
-         zone_process(rc, create);
+                         for (j = 0; j < create->zones[i].frame && j < rc->num_frames; j++) {
-                 internal_scale(rc);
+                                 rc->stats[j].zone_mode = XVID_ZONE_WEIGHT;
-     }else{
+                                 rc->stats[j].weight = 1.0;
-         /* external scaler: ignore zone */
-         for (i=0;i<rc->num_frames;i++) {
-             rc->stats[i].zone_mode = XVID_ZONE_WEIGHT;
-             rc->stats[i].weight = 1.0;
          }
-         rc->avg_weight = 1.0;
+                         n += create->zones[i].frame;
-         rc->tot_quant = 0;
      }
-         pre_process1(rc);
-     for (i=0; i<32;i++) {
+                 if (create->zones[i].mode == XVID_ZONE_WEIGHT) {
-         rc->pquant_error[i] = 0;
+                         for (j = create->zones[i].frame; j < next && j < rc->num_frames; j++ ) {
-         rc->bquant_error[i] = 0;
+                                 rc->stats[j].zone_mode = XVID_ZONE_WEIGHT;
-         rc->quant_count[i] = 0;
+                                 rc->stats[j].weight = (double)create->zones[i].increment / (double)create->zones[i].base;
+                         }
+                         next -= create->zones[i].frame;
+                         n += next;
+                 } else{  /* XVID_ZONE_QUANT */
+                         for (j = create->zones[i].frame; j < next && j < rc->num_frames; j++ ) {
+                                 rc->stats[j].zone_mode = XVID_ZONE_QUANT;
+                                 rc->stats[j].weight = (double)create->zones[i].increment / (double)create->zones[i].base;
+                                 rc->tot_quant += rc->stats[j].length;
+                                 rc->tot_quant_invariant += rc->stats[j].invariant;
+                         }
+                 }
      }
-     rc->fq_error = 0;
-     *handle = rc;
-         return(0);
  }
- static int rc_2pass2_destroy(rc_2pass2_t * rc, xvid_plg_destroy_t * destroy)
+ /* scale the curve */
+ static void
+ first_pass_scale_curve_internal(rc_2pass2_t *rc)
  {
-     free(rc->keyframe_locations);
+         int64_t target;
-     free(rc->stats);
+         int64_t total_invariant;
-         free(rc);
+         double scaler;
-         return(0);
+         int i, num_MBs;
+         /* We only scale texture data ! */
+         total_invariant  = rc->tot_invariant[XVID_TYPE_IVOP-1];
+         total_invariant += rc->tot_invariant[XVID_TYPE_PVOP-1];
+         total_invariant += rc->tot_invariant[XVID_TYPE_BVOP-1];
+         /* don't forget to substract header bytes used in quant zones, otherwise we
+          * counting them twice */
+         total_invariant -= rc->tot_quant_invariant;
+         /* We remove the bytes used by the fixed quantizer zones during first pass
+          * with the same quants, so we know very precisely how much that
+          * represents */
+         target  = rc->target;
+         target -= rc->tot_quant;
+         /* Let's compute a linear scaler in order to perform curve scaling */
+         scaler = (double)(target - total_invariant) / (double)(rc->tot_weighted);
+ #ifdef SMART_OVERFLOW_SETTING
+         if (scaler > 0.9) {
+                 rc->param.max_overflow_degradation *= 5;
+                 rc->param.max_overflow_improvement *= 5;
+                 rc->param.overflow_control_strength *= 3;
+         } else if (scaler > 0.6) {
+                 rc->param.max_overflow_degradation *= 2;
+                 rc->param.max_overflow_improvement *= 2;
+                 rc->param.overflow_control_strength *= 2;
+         } else {
+                 rc->min_quant = 2;
  }
+ #endif
+         /* Compute min frame lengths (for each frame type) according to the number
+          * of MBs. We sum all block type counters of frame 0, this gives us the
+          * number of MBs.
+          *
+          * We compare these hardcoded values with observed values in first pass
+          * (determined in pre_process0).Then we keep the real minimum. */
+         /* Number of MBs */
+         num_MBs  = rc->stats[0].blks[0];
+         num_MBs += rc->stats[0].blks[1];
+         num_MBs += rc->stats[0].blks[2];
+         /* Minimum for I frames */
+         if(rc->min_length[XVID_TYPE_IVOP-1] > ((num_MBs*22) + 240) / 8)
+                 rc->min_length[XVID_TYPE_IVOP-1] = ((num_MBs*22) + 240) / 8;
+         /* Minimum for P/S frames */
+         if(rc->min_length[XVID_TYPE_PVOP-1] > ((num_MBs) + 88)  / 8)
+                 rc->min_length[XVID_TYPE_PVOP-1] = ((num_MBs) + 88)  / 8;
+         /* Minimum for B frames */
+         if(rc->min_length[XVID_TYPE_BVOP-1] > 8)
+                 rc->min_length[XVID_TYPE_BVOP-1] = 8;
- static int rc_2pass2_before(rc_2pass2_t * rc, xvid_plg_data_t * data)
+         /* Perform an initial scale pass.
- {
+          *
-     stat_t * s = &rc->stats[data->frame_num];
+          * If a frame size is scaled underneath our hardcoded minimums, then we
-     int overflow;
+          * force the frame size to the minimum, and deduct the original & scaled
-     int desired;
+          * frame length from the original and target total lengths */
-     double dbytes;
+         for (i=0; i<rc->num_frames; i++) {
-     double curve_temp;
+                 twopass_stat_t * s = &rc->stats[i];
-     int capped_to_max_framesize = 0;
+                 int len;
-         /*
+                 /* No need to scale frame length for which a specific quantizer is
-          * This function is quite long but easy to understand. In order to simplify
+                  * specified thanks to zones */
-          * the code path (a bit), we treat 3 cases that can return immediatly.
+                 if (s->zone_mode == XVID_ZONE_QUANT) {
-          */
+                         s->scaled_length = s->length;
+                         continue;
+                 }
-         /* First case: Another plugin has already set a quantizer */
+                 /* Compute the scaled length -- only non invariant data length is scaled */
-     if (data->quant > 0)
+                 len = s->invariant + (int)((double)(s->length-s->invariant) * scaler * s->weight);
-                 return(0);
-         /* Second case: We are in a Quant zone */
+                 /* Compare with the computed minimum */
-         if (s->zone_mode == XVID_ZONE_QUANT) {
+                 if (len < rc->min_length[s->type-1]) {
+                         /* This is a 'forced size' frame, set its frame size to the
+                          * computed minimum */
+                         s->scaled_length = rc->min_length[s->type-1];
-                 rc->fq_error += s->weight;
+                         /* Remove both scaled and original size from their respective
-                 data->quant = (int)rc->fq_error;
+                          * total counters, as we prepare a second pass for 'regular'
-                 rc->fq_error -= data->quant;
+                          * frames */
+                         target -= s->scaled_length;
+                 } else {
+                         /* Do nothing for now, we'll scale this later */
+                         s->scaled_length = 0;
+                 }
+         }
-                 s->desired_length = s->length;
+         /* The first pass on data substracted all 'forced size' frames from the
+          * total counters. Now, it's possible to scale the 'regular' frames. */
-                 return(0);
+         /* Scaling factor for 'regular' frames */
+         scaler = (double)(target - total_invariant) / (double)(rc->tot_weighted);
+         /* Do another pass with the new scaler */
+         for (i=0; i<rc->num_frames; i++) {
+                 twopass_stat_t * s = &rc->stats[i];
+                 /* Ignore frame with forced frame sizes */
+                 if (s->scaled_length == 0)
+                         s->scaled_length = s->invariant + (int)((double)(s->length-s->invariant) * scaler * s->weight);
          }
-         /* Third case: insufficent stats data */
+         /* Job done */
-         if (data->frame_num >= rc->num_frames)
+         return;
-                 return 0;
+ }
-         /*
+ /* Apply all user settings to the scaled curve
-          * The last case is the one every normal minded developer should fear to
+  * This implies:
-          * maintain in a project :-)
+  *   keyframe boosting
-          */
+  *   high/low compression */
+ static void
+ scaled_curve_apply_advanced_parameters(rc_2pass2_t * rc)
+ {
+         int i;
+         int64_t ivop_boost_total;
-         /* XXX: why by 8 */
+         /* Reset the rate controller (per frame type) total byte counters */
-         overflow = rc->overflow / 8;
+         for (i=0; i<3; i++) rc->tot_scaled_length[i] = 0;
-         /*
+         /* Compute total bytes for each frame type */
-          * The rc->overflow field represents the overflow in current scene (between two
+         for (i=0; i<rc->num_frames;i++) {
-          * IFrames) so we must not forget to reset it if we are enetring a new scene
+                 twopass_stat_t *s = &rc->stats[i];
-          */
+                 rc->tot_scaled_length[s->type-1] += s->scaled_length;
-         if (s->type == XVID_TYPE_IVOP) {
-                 overflow = 0;
          }
-         desired = s->scaled_length;
+         /* First we compute the total amount of bits needed, as being described by
+          * the scaled distribution. During this pass over the complete stats data,
+          * we see how much bits two user settings will get/give from/to p&b frames:
+          *  - keyframe boosting
+          *  - keyframe distance penalty */
+         rc->KF_idx = 0;
+         ivop_boost_total = 0;
+         for (i=0; i<rc->num_frames; i++) {
+                 twopass_stat_t * s = &rc->stats[i];
-         dbytes = desired;
+                 /* Some more work is needed for I frames */
          if (s->type == XVID_TYPE_IVOP) {
-                 dbytes += desired * rc->param.keyframe_boost / 100;
+                         int ivop_boost;
-         }
-         dbytes /= rc->movie_curve;
-         /*
+                         /* Accumulate bytes needed for keyframe boosting */
-          * We are now entering in the hard part of the algo, it was first designed
+                         ivop_boost = s->scaled_length*rc->param.keyframe_boost/100;
-          * to work with i/pframes only streams, so the way it computes things is
-          * adapted to pframes only. However we can use it if we just take care to
-          * scale the bframes sizes to pframes sizes using the ratio avg_p/avg_p and
-          * then before really using values depending on frame sizes, scaling the
-          * value again with the inverse ratio
-          */
-         if (s->type == XVID_TYPE_BVOP) {
-                 dbytes *= rc->avg_length[XVID_TYPE_PVOP-1] / rc->avg_length[XVID_TYPE_BVOP-1];
-         }
-         /*
+ #if 0 /* ToDo: decide how to apply kfthresholding */
-          * Apply user's choosen Payback method. Payback helps bitrate to follow the
+ #endif
-          * scaled curve "paying back" past errors in curve previsions.
+                         /* If the frame size drops under the minimum length, then cap ivop_boost */
-          */
+                         if (ivop_boost + s->scaled_length < rc->min_length[XVID_TYPE_IVOP-1])
-         if (rc->param.payback_method == XVID_PAYBACK_BIAS) {
+                                 ivop_boost = rc->min_length[XVID_TYPE_IVOP-1] - s->scaled_length;
-                 desired =(int)(rc->curve_comp_error / rc->param.bitrate_payback_delay);
-         }else{
-                 desired = (int)(rc->curve_comp_error * dbytes /
-                                                 rc->avg_length[XVID_TYPE_PVOP-1] / rc->param.bitrate_payback_delay);
-                 if (labs(desired) > fabs(rc->curve_comp_error)) {
+                         /* Accumulate the ivop boost */
-                         desired = (int)rc->curve_comp_error;
+                         ivop_boost_total += ivop_boost;
+                         /* Don't forget to update the keyframe index */
+                         rc->KF_idx++;
                  }
          }
-         rc->curve_comp_error -= desired;
+         /* Initialize the IBoost tax ratio for P/S/B frames
+          *
+          * This ratio has to be applied to p/b/s frames in order to reserve
+          * additional bits for keyframes (keyframe boosting) or if too much
+          * keyframe distance is applied, bits retrieved from the keyframes.
+          *
+          * ie pb_length *= rc->pb_iboost_tax_ratio;
+          *
+          *    gives the ideal length of a p/b frame */
-         /*
+         /* Compute the total length of p/b/s frames (temporary storage into
-          * Alt curve treatment is not that hard to understand though the formulas
+          * movie_curve) */
-          * seem to be huge. Alt treatment is basically a way to soft/harden the
+         rc->pb_iboost_tax_ratio  = (double)rc->tot_scaled_length[XVID_TYPE_PVOP-1];
-          * curve flux applying sine/linear/cosine ratios
+         rc->pb_iboost_tax_ratio += (double)rc->tot_scaled_length[XVID_TYPE_BVOP-1];
-          */
+         /* Compute the ratio described above
+          *     taxed_total = sum(0, n, tax*scaled_length)
+          * <=> taxed_total = tax.sum(0, n, scaled_length)
+          * <=> tax = taxed_total / original_total */
+         rc->pb_iboost_tax_ratio =
+                 (rc->pb_iboost_tax_ratio - ivop_boost_total) /
+                 rc->pb_iboost_tax_ratio;
-         /* XXX: warning */
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- IFrame boost tax ratio:%.2f\n",
-         curve_temp = 0;
+                         rc->pb_iboost_tax_ratio);
-         if (rc->param.use_alt_curve) {
+         /* Compute the average size of frames per frame type */
-                 if (s->type != XVID_TYPE_IVOP)  {
+         for(i=0; i<3; i++) {
-                         if (dbytes > rc->avg_length[XVID_TYPE_PVOP-1]) {
+                 /* Special case for missing type or weird case */
-                                 if (dbytes >= rc->alt_curve_high) {
+                 if (rc->count[i] == 0 || rc->pb_iboost_tax_ratio == 0) {
-                                         curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev);
+                         rc->avg_length[i] = 1;
-                                 } else {
-                                         switch(rc->param.alt_curve_type) {
-                                         case XVID_CURVE_SINE :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * sin(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_high_diff)));
-                                                 break;
-                                         case XVID_CURVE_LINEAR :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) / rc->alt_curve_high_diff);
-                                                 break;
-                                         case XVID_CURVE_COSINE :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (1.0 - cos(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_high_diff))));
-                                         }
-                                 }
                          } else {
-                                 if (dbytes <= rc->alt_curve_low){
+                         rc->avg_length[i] = rc->tot_scaled_length[i];
-                                         curve_temp = dbytes;
+                         if (i == (XVID_TYPE_IVOP-1)) {
+                                 /* I Frames total has to be added the boost total */
+                                 rc->avg_length[i] += ivop_boost_total;
                                  } else {
-                                         switch(rc->param.alt_curve_type) {
+                                 /* P/B frames has to taxed */
-                                         case XVID_CURVE_SINE :
+                                 rc->avg_length[i] *= rc->pb_iboost_tax_ratio;
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * sin(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_low_diff)));
-                                                 break;
-                                         case XVID_CURVE_LINEAR :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual - rc->alt_curve_qual_dev * (dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) / rc->alt_curve_low_diff);
-                                                 break;
-                                         case XVID_CURVE_COSINE :
-                                                 curve_temp = dbytes * (rc->alt_curve_mid_qual + rc->alt_curve_qual_dev * (1.0 - cos(DEG2RAD * ((dbytes - rc->avg_length[XVID_TYPE_PVOP-1]) * 90.0 / rc->alt_curve_low_diff))));
                                          }
+                         /* Finally compute the average frame size */
+                         rc->avg_length[i] /= (double)rc->count[i];
                                  }
                          }
-                         /*
+         /* Assymetric curve compression */
-                          * End of code path for curve_temp, as told earlier, we are now
+         if (rc->param.curve_compression_high || rc->param.curve_compression_low) {
-                          * obliged to scale the value to a bframe one using the inverse
+                 double symetric_total;
-                          * ratio applied earlier
+                 double assymetric_delta_total;
-                          */
-                         if (s->type == XVID_TYPE_BVOP)
-                                 curve_temp *= rc->avg_length[XVID_TYPE_BVOP-1] / rc->avg_length[XVID_TYPE_PVOP-1];
-                         curve_temp = curve_temp * rc->curve_comp_scale + rc->alt_curve_curve_bias_bonus;
+                 /* Like I frame boosting, assymetric curve compression modifies the total
+                  * amount of needed bits, we must compute the ratio so we can prescale
+                  lengths */
+                 symetric_total = 0;
+                 assymetric_delta_total = 0;
+                 for (i=0; i<rc->num_frames; i++) {
+                         double assymetric_delta;
+                         double dbytes;
+                         twopass_stat_t * s = &rc->stats[i];
-                         desired += ((int)curve_temp);
+                         /* I Frames are not concerned by assymetric scaling */
-                         rc->curve_comp_error += curve_temp - (int)curve_temp;
+                         if (s->type == XVID_TYPE_IVOP)
-                 } else {
+                                 continue;
-                         /*
-                          * End of code path for dbytes, as told earlier, we are now
-                          * obliged to scale the value to a bframe one using the inverse
-                          * ratio applied earlier
-                          */
-                         if (s->type == XVID_TYPE_BVOP)
-                                 dbytes *= rc->avg_length[XVID_TYPE_BVOP-1] / rc->avg_length[XVID_TYPE_PVOP-1];
-                         desired += ((int)dbytes);
+                         /* During the real run, we would have to apply the iboost tax */
-                         rc->curve_comp_error += dbytes - (int)dbytes;
+                         dbytes = s->scaled_length * rc->pb_iboost_tax_ratio;
-                 }
-         } else if ((rc->param.curve_compression_high + rc->param.curve_compression_low) &&      s->type != XVID_TYPE_IVOP) {
+                         /* Update the symmetric curve compression total */
+                         symetric_total += dbytes;
-                 curve_temp = rc->curve_comp_scale;
+                         /* Apply assymetric curve compression */
-                 if (dbytes > rc->avg_length[XVID_TYPE_PVOP-1]) {
+                         if (dbytes > rc->avg_length[s->type-1])
-                         curve_temp *= ((double)dbytes + (rc->avg_length[XVID_TYPE_PVOP-1] - dbytes) * rc->param.curve_compression_high / 100.0);
+                                 assymetric_delta = (rc->avg_length[s->type-1] - dbytes) * (double)rc->param.curve_compression_high / 100.0f;
-                 } else {
+                         else
-                         curve_temp *= ((double)dbytes + (rc->avg_length[XVID_TYPE_PVOP-1] - dbytes) * rc->param.curve_compression_low / 100.0);
+                                 assymetric_delta = (rc->avg_length[s->type-1] - dbytes) * (double)rc->param.curve_compression_low  / 100.0f;
-                 }
-                 /*
+                         /* Cap to the minimum frame size if needed */
-                  * End of code path for curve_temp, as told earlier, we are now
+                         if (dbytes + assymetric_delta < rc->min_length[s->type-1])
-                  * obliged to scale the value to a bframe one using the inverse
+                                 assymetric_delta = rc->min_length[s->type-1] - dbytes;
-                  * ratio applied earlier
-                  */
+                         /* Accumulate after assymetric curve compression */
-                 if (s->type == XVID_TYPE_BVOP)
+                         assymetric_delta_total += assymetric_delta;
-                         curve_temp *= rc->avg_length[XVID_TYPE_BVOP-1] / rc->avg_length[XVID_TYPE_PVOP-1];
+                 }
-                 desired += (int)curve_temp;
+                 /* Compute the tax that all p/b frames have to pay in order to respect the
-                 rc->curve_comp_error += curve_temp - (int)curve_temp;
+                  * bit distribution changes that the assymetric compression curve imposes
+                  * We want assymetric_total = sum(0, n-1, tax.scaled_length)
+                  *      ie assymetric_total = ratio.sum(0, n-1, scaled_length)
+                  *         ratio = assymetric_total / symmetric_total */
+                 rc->assymetric_tax_ratio = ((double)symetric_total - (double)assymetric_delta_total) / (double)symetric_total;
          } else {
-                 /*
+                 rc->assymetric_tax_ratio = 1.0f;
-                  * End of code path for dbytes, as told earlier, we are now
-                  * obliged to scale the value to a bframe one using the inverse
-                  * ratio applied earlier
-                  */
-                 if (s->type == XVID_TYPE_BVOP){
-                         dbytes *= rc->avg_length[XVID_TYPE_BVOP-1] / rc->avg_length[XVID_TYPE_PVOP-1];
                  }
-                 desired += (int)dbytes;
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- Assymetric tax ratio:%.2f\n", rc->assymetric_tax_ratio);
-                 rc->curve_comp_error += dbytes - (int)dbytes;
+         /* Last bits that need to be reset */
+         rc->overflow = 0;
+         rc->KFoverflow = 0;
+         rc->KFoverflow_partial = 0;
+         rc->KF_idx = 0;
+         rc->desired_total = 0;
+         rc->real_total = 0;
+         /* Job done */
+         return;
          }
+ /*****************************************************************************
+  * VBV compliancy check and scale
+  * MPEG-4 standard specifies certain restrictions for bitrate/framesize in VBR
+  * to enable playback on devices with limited readspeed and memory (and which
+  * aren't...)
+  *
+  * DivX profiles have 2 criteria: VBV as in MPEG standard
+  *                                a limit on peak bitrate for any 3 seconds
+  *
+  * But if VBV is fulfilled, peakrate is automatically fulfilled in any profile
+  * define so far, so we check for it (for completeness) but correct only VBV
+  *
+  *****************************************************************************/
+ #define VBV_COMPLIANT 0
+ #define VBV_UNDERFLOW 1 /* video buffer runs empty */
+ #define VBV_OVERFLOW 2  /* doesn't exist for VBR encoding */
+ #define VBV_PEAKRATE 4  /* peak bitrate (within 3s) violated */
-         /*
+ static int
-          * We can't do bigger frames than first pass, this would be stupid as first
+ check_curve_for_vbv_compliancy(rc_2pass2_t * rc, const float fps)
-          * pass is quant=2 and that reaching quant=1 is not worth it. We would lose
+ {
-          * many bytes and we would not not gain much quality.
+         /* We do all calculations in float, for higher accuracy,
+          * and in bytes for convenience.
+          *
+          * typical values from DivX Home Theater profile:
+          *  vbv_size= 384*1024 (384kB)
+          *  vbv_initial= 288*1024 (75% fill)
+          *  maxrate= 4854000 (4.854MBps)
+          *  peakrate= 8000000 (8MBps)
+          *
+          *  PAL: offset3s = 75 (3 seconds of 25fps)
+          *  NTSC: offset3s = 90 (3 seconds of 29.97fps) or 72 (3 seconds of 23.976fps)
           */
-         if (desired > s->length) {
-                 rc->curve_comp_error += desired - s->length;
-                 desired = s->length;
-         }else{
-                 if (desired < rc->min_length[s->type-1]) {
-                         if (s->type == XVID_TYPE_IVOP){
-                                 rc->curve_comp_error -= rc->min_length[XVID_TYPE_IVOP-1] - desired;
-                         }
-                         desired = rc->min_length[s->type-1];
-                 }
-         }
-         s->desired_length = desired;
+         const float vbv_size = (float)rc->param.vbv_size/8.f;
+         float vbvfill = (float)rc->param.vbv_initial/8.f;
+         float vbvmin;
+         const float maxrate = (float)rc->param.vbv_maxrate;
+         const float peakrate = (float)rc->param.vbv_peakrate;
+         const float r0 = (int)(maxrate/fps+0.5)/8.f;
+         int bytes3s = 0;
+         int offset3s = (int)(3.f*fps+0.5);
+         int i;
-         /* if this keyframe is too close to the next, reduce it's byte allotment
+         /* 1Gbit should be enough to inuitialize the vbvmin
-            XXX: why do we do this after setting the desired length  */
+          *      an arbitrary high value */
+         vbvmin = 1000*1000*1000;
-         if (s->type == XVID_TYPE_IVOP) {
+         for (i=0; i<rc->num_frames; i++) {
-                 int KFdistance = rc->keyframe_locations[rc->KF_idx] - rc->keyframe_locations[rc->KF_idx - 1];
+                 /* DivX 3s peak bitrate check  */
+                 bytes3s += rc->stats[i].scaled_length;
+                 if (i>=offset3s)
+                         bytes3s -= rc->stats[i-offset3s].scaled_length;
-                 if (KFdistance < rc->param.kftreshold) {
+     /* ignore peakrate constraint if peakrate is <= 0.f */
+                 if (peakrate>0.f && 8.f*bytes3s > 3*peakrate)
+                         return(VBV_PEAKRATE);
-                         KFdistance = KFdistance - rc->param.min_key_interval;
+                 /* update vbv fill level */
+                 vbvfill += r0 - rc->stats[i].scaled_length;
-                         if (KFdistance >= 0) {
+                 /* this check is _NOT_ an "overflow"! only reading from disk stops then */
-                                 int KF_min_size;
+                 if (vbvfill > vbv_size)
+                         vbvfill = vbv_size;
-                                 KF_min_size = desired * (100 - rc->param.kfreduction) / 100;
+                 /* but THIS would be an underflow. report it! */
-                                 if (KF_min_size < 1)
+                 if (vbvfill < 0)
-                                         KF_min_size = 1;
+                         return(VBV_UNDERFLOW);
-                                 desired = KF_min_size + (desired - KF_min_size) * KFdistance /
+                 /* Store the minimum buffer filling */
-                                         (rc->param.kftreshold - rc->param.min_key_interval);
+                 if (vbvfill < vbvmin)
+                         vbvmin = vbvfill;
+         }
+         DPRINTF(XVID_DEBUG_RC, "[xvid rc] Minimum buffer fill: %f bytes\n", vbvmin);
-                                 if (desired < 1)
+         return(VBV_COMPLIANT);
-                                         desired = 1;
                          }
+ static int
+ scale_curve_for_vbv_compliancy(rc_2pass2_t * rc, const float fps)
+ {
+         /* correct any VBV violations. Peak bitrate violations disappears
+          * by this automatically
+          *
+          * This implementation follows
+          *
+          * Westerink, Rajagopalan, Gonzales "Two-pass MPEG-2 variable-bitrate encoding"
+          * IBM J. RES. DEVELOP. VOL 43, No. 4, July 1999, p.471--488
+          *
+          * Thanks, guys! This paper rocks!!! */
+         /* For each scene of len N, we have to check up to N^2 possible buffer fills.
+          * This works well with MPEG-2 where N==12 or so, but for MPEG-4 it's a
+          * little slow...
+          *
+          * TODO: Better control on VBVfill between scenes */
+         const float vbv_size = (float)rc->param.vbv_size/8.f;
+         const float vbv_initial = (float)rc->param.vbv_initial/8.f;
+         const float maxrate = 0.9*rc->param.vbv_maxrate;
+         const float vbv_low = 0.10f*vbv_size;
+         const float r0 = (int)(maxrate/fps+0.5)/8.f;
+         int i,k,l,n,violation = 0;
+         float *scenefactor;
+         int *scenestart;
+         int *scenelength;
+         /* first step: determine how many "scenes" there are and store their
+          * boundaries we could get all this from existing keyframe_positions,
+          * somehow, but there we don't have a min_scenelength, and it's no big
+          * deal to get it again. */
+         const int min_scenelength = (int)(fps+0.5);
+         int num_scenes = 0;
+         int last_scene = -999;
+         for (i=0; i<rc->num_frames; i++) {
+                 if ((rc->stats[i].type == XVID_TYPE_IVOP) && (i-last_scene>min_scenelength)) {
+                         last_scene = i;
+                         num_scenes++;
                  }
          }
-         overflow = (int)((double)overflow * desired / rc->avg_length[XVID_TYPE_PVOP-1]);
+         scenefactor = (float*)malloc(num_scenes*sizeof(float));
+         scenestart = (int*)malloc(num_scenes*sizeof(int));
+         scenelength = (int*)malloc(num_scenes*sizeof(int));
-         /* Reign in overflow with huge frames */
+         if ((!scenefactor) || (!scenestart) || (!scenelength) ) {
-         if (labs(overflow) > labs(rc->overflow)) {
+                 free(scenefactor);
-                 overflow = rc->overflow;
+                 free(scenestart);
+                 free(scenelength);
+                 /* remember: free(0) is valid and does exactly nothing. */
+                 return(-1);
          }
-         /* Make sure overflow doesn't run away */
+         /* count again and safe the length/position */
-         if (overflow > desired * rc->param.max_overflow_improvement / 100) {
-                 desired += (overflow <= desired) ? desired * rc->param.max_overflow_improvement / 100 :
-                         overflow * rc->param.max_overflow_improvement / 100;
-         } else if (overflow < desired * rc->param.max_overflow_degradation / -100){
-                 desired += desired * rc->param.max_overflow_degradation / -100;
-         } else {
-                 desired += overflow;
-         }
-         /* Make sure we are not higher than desired frame size */
+         num_scenes = 0;
-         if (desired > rc->max_length) {
+         last_scene = -999;
-                 capped_to_max_framesize = 1;
+         for (i=0; i<rc->num_frames; i++) {
-                 desired = rc->max_length;
+                 if ((rc->stats[i].type == XVID_TYPE_IVOP) && (i-last_scene>min_scenelength)) {
+                         if (num_scenes>0) {
+                                 scenelength[num_scenes-1]=i-last_scene;
+                         }
+                         scenestart[num_scenes]=i;
+                         num_scenes++;
+                         last_scene = i;
          }
+         }
+         scenelength[num_scenes-1]=i-last_scene;
-         /* Make sure to not scale below the minimum framesize */
+         /* second step: check for each scene, how much we can scale its frames up or
-         if (desired < rc->min_length[s->type-1])
+          * down such that the VBV restriction is just fulfilled */
-                 desired = rc->min_length[s->type-1];
+ #define R(k,n) (((n)+1-(k))*r0)     /* how much enters the buffer between frame k and n */
+         for (l=0; l<num_scenes;l++) {
+                 const int start = scenestart[l];
+                 const int length = scenelength[l];
+                 twopass_stat_t * frames = &rc->stats[start];
-         /*
+                 float S0n,Skn;
-          * Don't laugh at this very 'simple' quant<->filesize relationship, it
+                 float f,minf = 99999.f;
-          * proves to be acurate enough for our algorithm
-          */
-         data->quant= (s->quant * s->length) / desired;
-         /* Let's clip the computed quantizer, if needed */
+                 S0n=0.;
-         if (data->quant < 1) {
+                 for (n=0;n<=length-1;n++) {
-                 data->quant = 1;
+                         S0n += frames[n].scaled_length;
-         } else if (data->quant > 31) {
-                 data->quant = 31;
-         } else if (s->type != XVID_TYPE_IVOP) {
-                 /*
+                         k = 0;
-                  * The frame quantizer has not been clipped, this appear to be a good
+                         Skn = S0n;
-                  * computed quantizer, however past frames give us some info about how
+                         f = (R(k,n-1) + (vbv_initial - vbv_low)) / Skn;
-                  * this quantizer performs against the algo prevision. Let's use this
+                         if (f < minf)
-                  * prevision to increase the quantizer when we observe a too big
+                                 minf = f;
-                  * accumulated error
-                  */
-                 if (s->type== XVID_TYPE_BVOP) {
-                         rc->bquant_error[data->quant] += ((double)(s->quant * s->length) / desired) - data->quant;
-                         if (rc->bquant_error[data->quant] >= 1.0) {
+                         for (k=1;k<=n;k++) {
-                                 rc->bquant_error[data->quant] -= 1.0;
+                                 Skn -= frames[k].scaled_length;
-                                 data->quant++;
-                         }
-                 } else {
-                         rc->pquant_error[data->quant] += ((double)(s->quant * s->length) / desired) - data->quant;
-                         if (rc->pquant_error[data->quant] >= 1.0) {
+                                 f = (R(k,n-1) + (vbv_size - vbv_low)) / Skn;
-                                 rc->pquant_error[data->quant] -= 1.0;
+                                 if (f < minf)
-                                 ++data->quant;
+                                         minf = f;
                          }
                  }
+                 /* special case: at the end, fill buffer up to vbv_initial again
+                  *
+                  * TODO: Allow other values for buffer fill between scenes
+                  * e.g. if n=N is smallest f-value, then check for better value */
+                 n=length;
+                 k=0;
+                 Skn = S0n;
+                 f = R(k,n-1)/Skn;
+                 if (f < minf)
+                         minf = f;
+                 for (k=1;k<=n-1;k++) {
+                         Skn -= frames[k].scaled_length;
+                         f = (R(k,n-1) + (vbv_initial - vbv_low)) / Skn;
+                         if (f < minf)
+                                 minf = f;
          }
-         /*
+                 DPRINTF(XVID_DEBUG_RC, "[xvid rc] Scene %d (Frames %d-%d): VBVfactor %f\n",
-          * Now we have a computed quant that is in the right quante range, with a
+                                 l, start, start+length-1 , minf);
-          * possible +1 correction due to cumulated error. We can now safely clip
-          * the quantizer again with user's quant ranges. "Safely" means the Rate
+                 scenefactor[l] = minf;
-          * Control could learn more about this quantizer, this knowledge is useful
-          * for future frames even if it this quantizer won't be really used atm,
-          * that's why we don't perform this clipping earlier.
-          */
-         if (data->quant < data->min_quant[s->type-1]) {
-                 data->quant = data->min_quant[s->type-1];
-         } else if (data->quant > data->max_quant[s->type-1]) {
-                 data->quant = data->max_quant[s->type-1];
          }
+ #undef R
-         /*
+         /* last step: now we know of any scene how much it can be scaled up or down
-          * To avoid big quality jumps from frame to frame, we apply a "security"
+          * without violating VBV. Next, distribute bits from the evil scenes to the
-          * rule that makes |last_quant - new_quant| <= 2. This rule only applies
+          * good ones */
-          * to predicted frames (P and B)
+         do {
-          */
+                 float S_red = 0.f;    /* how much to redistribute */
-         if (s->type != XVID_TYPE_IVOP && rc->last_quant[s->type-1] && capped_to_max_framesize == 0) {
+                 float S_elig = 0.f;   /* sum of bit for those scenes you can still swallow something*/
+                 float f_red;
+                 int l;
-                 if (data->quant > rc->last_quant[s->type-1] + 2) {
+                 /* check how much is wrong */
-                         data->quant = rc->last_quant[s->type-1] + 2;
+                 for (l=0;l<num_scenes;l++) {
-                         DPRINTF(XVID_DEBUG_RC, "p/b-frame quantizer prevented from rising too steeply\n");
+                         const int start = scenestart[l];
+                         const int length = scenelength[l];
+                         twopass_stat_t * frames = &rc->stats[start];
+                         /* exactly 1 means "don't touch this anymore!" */
+                         if (scenefactor[l] == 1.)
+                                 continue;
+                         /* within limits */
+                         if (scenefactor[l] > 1.) {
+                                 for (n= 0; n < length; n++)
+                                         S_elig += frames[n].scaled_length;
+                         } else {
+                                 /* underflowing segment */
+                                 for (n= 0; n < length; n++) {
+                                         float newbytes = (float)frames[n].scaled_length * scenefactor[l];
+                                         S_red += (float)frames[n].scaled_length - (float)newbytes;
+                                         frames[n].scaled_length =(int)newbytes;
                  }
-                 if (data->quant < rc->last_quant[s->type-1] - 2) {
+                                 scenefactor[l] = 1.f;
-                         data->quant = rc->last_quant[s->type-1] - 2;
-                         DPRINTF(XVID_DEBUG_RC, "p/b-frame quantizer prevented from falling too steeply\n");
                  }
          }
-         /*
+                 /* no more underflows */
-          * We don't want to pollute the RC history results when our computed quant
+                 if (S_red < 1.f)
-          * has been computed from a capped frame size
+                         break;
-          */
-         if (capped_to_max_framesize == 0) {
-                 rc->last_quant[s->type-1] = data->quant;
-         }
-         return 0;
+                 if (S_elig < 1.f) {
+                         DPRINTF(XVID_DEBUG_RC, "[xvid rc] Everything underflowing.\n");
+                         free(scenefactor);
+                         free(scenestart);
+                         free(scenelength);
+                         return(-2);
  }
+                 f_red = (1.f + S_red/S_elig);
+                 DPRINTF(XVID_DEBUG_RC, "[xvid rc] Moving %.0f kB to avoid buffer underflow, correction factor: %.5f\n",
+                                 S_red/1024.f, f_red);
- static int rc_2pass2_after(rc_2pass2_t * rc, xvid_plg_data_t * data)
+                 violation=0;
- {
+                 /* scale remaining scenes up to meet total size */
-     stat_t * s = &rc->stats[data->frame_num];
+                 for (l=0; l<num_scenes; l++) {
+                         const int start = scenestart[l];
+                         const int length = scenelength[l];
+                         twopass_stat_t * frames = &rc->stats[start];
-         /* Insufficent stats data */
+                         if (scenefactor[l] == 1.)
-     if (data->frame_num >= rc->num_frames)
+                                 continue;
-         return 0;
-     rc->quant_count[data->quant]++;
+                         /* there shouldn't be any segments with factor<1 left, so all the rest is >1 */
+                         for (n= 0; n < length; n++) {
+                                 frames[n].scaled_length = (int)(frames[n].scaled_length * f_red + 0.5);
+                         }
-     if (data->type == XVID_TYPE_IVOP) {
+                         scenefactor[l] /= f_red;
-         int kfdiff = (rc->keyframe_locations[rc->KF_idx] -      rc->keyframe_locations[rc->KF_idx - 1]);
+                         if (scenefactor[l] < 1.f)
+                                 violation=1;
+                 }
-         rc->overflow += rc->KFoverflow;
+         } while (violation);
-         rc->KFoverflow = s->desired_length - data->length;
-         if (kfdiff > 1) {  // non-consecutive keyframes
+         free(scenefactor);
-             rc->KFoverflow_partial = rc->KFoverflow / (kfdiff - 1);
+         free(scenestart);
-         }else{ // consecutive keyframes
+         free(scenelength);
-                         rc->overflow += rc->KFoverflow;
+         return(0);
-                         rc->KFoverflow = 0;
-                         rc->KFoverflow_partial = 0;
          }
-         rc->KF_idx++;
-     }else{
-         // distribute part of the keyframe overflow
+ /*****************************************************************************
-         rc->overflow += s->desired_length - data->length + rc->KFoverflow_partial;
+  * Still more low level stuff (nothing to do with stats treatment)
-         rc->KFoverflow -= rc->KFoverflow_partial;
+  ****************************************************************************/
+ /* This function returns an allocated string containing a complete line read
+  * from the file starting at the current position */
+ static char *
+ readline(FILE *f)
+ {
+         char *buffer = NULL;
+         int buffer_size = 0;
+         int pos = 0;
+         do {
+                 int c;
+                 /* Read a character from the stream */
+                 c = fgetc(f);
+                 /* Is that EOF or new line ? */
+                 if(c == EOF || c == '\n')
+                         break;
+                 /* Do we have to update buffer ? */
+                 if(pos >= buffer_size - 1) {
+                         buffer_size += BUF_SZ;
+                         buffer = (char*)realloc(buffer, buffer_size);
+                         if (buffer == NULL)
+                                 return(NULL);
      }
-     DPRINTF(XVID_DEBUG_RC, "[%i] quant:%i stats1:%i scaled:%i actual:%i overflow:%i\n",
+                 buffer[pos] = c;
-         data->frame_num,
+                 pos++;
-         data->quant,
+         } while(1);
-         s->length,
-         s->scaled_length,
-         data->length,
-         rc->overflow);
-     return(0);
+         /* Read \n or EOF */
+         if (buffer == NULL) {
+                 /* EOF, so we reached the end of the file, return NULL */
+                 if(feof(f))
+                         return(NULL);
+                 /* Just an empty line with just a newline, allocate a 1 byte buffer to
+                  * store a zero length string */
+                 buffer = (char*)malloc(1);
+                 if(buffer == NULL)
+                         return(NULL);
  }
+         /* Zero terminated string */
+         buffer[pos] = '\0';
+         return(buffer);
+ }
- int xvid_plugin_2pass2(void * handle, int opt, void * param1, void * param2)
+ /* This function returns a pointer to the first non space char in the given
+  * string */
+ static char *
+ skipspaces(char *string)
  {
-     switch(opt)
+         const char spaces[] =
      {
-     case XVID_PLG_INFO :
+                         ' ','\t','\0'
-         return 0;
+                 };
+         const char *spacechar = spaces;
+         if (string == NULL) return(NULL);
+         while (*string != '\0') {
+                 /* Test against space chars */
+                 while (*spacechar != '\0') {
+                         if (*string == *spacechar) {
+                                 string++;
+                                 spacechar = spaces;
+                                 break;
+                         }
+                         spacechar++;
+                 }
-     case XVID_PLG_CREATE :
+                 /* No space char */
-         return rc_2pass2_create((xvid_plg_create_t*)param1, param2);
+                 if (*spacechar == '\0') return(string);
+         }
-     case XVID_PLG_DESTROY :
+         return(string);
-         return rc_2pass2_destroy((rc_2pass2_t*)handle, (xvid_plg_destroy_t*)param1);
+ }
-     case XVID_PLG_BEFORE :
+ /* This function returns a boolean that tells if the string is only a
-         return rc_2pass2_before((rc_2pass2_t*)handle, (xvid_plg_data_t*)param1);
+  * comment */
+ static int
+ iscomment(char *string)
+ {
+         const char comments[] =
+                 {
+                         '#',';', '%', '\0'
+                 };
+         const char *cmtchar = comments;
+         int iscomment = 0;
+         if (string == NULL) return(1);
+         string = skipspaces(string);
+         while(*cmtchar != '\0') {
+                 if(*string == *cmtchar) {
+                         iscomment = 1;
+                         break;
+                 }
+                 cmtchar++;
+         }
-     case XVID_PLG_AFTER :
+         return(iscomment);
-         return rc_2pass2_after((rc_2pass2_t*)handle, (xvid_plg_data_t*)param1);
      }
-     return XVID_ERR_FAIL;
+ #if 0
+ static void
+ stats_print(rc_2pass2_t * rc)
+ {
+         int i;
+         const char frame_type[4] = { 'i', 'p', 'b', 's'};
+         for (i=0; i<rc->num_frames; i++) {
+                 twopass_stat_t *s = &rc->stats[i];
+                 DPRINTF(XVID_DEBUG_RC, "[xvid rc] -- frame:%d type:%c quant:%d stats:%d scaled:%d desired:%d actual:%d overflow(%c):%.2f\n",
+                                 i, frame_type[s->type-1], -1, s->length, s->scaled_length,
+                                 s->desired_length, -1, frame_type[s->type-1], -1.0f);
+         }
  }
+ #endif

 Legend:



Removed from v.1.1.2.8
 


changed lines


 
Added in v.1.8
 Legend:



Removed from v.1.1.2.8
 


changed lines


 
Added in v.1.8
-Removed from v.1.1.2.8
+Added in v.1.8

No admin address has been configured	ViewVC Help
Powered by ViewVC 1.0.4