[FFmpeg-devel] [PATCH] Wave64 demuxer

Daniel Verkamp daniel
Fri Aug 7 23:26:37 CEST 2009


On Fri, Aug 7, 2009 at 5:20 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
> On Fri, Aug 07, 2009 at 04:37:25PM -0400, Daniel Verkamp wrote:
>> On Fri, Aug 7, 2009 at 4:07 PM, Michael Niedermayer<michaelni at gmx.at> wrote:
>> > On Fri, Aug 07, 2009 at 03:04:25PM -0400, Daniel Verkamp wrote:
>> >> On Fri, Aug 7, 2009 at 2:44 PM, Daniel Verkamp<daniel at drv.nu> wrote:
>> >> > On Fri, Aug 7, 2009 at 1:52 PM, Reimar
>> >> > D?ffinger<Reimar.Doeffinger at gmx.de> wrote:
>> >> [...]
>> >> >> But this whole thing looks a lot like a normal wav file just with
>> >> >> some extension, are you sure this should be a completely new demuxer
>> >> >> instead of an extension to the "normal" one?
>> >> >
>> >> > I thought about that too, but I don't think it would be any cleaner or
>> >> > shorter. ?The header is different except for the parts that are
>> >> > already shared in ff_get_wav_header(), so it would just be a big mess
>> >> > of if(w64){}else{}. ?I can reconsider combining them if that's really
>> >> > desirable, though. ?I suppose read_packet could be shared with some
>> >> > changes...
>> >> >
>> >>
>> >> Quick attempt at reusing more WAV demuxer code for W64... I prefer the
>> >> previous patch, but this works too.
>> >
>> > this patch seems smaller ...
>> >
>> >
>> >>
>> >> Thanks,
>> >> -- Daniel Verkamp
>> >
>> >> From 2645f38fa1ec2f2f68234f2c91353e382cfc946e Mon Sep 17 00:00:00 2001
>> >> From: Daniel Verkamp <daniel at drv.nu>
>> >> Date: Fri, 7 Aug 2009 15:02:41 -0400
>> >> Subject: [PATCH] Wave64 demuxer combined with WAV
>> >>
>> >> ---
>> >> ?Changelog ? ? ? ? ? ? ? ?| ? ?1 +
>> >> ?doc/general.texi ? ? ? ? | ? ?1 +
>> >> ?libavformat/Makefile ? ? | ? ?1 +
>> >> ?libavformat/allformats.c | ? ?1 +
>> >> ?libavformat/w64.c ? ? ? ?| ? 35 +++++++++++++++
>> >> ?libavformat/w64.h ? ? ? ?| ? 32 ++++++++++++++
>> >> ?libavformat/wav.c ? ? ? ?| ?107 +++++++++++++++++++++++++++++++++++++++++++++-
>> >> ?7 files changed, 177 insertions(+), 1 deletions(-)
>> >> ?create mode 100644 libavformat/w64.c
>> >> ?create mode 100644 libavformat/w64.h
>> >>
>> >> diff --git a/Changelog b/Changelog
>> >> index 03599fd..1725e75 100644
>> >> --- a/Changelog
>> >> +++ b/Changelog
>> >> @@ -31,6 +31,7 @@ version <next>:
>> >> ?- RTP depacketization of ASF and RTSP from WMS servers
>> >> ?- RTMP support in libavformat
>> >> ?- noX handling for OPT_BOOL X options
>> >> +- Wave64 demuxer
>> >>
>> >>
>> >>
>> >> diff --git a/doc/general.texi b/doc/general.texi
>> >> index 6326df9..b83158e 100644
>> >> --- a/doc/general.texi
>> >> +++ b/doc/general.texi
>> >> @@ -210,6 +210,7 @@ library:
>> >> ?@item Sony OpenMG (OMA) ? ? ? ? @tab ? @tab X
>> >> ? ? ?@tab Audio format used in Sony Sonic Stage and Sony Vegas.
>> >> ?@item Sony PlayStation STR ? ? ?@tab ? @tab X
>> >> + at item Sony Wave64 (W64) ? ? ? ? @tab ? @tab X
>> >> ?@item SoX native format ? ? ? ? @tab X @tab X
>> >> ?@item SUN AU format ? ? ? ? ? ? @tab X @tab X
>> >> ?@item THP ? ? ? ? ? ? ? ? ? ? ? @tab ? @tab X
>> >> diff --git a/libavformat/Makefile b/libavformat/Makefile
>> >> index 7838efe..e5b9afc 100644
>> >> --- a/libavformat/Makefile
>> >> +++ b/libavformat/Makefile
>> >> @@ -216,6 +216,7 @@ OBJS-$(CONFIG_VMD_DEMUXER) ? ? ? ? ? ? ? += sierravmd.o
>> >> ?OBJS-$(CONFIG_VOC_DEMUXER) ? ? ? ? ? ? ? += vocdec.o voc.o
>> >> ?OBJS-$(CONFIG_VOC_MUXER) ? ? ? ? ? ? ? ? += vocenc.o voc.o
>> >> ?OBJS-$(CONFIG_VQF_DEMUXER) ? ? ? ? ? ? ? += vqf.o
>> >> +OBJS-$(CONFIG_W64_DEMUXER) ? ? ? ? ? ? ? += w64.o wav.o riff.o raw.o
>> >> ?OBJS-$(CONFIG_WAV_DEMUXER) ? ? ? ? ? ? ? += wav.o riff.o raw.o
>> >> ?OBJS-$(CONFIG_WAV_MUXER) ? ? ? ? ? ? ? ? += wav.o riff.o
>> >> ?OBJS-$(CONFIG_WC3_DEMUXER) ? ? ? ? ? ? ? += wc3movie.o
>> >> diff --git a/libavformat/allformats.c b/libavformat/allformats.c
>> >> index 569ef50..0990598 100644
>> >> --- a/libavformat/allformats.c
>> >> +++ b/libavformat/allformats.c
>> >> @@ -193,6 +193,7 @@ void av_register_all(void)
>> >> ? ? ?REGISTER_DEMUXER ?(VMD, vmd);
>> >> ? ? ?REGISTER_MUXDEMUX (VOC, voc);
>> >> ? ? ?REGISTER_DEMUXER ?(VQF, vqf);
>> >> + ? ?REGISTER_DEMUXER ?(W64, w64);
>> >> ? ? ?REGISTER_MUXDEMUX (WAV, wav);
>> >> ? ? ?REGISTER_DEMUXER ?(WC3, wc3);
>> >> ? ? ?REGISTER_DEMUXER ?(WSAUD, wsaud);
>> >
>> >> diff --git a/libavformat/w64.c b/libavformat/w64.c
>> >> new file mode 100644
>> >> index 0000000..77c837a
>> >> --- /dev/null
>> >> +++ b/libavformat/w64.c
>> >> @@ -0,0 +1,35 @@
>> >> +/*
>> >> + * Sony Wave64 constants
>> >> + * Copyright (c) 2009 Daniel Verkamp
>> >> + *
>> >> + * This file is part of FFmpeg.
>> >> + *
>> >> + * FFmpeg is free software; you can redistribute it and/or
>> >> + * modify it under the terms of the GNU Lesser General Public
>> >> + * License as published by the Free Software Foundation; either
>> >> + * version 2.1 of the License, or (at your option) any later version.
>> >> + *
>> >> + * FFmpeg is distributed in the hope that it will be useful,
>> >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. ?See the GNU
>> >> + * Lesser General Public License for more details.
>> >> + *
>> >> + * You should have received a copy of the GNU Lesser General Public
>> >> + * License along with FFmpeg; if not, write to the Free Software
>> >> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
>> >> + */
>> >> +
>> >> +#include "avformat.h"
>> >> +#include "w64.h"
>> >> +
>> >> +const uint8_t ff_w64_guid_riff[16] = { 'r', 'i', 'f', 'f',
>> >> + ? ?0x2E, 0x91, 0xCF, 0x11, 0xA5, 0xD6, 0x28, 0xDB, 0x04, 0xC1, 0x00, 0x00 };
>> >> +
>> >> +const uint8_t ff_w64_guid_wave[16] = { 'w', 'a', 'v', 'e',
>> >> + ? ?0xF3, 0xAC, 0xD3, 0x11, 0x8C, 0xD1, 0x00, 0xC0, 0x4F, 0x8E, 0xDB, 0x8A };
>> >> +
>> >> +const uint8_t ff_w64_guid_fmt [16] = { 'f', 'm', 't', ' ',
>> >> + ? ?0xF3, 0xAC, 0xD3, 0x11, 0x8C, 0xD1, 0x00, 0xC0, 0x4F, 0x8E, 0xDB, 0x8A };
>> >> +
>> >> +const uint8_t ff_w64_guid_data[16] = { 'd', 'a', 't', 'a',
>> >> + ? ?0xF3, 0xAC, 0xD3, 0x11, 0x8C, 0xD1, 0x00, 0xC0, 0x4F, 0x8E, 0xDB, 0x8A };
>> >> diff --git a/libavformat/w64.h b/libavformat/w64.h
>> >> new file mode 100644
>> >> index 0000000..671d286
>> >> --- /dev/null
>> >> +++ b/libavformat/w64.h
>> >> @@ -0,0 +1,32 @@
>> >> +/*
>> >> + * Sony Wave64 constants
>> >> + * Copyright (c) 2009 Daniel Verkamp
>> >> + *
>> >> + * This file is part of FFmpeg.
>> >> + *
>> >> + * FFmpeg is free software; you can redistribute it and/or
>> >> + * modify it under the terms of the GNU Lesser General Public
>> >> + * License as published by the Free Software Foundation; either
>> >> + * version 2.1 of the License, or (at your option) any later version.
>> >> + *
>> >> + * FFmpeg is distributed in the hope that it will be useful,
>> >> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
>> >> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. ?See the GNU
>> >> + * Lesser General Public License for more details.
>> >> + *
>> >> + * You should have received a copy of the GNU Lesser General Public
>> >> + * License along with FFmpeg; if not, write to the Free Software
>> >> + * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
>> >> + */
>> >> +
>> >> +#ifndef LIBAVFORMAT_W64_H
>> >> +#define LIBAVFORMAT_W64_H
>> >> +
>> >> +#include "avformat.h"
>> >> +
>> >> +extern const uint8_t ff_w64_guid_riff[16];
>> >> +extern const uint8_t ff_w64_guid_wave[16];
>> >> +extern const uint8_t ff_w64_guid_fmt [16];
>> >> +extern const uint8_t ff_w64_guid_data[16];
>> >> +
>> >> +#endif /* LIBAVFORMAT_W64_H */
>> >
>> > that stuff is a little pointless, it could just be put in wav.c where it is used
>> >
>>
>> Moved to wav.c, ff_w64_ prefix removed.
>>
>> >
>> > [...]
>> >
>> >> +/* return the size of the found chunk */
>> >> +static int64_t find_guid(ByteIOContext *pb, const uint8_t guid1[16])
>> >
>> > doxy
>> >
>>
>> Fixed
>>
>> >
>> > [...]
>> >> @@ -209,10 +294,16 @@ static int wav_read_packet(AVFormatContext *s,
>> >>
>> >> ? ? ?left= wav->data_end - url_ftell(s->pb);
>> >> ? ? ?if(left <= 0){
>> >> + ? ? ? ?if (wav->w64) {
>> >> + ? ? ? ? ? ?left = find_guid(s->pb, ff_w64_guid_data);
>> >
>> > will fail to compile due to missing CONFIG_W64_DEMUXER
>> >
>>
>> Fixed
> [...]
>> @@ -209,6 +306,11 @@ static int wav_read_packet(AVFormatContext *s,
>>
>> ? ? ?left= wav->data_end - url_ftell(s->pb);
>> ? ? ?if(left <= 0){
>> +#if CONFIG_W64_DEMUXER
>> + ? ? ? ?if (wav->w64) {
>
> if(CONFIG_W64_DEMUXER && wav->w64) {
>

Ok... (does this actually work with all supported compilers?)
-------------- next part --------------
>From b44f5507b3987354c02c5f841cc59c34873bf7b5 Mon Sep 17 00:00:00 2001
From: Daniel Verkamp <daniel at drv.nu>
Date: Fri, 7 Aug 2009 16:27:35 -0400
Subject: [PATCH] Wave64 demuxer

---
 libavformat/Makefile     |    1 +
 libavformat/allformats.c |    1 +
 libavformat/wav.c        |  116 +++++++++++++++++++++++++++++++++++++++++++++-
 3 files changed, 117 insertions(+), 1 deletions(-)

diff --git a/libavformat/Makefile b/libavformat/Makefile
index 7838efe..7280c77 100644
--- a/libavformat/Makefile
+++ b/libavformat/Makefile
@@ -216,6 +216,7 @@ OBJS-$(CONFIG_VMD_DEMUXER)               += sierravmd.o
 OBJS-$(CONFIG_VOC_DEMUXER)               += vocdec.o voc.o
 OBJS-$(CONFIG_VOC_MUXER)                 += vocenc.o voc.o
 OBJS-$(CONFIG_VQF_DEMUXER)               += vqf.o
+OBJS-$(CONFIG_W64_DEMUXER)               += wav.o riff.o raw.o
 OBJS-$(CONFIG_WAV_DEMUXER)               += wav.o riff.o raw.o
 OBJS-$(CONFIG_WAV_MUXER)                 += wav.o riff.o
 OBJS-$(CONFIG_WC3_DEMUXER)               += wc3movie.o
diff --git a/libavformat/allformats.c b/libavformat/allformats.c
index 569ef50..0990598 100644
--- a/libavformat/allformats.c
+++ b/libavformat/allformats.c
@@ -193,6 +193,7 @@ void av_register_all(void)
     REGISTER_DEMUXER  (VMD, vmd);
     REGISTER_MUXDEMUX (VOC, voc);
     REGISTER_DEMUXER  (VQF, vqf);
+    REGISTER_DEMUXER  (W64, w64);
     REGISTER_MUXDEMUX (WAV, wav);
     REGISTER_DEMUXER  (WC3, wc3);
     REGISTER_DEMUXER  (WSAUD, wsaud);
diff --git a/libavformat/wav.c b/libavformat/wav.c
index 051cd84..2caa155 100644
--- a/libavformat/wav.c
+++ b/libavformat/wav.c
@@ -2,6 +2,9 @@
  * WAV muxer and demuxer
  * Copyright (c) 2001, 2002 Fabrice Bellard
  *
+ * Sony Wave64 demuxer
+ * Copyright (c) 2009 Daniel Verkamp
+ *
  * This file is part of FFmpeg.
  *
  * FFmpeg is free software; you can redistribute it and/or
@@ -28,6 +31,7 @@ typedef struct {
     int64_t minpts;
     int64_t maxpts;
     int last_duration;
+    int w64;
 } WAVContext;
 
 #if CONFIG_WAV_MUXER
@@ -194,12 +198,105 @@ static int wav_read_header(AVFormatContext *s,
     return 0;
 }
 
+#if CONFIG_W64_DEMUXER
+
+static const uint8_t guid_riff[16] = { 'r', 'i', 'f', 'f',
+    0x2E, 0x91, 0xCF, 0x11, 0xA5, 0xD6, 0x28, 0xDB, 0x04, 0xC1, 0x00, 0x00 };
+
+static const uint8_t guid_wave[16] = { 'w', 'a', 'v', 'e',
+    0xF3, 0xAC, 0xD3, 0x11, 0x8C, 0xD1, 0x00, 0xC0, 0x4F, 0x8E, 0xDB, 0x8A };
+
+static const uint8_t guid_fmt [16] = { 'f', 'm', 't', ' ',
+    0xF3, 0xAC, 0xD3, 0x11, 0x8C, 0xD1, 0x00, 0xC0, 0x4F, 0x8E, 0xDB, 0x8A };
+
+static const uint8_t guid_data[16] = { 'd', 'a', 't', 'a',
+    0xF3, 0xAC, 0xD3, 0x11, 0x8C, 0xD1, 0x00, 0xC0, 0x4F, 0x8E, 0xDB, 0x8A };
+
+static int w64_probe(AVProbeData *p)
+{
+    if (p->buf_size <= 40)
+        return 0;
+    if (!memcmp(p->buf,      guid_riff, 16) &&
+        !memcmp(p->buf + 24, guid_wave, 16)) {
+        return AVPROBE_SCORE_MAX;
+    } else
+        return 0;
+}
+
+/** Find chunk with w64 GUID by skipping over other chunks
+ * @return the size of the found chunk
+ */
+static int64_t find_guid(ByteIOContext *pb, const uint8_t guid1[16])
+{
+    uint8_t guid[16];
+    int64_t size;
+
+    while (!url_feof(pb)) {
+        get_buffer(pb, guid, 16);
+        size = get_le64(pb);
+        if (!memcmp(guid, guid1, 16))
+            return size;
+        url_fskip(pb, FFALIGN(size, INT64_C(8)));
+    }
+    return -1;
+}
+
+static int w64_read_header(AVFormatContext *s, AVFormatParameters *ap)
+{
+    int64_t size;
+    ByteIOContext *pb = s->pb;
+    WAVContext *wav   = s->priv_data;
+    AVStream *st;
+    uint8_t guid[16];
+
+    get_buffer(pb, guid, 16);
+    if (memcmp(guid, guid_riff, 16))
+        return -1;
+
+    if (get_le64(pb) < 16 + 8 + 16 + 8 + 16 + 8) /* riff + wave + fmt + sizes */
+        return -1;
+
+    get_buffer(pb, guid, 16);
+    if (memcmp(guid, guid_wave, 16))
+        return -1;
+
+    size = find_guid(pb, guid_fmt);
+    if (size < 0) {
+        av_log(s, AV_LOG_ERROR, "could not find fmt guid\n");
+        return -1;
+    }
+
+    st = av_new_stream(s, 0);
+    if (!st)
+        return AVERROR(ENOMEM);
+
+    /* subtract chunk header size - normal wav file doesn't count it */
+    ff_get_wav_header(pb, st->codec, size - 24);
+    url_fskip(pb, FFALIGN(size, INT64_C(8)) - size);
+
+    st->need_parsing = AVSTREAM_PARSE_FULL;
+
+    av_set_pts_info(st, 64, 1, st->codec->sample_rate);
+
+    size = find_guid(pb, guid_data);
+    if (size < 0) {
+        av_log(s, AV_LOG_ERROR, "could not find data guid\n");
+        return -1;
+    }
+    wav->data_end = url_ftell(pb) + size - 24;
+    wav->w64 = 1;
+
+    return 0;
+}
+#endif /* CONFIG_W64_DEMUXER */
+
 #define MAX_SIZE 4096
 
 static int wav_read_packet(AVFormatContext *s,
                            AVPacket *pkt)
 {
-    int ret, size, left;
+    int ret, size;
+    int64_t left;
     AVStream *st;
     WAVContext *wav = s->priv_data;
 
@@ -209,6 +306,9 @@ static int wav_read_packet(AVFormatContext *s,
 
     left= wav->data_end - url_ftell(s->pb);
     if(left <= 0){
+        if (CONFIG_W64_DEMUXER && wav->w64) {
+            left = find_guid(s->pb, guid_data);
+        } else
         left = find_tag(s->pb, MKTAG('d', 'a', 't', 'a'));
         if (left < 0) {
             return AVERROR(EIO);
@@ -282,3 +382,17 @@ AVOutputFormat wav_muxer = {
     .codec_tag= (const AVCodecTag* const []){ff_codec_wav_tags, 0},
 };
 #endif
+#if CONFIG_W64_DEMUXER
+AVInputFormat w64_demuxer = {
+    "w64",
+    NULL_IF_CONFIG_SMALL("Sony Wave64 format"),
+    sizeof(WAVContext),
+    w64_probe,
+    w64_read_header,
+    wav_read_packet,
+    NULL,
+    wav_read_seek,
+    .flags = AVFMT_GENERIC_INDEX,
+    .codec_tag = (const AVCodecTag* const []){ff_codec_wav_tags, 0},
+};
+#endif
-- 
1.6.4



More information about the ffmpeg-devel mailing list