[FFmpeg-devel] [PATCH]Force the DAR for AVCI50 to 16:9

Carl Eugen Hoyos cehoyos at ag.or.at
Fri Jan 4 14:18:34 CET 2013


On Friday 04 January 2013 02:09:00 pm Carl Eugen Hoyos wrote:
> Hi!
>
> Attached patch forces the DAR of
> http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket1294/AVCI50_Test.mxf to
> 16:9 (by forcing SAR to 4:3, it is currently 3:4).

Attached is an alternative patch that also fixes 
http://samples.ffmpeg.org/ffmpeg-bugs/trac/ticket1294/AVCI50_Test_xf.mxf and 
to my deepest surprise does not brake fate...

Please review, Carl Eugen
-------------- next part --------------
diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
index a68645b..cfbb364 100644
--- a/libavcodec/h264_ps.c
+++ b/libavcodec/h264_ps.c
@@ -185,8 +185,8 @@ static inline int decode_vui_parameters(H264Context *h, SPS *sps){
     if( aspect_ratio_info_present_flag ) {
         aspect_ratio_idc= get_bits(&s->gb, 8);
         if( aspect_ratio_idc == EXTENDED_SAR ) {
-            sps->sar.num= get_bits(&s->gb, 16);
             sps->sar.den= get_bits(&s->gb, 16);
+            sps->sar.num= get_bits(&s->gb, 16);
         }else if(aspect_ratio_idc < FF_ARRAY_ELEMS(pixel_aspect)){
             sps->sar=  pixel_aspect[aspect_ratio_idc];
         }else{


More information about the ffmpeg-devel mailing list