[FFmpeg-devel] [Bugreport] and [Patch]: duration of TS streams is wrong when used near 2^33 wrap

[Michael Niedermayer]

Hi

On Mon, Oct 29, 2007 at 04:52:52PM +0100, Thorsten Jordan wrote:
> Rich Felker schrieb:
> > On Mon, Oct 29, 2007 at 01:01:38PM +0100, Michael Niedermayer wrote:
> >>> I'm inclined to say that rather than trying to guess the unknown, we should
> >>> simply report the duration of these files as unknown.
> >> yes, fully agree
> > 
> > Me too. I'm inclined to always agree with this principle. Returning
> > guesses in fields documented as providing reliable information is very
> > bad.
> following that argumentation, the current implementation to compute
> duration would have to be removed too, because it is a guess as well, as
> Mans said. But who would want that?
> The proposed change wouldn't make the current situation much worse imho,
> but ok, it is your decision.

the change makes the code worse
as it turns negative durations (which would be ignored) into valid positive
ones, in practice such negative durations are more likely caused by
timestamp resets instead of 33bit wraping

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

If you really think that XML is the answer, then you definitly missunderstood
the question -- Attila Kinali
-------------- 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/20071029/d0de6c3a/attachment.pgp>