diff options
author | Diego Biurrun <diego@biurrun.de> | 2010-01-05 13:18:01 +0300 |
---|---|---|
committer | Diego Biurrun <diego@biurrun.de> | 2010-01-05 13:18:01 +0300 |
commit | 50fe16960bb44d9ad5cca5bf0e6cafd89e77a011 (patch) | |
tree | 4b579cb32d7a33405d672de0528f35d58fc1327d /libavcodec/h263.c | |
parent | 71fe4573bc6c70a04fd0410173e2b97d705b29b6 (diff) |
Add a few explanatory comments to cornercase workarounds.
Originally committed as revision 21030 to svn://svn.ffmpeg.org/ffmpeg/trunk
Diffstat (limited to 'libavcodec/h263.c')
-rw-r--r-- | libavcodec/h263.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/libavcodec/h263.c b/libavcodec/h263.c index ae4ef3d956..807dbc60c6 100644 --- a/libavcodec/h263.c +++ b/libavcodec/h263.c @@ -5732,9 +5732,9 @@ no_cplx_est: if( h_sampling_factor_n==0 || h_sampling_factor_m==0 || v_sampling_factor_n==0 || v_sampling_factor_m==0){ - + /* illegal scalability header (VERY broken encoder), + * trying to workaround */ s->scalability=0; - *gb= bak; }else av_log(s->avctx, AV_LOG_ERROR, "scalability not supported\n"); @@ -5847,6 +5847,8 @@ static int decode_vop_header(MpegEncContext *s, GetBitContext *gb){ s->time= s->time_base*s->avctx->time_base.den + time_increment; if(s->workaround_bugs&FF_BUG_UMP4){ if(s->time < s->last_non_b_time){ + /* header is not mpeg-4-compatible, broken encoder, + * trying to workaround */ s->time_base++; s->time+= s->avctx->time_base.den; } @@ -5857,6 +5859,7 @@ static int decode_vop_header(MpegEncContext *s, GetBitContext *gb){ s->time= (s->last_time_base + time_incr)*s->avctx->time_base.den + time_increment; s->pb_time= s->pp_time - (s->last_non_b_time - s->time); if(s->pp_time <=s->pb_time || s->pp_time <= s->pp_time - s->pb_time || s->pp_time<=0){ + /* messed up order, maybe after seeking? skipping current b-frame */ return FRAME_SKIPPED; } ff_mpeg4_init_direct_mv(s); |