avfilter/af_silenceremove: use duration option for window option

Paul B Mahol committed 5 years ago

Commit 4e1ee161 parent 15e844e1 file · +21 −18Browse files

Showing the changes to libavfilter (1 of 1 file). Show all

Unified · Split

     int64_t stop_silence_opt;
     int stop_mode;
+    int64_t window_duration_opt;
     AVFrame *start_holdoff;
     AVFrame *start_silence_hold;
     size_t start_holdoff_offset;
     size_t stop_silence_end;
     int    stop_found_periods;
-    double window_ratio;
     AVFrame *window;
     int window_offset;
-    int window_size;
+    int64_t window_duration;
     double sum;
     int restart;
     { "detection",       "set how silence is detected",                        OFFSET(detection),           AV_OPT_TYPE_INT,      {.i64=D_RMS}, D_PEAK,D_RMS, AF, "detection" },
     {   "peak",          "use absolute values of samples",                     0,                           AV_OPT_TYPE_CONST,    {.i64=D_PEAK},0,         0, AF, "detection" },
     {   "rms",           "use squared values of samples",                      0,                           AV_OPT_TYPE_CONST,    {.i64=D_RMS}, 0,         0, AF, "detection" },
-    { "window",          "set duration of window in seconds",                  OFFSET(window_ratio),        AV_OPT_TYPE_DOUBLE,   {.dbl=0.02},  0,        10, AF },
+    { "window",          "set duration of window for silence detection",       OFFSET(window_duration_opt), AV_OPT_TYPE_DURATION, {.i64=20000}, 0, 100000000, AF },
     { NULL }
 };
     new_sum -= wsample;
     new_sum += fabs(sample);
-    return new_sum / s->window_size;
+    return new_sum / s->window_duration;
 }
 static void update_peak_double(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
     new_sum -= wsample;
     new_sum += fabsf(sample);
-    return new_sum / s->window_size;
+    return new_sum / s->window_duration;
 }
 static void update_peak_float(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
     new_sum -= wsample;
     new_sum += sample * sample;
-    return sqrt(new_sum / s->window_size);
+    return sqrt(new_sum / s->window_duration);
 }
 static void update_rms_double(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
     new_sum -= wsample;
     new_sum += sample * sample;
-    return sqrtf(new_sum / s->window_size);
+    return sqrtf(new_sum / s->window_duration);
 }
 static void update_rms_float(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
     new_sum -= wsample;
     new_sum += fabs(sample);
-    return new_sum / s->window_size;
+    return new_sum / s->window_duration;
 }
 static void update_peak_doublep(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
     new_sum -= wsample;
     new_sum += fabsf(sample);
-    return new_sum / s->window_size;
+    return new_sum / s->window_duration;
 }
 static void update_peak_floatp(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
     new_sum -= wsample;
     new_sum += sample * sample;
-    return sqrt(new_sum / s->window_size);
+    return sqrt(new_sum / s->window_duration);
 }
 static void update_rms_doublep(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
     new_sum -= wsample;
     new_sum += sample * sample;
-    return sqrtf(new_sum / s->window_size);
+    return sqrtf(new_sum / s->window_duration);
 }
 static void update_rms_floatp(SilenceRemoveContext *s, AVFrame *frame, int ch, int offset)
 static void clear_window(SilenceRemoveContext *s)
 {
-    av_samples_set_silence(s->window->extended_data, 0, s->window_size,
+    av_samples_set_silence(s->window->extended_data, 0, s->window_duration,
                            s->window->channels, s->window->format);
     s->window_offset = 0;
     SilenceRemoveContext *s = ctx->priv;
     s->next_pts = AV_NOPTS_VALUE;
-    s->window_size = FFMAX((inlink->sample_rate * s->window_ratio), 1);
+    s->window_duration = av_rescale(s->window_duration_opt, inlink->sample_rate,
-    s->window = ff_get_audio_buffer(ctx->outputs[0], s->window_size);
+                                   AV_TIME_BASE);
+    s->window_duration = FFMAX(1, s->window_duration);
+    s->window = ff_get_audio_buffer(ctx->outputs[0], s->window_duration);
     if (!s->window)
         return AVERROR(ENOMEM);
                 }
                 s->window_offset++;
-                if (s->window_offset >= s->window_size)
+                if (s->window_offset >= s->window_duration)
                     s->window_offset = 0;
                 s->start_holdoff_end++;
                 nb_samples_read++;
                 }
                 s->window_offset++;
-                if (s->window_offset >= s->window_size)
+                if (s->window_offset >= s->window_duration)
                     s->window_offset = 0;
                 nb_samples_read++;
                 s->start_silence_offset++;
                     }
                     s->window_offset++;
-                    if (s->window_offset >= s->window_size)
+                    if (s->window_offset >= s->window_duration)
                         s->window_offset = 0;
                     nb_samples_read++;
                     nb_samples_written++;
                     }
                     s->window_offset++;
-                    if (s->window_offset >= s->window_size)
+                    if (s->window_offset >= s->window_duration)
                         s->window_offset = 0;
                     nb_samples_read++;
                     s->stop_holdoff_end++;

avfilter/af_silenceremove: use duration option for window option

Keyboard shortcuts

General

Issues and pull requests

Issues, pull requests and Inbox

Repository code