[FFmpeg-devel] [PATCH] flv 32 bit signed timestamps

[Michael Niedermayer]

On Sun, Jul 27, 2008 at 10:19:17PM -0700, Baptiste Coudurier wrote:
> Hi,
> 
> Like figured out, flv uses 32 bit signed timestamps, meaning that
> pts_wrap_bits must be set to 31.

What fails, why does it fail and how is it fixed by this change?


[...]
> @@ -362,7 +366,7 @@
>          put_byte(pb,1); // AAC raw
>      else if (enc->codec_id == CODEC_ID_H264) {
>          put_byte(pb,1); // AVC NALU
> -        put_be24(pb,pkt->pts - (int32_t)pkt->dts);
> +        put_be24(pb,pkt->pts - pkt->dts);
>      }

Why was that int32_t cast there?

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

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080728/15e7caed/attachment.pgp>