[FFmpeg-devel] [PATCH]: Fix decoding of mpegts streams with h264 video that does *NOT* have b frames

[Michael Niedermayer]

On Mon, May 02, 2011 at 11:48:12AM -0400, Tony Strauss wrote:
> On Sat, Apr 30, 2011 at 10:37 PM, Michael Niedermayer <michaelni at gmx.at>wrote:
> 
> > applied & pushed
> > thx
> >
> > [..]
> 
> 
> I still also think that my patch is a good idea, because:
> * It fixes ffmpeg for the universe of 264 encoders
> * The parser and the codec should initialize the context in the same manner
> (where that makes sense).  There may be other bugs of this ilk caused by the
> context not being initialized properly by the parser.
> 
> Does anyone have any comments on my patch?

It looks wrong, for example the if(!avctx->has_b_frames) check is
specific to being between parser and decoder.nothing prior of the
parser would set has_b_frames for a decoder.

also some other code in there looks decoder specific like the vlc
stuff

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you think the mosad wants you dead since a long time then you are either
wrong or dead since a long time.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110502/f40f1402/attachment.asc>