[FFmpeg-devel] [PATCH] Chromium MKV patches

Michael Niedermayer michaelni
Wed May 5 13:17:50 CEST 2010

On Wed, May 05, 2010 at 05:18:20AM -0400, David Conrad wrote:
> Hi,
> Here are the useful patches from chromium for the mkv demuxer. 1 doesn't fix any actual issues that I'm aware of, but might make it easier to spot any if they exist.

> Another change made was ensuring sample_rate was nonzero, but AFAIK a zero sample_rate meant unknown and applications should handle that instead of the demuxer inventing a rate?

i was unaware that matroska allowed a undefined sample_rate, thats broken
design for a container that normally stores the sample_rate
the solution may be to set needs_parsing to headers only if its not a
higher value alraedy
having the problematic mkv file woulf of course help to see what exactly
is the problem

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Democracy is the form of government in which you can choose your dictator
-------------- 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/20100505/82b01827/attachment.pgp>

More information about the ffmpeg-devel mailing list