[FFmpeg-devel] [PATCH] make mpeg4 parser set has_b_frames

elupus elupus
Thu Nov 22 00:45:01 CET 2007

I'm not entirely sure this is correct, but it seems right.

Currently the mpeg4 parser never set's has_b_frames, thus the dts 
calculation code is never used. This patch set's it from the low_delay flag 
in the mpeg4 stream.

On a different but related note, should pts really be set to dts in 
compute_pkt_fields if pts is missing? wouldn't it be better to leave that up 
to the player? or atleast be conditional on GENPTS flag.

Reason i'm asking is that our player can handle both sync on pts and dts. 
Normaly pts is used, and dts only used if pts is missing. But since pts 
get's set to dts, an incorrect pts is used in player when B frames are 

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: has_b_frames.patch
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20071122/e7d65bd6/attachment.asc>

More information about the ffmpeg-devel mailing list