[Ffmpeg-cvslog] CVS: ffmpeg/libavcodec mpeg12.c,1.248,1.249

[Ivan Kalvachev]

2006/3/2, Michael Niedermayer CVS <michael at mplayerhq.hu>:
> Update of /cvsroot/ffmpeg/ffmpeg/libavcodec
> In directory mail:/var2/tmp/cvs-serv31999
>
> Modified Files:
>         mpeg12.c
> Log Message:
> fix decoding of mpeg2-trunc.vob
>
>
> Index: mpeg12.c
> ===================================================================
> RCS file: /cvsroot/ffmpeg/ffmpeg/libavcodec/mpeg12.c,v
> retrieving revision 1.248
> retrieving revision 1.249
> diff -u -d -r1.248 -r1.249
> --- mpeg12.c    4 Feb 2006 20:32:02 -0000       1.248
> +++ mpeg12.c    1 Mar 2006 22:59:22 -0000       1.249
> @@ -3114,7 +3114,7 @@
>                          /* skip b frames if we dont have reference frames */
>                              if(s2->pict_type==B_TYPE) break;
>                          /* skip P frames if we dont have reference frame no valid header */
> -                            if(s2->pict_type==P_TYPE && !s2->first_slice) break;
> +//                            if(s2->pict_type==P_TYPE && s2->first_field && !s2->first_slice) break;
>                          }

If I get it right, this is for the case where the second field is
P_TYPE and compensates from the first field that is I_TYPE, all in the
first frame.

So I wonder why the new code is commented out?