[FFmpeg-devel] [PATCH] add dumpwave filter
Дмитрий Гуменюк
dmitry.gumenyuk at gmail.com
Wed Jan 10 11:16:57 EET 2018
> wavesurer.js - Web Audio API
I mean its would be hard to do the same for large files
https://developer.mozilla.org/en-US/docs/Web/API/BaseAudioContext/decodeAudioData
> On 10 Jan 2018, at 09:04, Дмитрий Гуменюк <dmitry.gumenyuk at gmail.com> wrote:
>
> Hi,
> While Waveform.js converts old SoundCloud PNGs, wavesurer.js is using Web Audio API which is limited/not supported by all browsers
>
>> On 10 Jan 2018, at 08:51, Дмитрий Гуменюк <dmitry.gumenyuk at gmail.com> wrote:
>>
>> There is no rush on this. Could you please do a code review so I can see how to do things properly?
>>> On 10 Jan 2018, at 08:43, Kyle Swanson <k at ylo.ph> wrote:
>>>
>>> Hi,
>>>
>>> On Tue, Jan 9, 2018 at 3:49 PM, <dmitry.gumenyuk at gmail.com> wrote:
>>>> From: Dmytro Humeniuk <dmitry.gumenyuk at gmail.com>
>>>>
>>>> Signed-off-by: Dmytro Humeniuk <dmitry.gumenyuk at gmail.com>
>>>> ---
>>>> Changelog | 1 +
>>>> doc/filters.texi | 23 ++++
>>>> libavfilter/Makefile | 1 +
>>>> libavfilter/af_dumpwave.c | 285
>>> +++++++++++++++++++++++++++++++++++++++++
>>>> libavfilter/allfilters.c | 1 +
>>>> libavfilter/version.h | 4 +-
>>>> tests/fate/filter-audio.mak | 5 +
>>>> tests/ref/fate/filter-dumpwave | 1 +
>>>> 8 files changed, 319 insertions(+), 2 deletions(-)
>>>> create mode 100644 libavfilter/af_dumpwave.c
>>>> create mode 100644 tests/ref/fate/filter-dumpwave
>>>
>>> I could see this possibly being a useful filter, but I'm confused about
>>> where the JSON schema came from. The two JS libraries that do this type of
>>> thing (waveform.js, and wavesurer.js) both just load waveform data as an
>>> array of floats. If we're going to add something like this to libavfilter
>>> it should be as generic and extensible as possible. I'm not wild about the
>>> string stuff, and the big sample format switch isn't necessary. I could do
>>> a code review, but it might just be faster if I rewrite it and send another
>>> patch. Is that OK with you?
>>>
>>> Thanks,
>>> Kyle
>>> _______________________________________________
>>> ffmpeg-devel mailing list
>>> ffmpeg-devel at ffmpeg.org
>>> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 3872 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180110/3b44e63b/attachment.bin>
More information about the ffmpeg-devel
mailing list