[MPlayer-cvslog] r33448 - trunk/gui/mplayer/gui_common.c

[Ingo Brückl]

Reimar Döffinger wrote on Mon, 9 May 2011 20:33:28 +0200:

> On 9 May 2011, at 13:17, ib <subversion at mplayerhq.hu> wrote:
>> Author: ib
>> Date: Mon May  9 13:17:43 2011
>> New Revision: 33448
>>
>> Log:
>> Check for reasonable time values.
>>
>> Some A/V files don't provide reasonable time information in which case
>> the GUI produces a junk dlabel, so set values to zero.

> A sample would be a good idea. Also using FFMAX should simplify the code.

What exactly do you mean by sample? I have a captured 500 MB wmv live stream,
returning a time length of 1844674407365.95508. Because the time length
variables used are of type int, this gets printf'ed to "-596523:-14:-8".

Rather than r33448, I'd like to fix it with the attached patch (revising my
suggested [PATCH] Check for reasonable time length), because this patch would
solve OSD problems as well as GUI problems, and I could revert r33448.

Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: timelength.patch
Type: text/x-diff
Size: 345 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20110510/a6fc951c/attachment.bin>