[FFmpeg-cvslog] r15015 - trunk/libavformat/utils.c

[Baptiste Coudurier]

Hi Michael,

michael wrote:
> Author: michael
> Date: Fri Aug 29 03:43:27 2008
> New Revision: 15015
> 
> Log:
> Do not truncate timestamps before the muxer as it makes simple
> things like last_pts - pts rather tricky and is not good for anything.
> Timestamps should be truncated just before storing when needed.
> 

Nice I had the same idea.

> Modified:
>    trunk/libavformat/utils.c
> 
> Modified: trunk/libavformat/utils.c
> ==============================================================================
> --- trunk/libavformat/utils.c	(original)
> +++ trunk/libavformat/utils.c	Fri Aug 29 03:43:27 2008
> @@ -2574,18 +2574,6 @@ static int compute_pkt_fields2(AVStream 
>      return 0;
>  }
>  
> -static void truncate_ts(AVStream *st, AVPacket *pkt){
> -    int64_t pts_mask = (2LL << (st->pts_wrap_bits-1)) - 1;
> -

'pts_wrap_bits field' is now useless I think. Should we remove it ?

-- 
Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
Smartjog USA Inc.                                http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA