[FFmpeg-devel] Change 9963 introduces regression - start_time in asf.c

[Michael Niedermayer]

On Sat, Aug 11, 2007 at 07:52:51AM +1000, Neil Brown wrote:
[...]
> > > Could it be that with this change, start_time isn't being set at all?
> > > If it add code to explicitly set it to zero, I get the correct clip
> > > length again.
> > > Before I set it to 0, it's value is
> > >   0x8000000000000000
> > 
> > start time is correct, your attempt to set it to 0 is _WRONG_
> > 
> 
> Can you explain why?  Quite a lot of decoders seem to do it.

start time is and MUST be the start time its not a random number
for that there is random()
it MUST only be set to 0 if the first packet has a pts of 0 if this
cannot be guranteed then it must not be set (=INT64_MIN=AV_NOPTS_VALUE)

all demuxers which set it to 0 either gurantee that the first pts is 0
or they are buggy

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

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- 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/20070811/aa2f0684/attachment.pgp>