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

[Ingo Brückl]

I wrote on Tue, 10 May 2011 20:58:08 +0200:

> Reimar Döffinger wrote on Tue, 10 May 2011 18:38:13 +0200:

>> On Tue, May 10, 2011 at 12:59:18PM +0200, Ingo Brückl wrote:
>>> 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.
>>>

>> Yes, but why is that so?

> That's beyond my knowledge.

But fortunately my knowledge improves. :-)

>> This seems likely to be a demuxer bug and it would be a good idea to fix
>> that bug and not _only_ "sweep it under the rug".

> I certainly have no idea [...]

Now I have.

After reading http://msdn.microsoft.com/en-us/library/aa368932(v=vs.85).aspx
I found a bug in libmpdemux/asfheader.c.

Ingo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: asfheader.patch
Type: text/x-diff
Size: 645 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20110513/3d160e3f/attachment.bin>