[MPlayer-dev-eng] [PATCH] demux_audio - be stricter when recognizing WAV format

Daniel Verkamp daniel at drv.nu
Thu Sep 10 18:45:50 CEST 2009


On Tue, Sep 8, 2009 at 7:15 PM, compn<tempn at twmi.rr.com> wrote:
> On Wed, 9 Sep 2009 01:22:33 +0200, Reimar Döffinger wrote:
>>On Tue, Sep 08, 2009 at 05:23:59PM -0500, Daniel Verkamp wrote:
>>> Currently, demux_audio picks up Sony Wave64 files as plain WAV due to
>>> the fmt chunk being recognized.  The attached patch adds an extra
>>> check so that the RIFF and WAVE chunks must be seen first to recognize
>>> a fmt chunk as indicating a WAV-format file.  This allows W64 files to
>>> fall through to lavf.
>>
>>The simpler and probably better solution would be to add your demuxer to
>>the preferred formats.
>
> in demux_lavf.c or extension.c ?
>
> but yeah, your change might break some wav files ...
>
> -compn

Presumably this is the desired change; I guess the list is not ordered
any particular way?

Thanks,
-- Daniel Verkamp
-------------- next part --------------
Index: libmpdemux/demux_lavf.c
===================================================================
--- libmpdemux/demux_lavf.c	(revision 29664)
+++ libmpdemux/demux_lavf.c	(working copy)
@@ -184,6 +184,7 @@
     "mpc",
     "mpc8",
     "vqf",
+    "w64",
     NULL
 };
 


More information about the MPlayer-dev-eng mailing list