Further discussion with Jason (aka Dark_Shikari):

10:00 <Dark_Shikari> You inserted a variable into x264_param_t.
10:00 <Dark_Shikari> This shifts down all the other variables.                  
                                    

>From the diff, I spot this change:
http://launchpadlibrarian.net/59039785/x264_2:0.98.1653%2Bgit88b90d9-3_2:0.98.1653%2Bgit88b90d9-3ubuntu1.diff.gz


+=== modified file 'x264.h'
+--- old/x264.h 2010-11-02 19:26:22 +0000
++++ new/x264.h 2010-11-02 19:26:35 +0000
+@@ -328,7 +328,9 @@
+     int b_annexb;               /* if set, place start codes (4 bytes) before 
NAL units,
+                                  * otherwise place size (4 bytes) before NAL 
units. */
+     int i_sps_id;               /* SPS and PPS id number */
+-    int b_vfr_input;            /* VFR input */
++    int b_vfr_input;            /* VFR input.  If 1, use timebase and 
timestamps for ratecontrol purposes.
++                                 * If 0, use fps only. */
++    int b_pulldown;             /* use explicity set timebase for CFR */
+     uint32_t i_fps_num;
+     uint32_t i_fps_den;
+     uint32_t i_timebase_num;    /* Timebase numerator */
+

This strictly speaking requires a SONAME bump and therefore is totally
unacceptable for an SRU. I strongly suggest to revert this change.

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/690296

Title:
  libx264 2:0.98.1653+git88b90d9-3ubuntu1 messes up bitrate when encoding (at 
least using mencoder)

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to