[FFmpeg-devel] [PATCH] make mpeg4 parser set has_b_frames
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...
More information about the ffmpeg-devel