Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-31 Thread Greg Rowe
On 2016-10-18 2:20 PM, Michael Niedermayer wrote: On Tue, Oct 18, 2016 at 12:46:56PM -0400, Greg Rowe wrote: see libavfilter/asrc_sine.c this code should probably be reused / factored (note, any code moving/factoring of existing code should be in a seperate patch) Since silenceremove works

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-18 Thread Greg Rowe
the tone and then convert it to double samples or some other approach? Thanks, Greg -- Greg Rowe www.shoretel.com ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-14 Thread Greg Rowe
On Fri, Oct 14, 2016 at 18:09:51 +, Greg Rowe wrote: You attached the old patch. ;-) Is there a good way to generate the tone while avoiding floating point operations? Well that's embarrassing. I'll fix that after we figure out how best to handle the tone generation. :)

Re: [FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-14 Thread Greg Rowe
ation to reflect that. I moved the tone generation to an initialization function and fill a buffer that exists for the duration of the filter instead of needlessly generating the tone on the fly. Thanks, Greg -- Greg Rowe www.shoretel.com From 41405e90cb2fb41441a6cf29c7a0d14362fd1b1f Mon

[FFmpeg-devel] [PATCH] avfilter/af_silenceremove: add optional tone when silence is removed

2016-10-13 Thread Greg Rowe
feature and tone_hz which allows you to specify the frequency of the tone. Thanks, Greg -- Greg Rowe www.shoretel.com From 41405e90cb2fb41441a6cf29c7a0d14362fd1b1f Mon Sep 17 00:00:00 2001 From: Greg Rowe Date: Fri, 7 Oct 2016 13:39:58 -0400 Subject: [PATCH] avfilter/af_silenceremove: add