avfilter/aresample: use init_dict system and set child AVOption context correctly
authorMichael Niedermayer <michaelni@gmx.at>
Thu, 11 Apr 2013 18:23:52 +0000 (20:23 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Thu, 11 Apr 2013 18:40:59 +0000 (20:40 +0200)
Parts of this commit are based on: 4fa1f52e

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavfilter/af_aresample.c
libavfilter/avfilter.c

index 80351c3..b88c5f8 100644 (file)
 #include "internal.h"
 
 typedef struct {
+    const AVClass *class;
+    int sample_rate_arg;
     double ratio;
     struct SwrContext *swr;
     int64_t next_pts;
     int req_fullfilled;
 } AResampleContext;
 
-static av_cold int init(AVFilterContext *ctx, const char *args)
+static av_cold int init_dict(AVFilterContext *ctx, AVDictionary **opts)
 {
     AResampleContext *aresample = ctx->priv;
     int ret = 0;
-    char *argd = av_strdup(args);
 
     aresample->next_pts = AV_NOPTS_VALUE;
     aresample->swr = swr_alloc();
@@ -54,27 +55,20 @@ static av_cold int init(AVFilterContext *ctx, const char *args)
         goto end;
     }
 
-    if (args) {
-        char *ptr = argd, *token;
-
-        while (token = av_strtok(ptr, ":", &ptr)) {
-            char *value;
-            av_strtok(token, "=", &value);
-
-            if (value) {
-                if ((ret = av_opt_set(aresample->swr, token, value, 0)) < 0)
-                    goto end;
-            } else {
-                int out_rate;
-                if ((ret = ff_parse_sample_rate(&out_rate, token, ctx)) < 0)
-                    goto end;
-                if ((ret = av_opt_set_int(aresample->swr, "osr", out_rate, 0)) < 0)
-                    goto end;
-            }
+    if (opts) {
+        AVDictionaryEntry *e = NULL;
+
+        while ((e = av_dict_get(*opts, "", e, AV_DICT_IGNORE_SUFFIX))) {
+            const char *token = e->key;
+            const char *value = e->value;
+            if ((ret = av_opt_set(aresample->swr, token, value, 0)) < 0)
+                goto end;
         }
+        av_dict_free(opts);
     }
+    if (aresample->sample_rate_arg > 0)
+        av_opt_set_int(aresample->swr, "osr", aresample->sample_rate_arg, 0);
 end:
-    av_free(argd);
     return ret;
 }
 
@@ -257,6 +251,34 @@ static int request_frame(AVFilterLink *outlink)
     return ret;
 }
 
+static const AVClass *resample_child_class_next(const AVClass *prev)
+{
+    return prev ? NULL : swr_get_class();
+}
+
+static void *resample_child_next(void *obj, void *prev)
+{
+    AResampleContext *s = obj;
+    return prev ? NULL : s->swr;
+}
+
+#define OFFSET(x) offsetof(AResampleContext, x)
+#define FLAGS AV_OPT_FLAG_AUDIO_PARAM|AV_OPT_FLAG_FILTERING_PARAM
+
+static const AVOption options[] = {
+    {"sample_rate", NULL, OFFSET(sample_rate_arg), AV_OPT_TYPE_INT, {.i64=0},  0,        INT_MAX, FLAGS },
+    {NULL}
+};
+
+static const AVClass aresample_class = {
+    .class_name       = "aresample",
+    .item_name        = av_default_item_name,
+    .option           = options,
+    .version          = LIBAVUTIL_VERSION_INT,
+    .child_class_next = resample_child_class_next,
+    .child_next       = resample_child_next,
+};
+
 static const AVFilterPad aresample_inputs[] = {
     {
         .name         = "default",
@@ -279,10 +301,11 @@ static const AVFilterPad aresample_outputs[] = {
 AVFilter avfilter_af_aresample = {
     .name          = "aresample",
     .description   = NULL_IF_CONFIG_SMALL("Resample audio data."),
-    .init          = init,
+    .init_dict     = init_dict,
     .uninit        = uninit,
     .query_formats = query_formats,
     .priv_size     = sizeof(AResampleContext),
+    .priv_class    = &aresample_class,
     .inputs        = aresample_inputs,
     .outputs       = aresample_outputs,
 };
index 3290402..9655fcf 100644 (file)
@@ -676,7 +676,6 @@ static int process_options(AVFilterContext *ctx, AVDictionary **options,
 static const char *const filters_left_to_update[] = {
     "abuffer",
     "aconvert",
-    "aresample",
     "atempo",
     "buffer",
     "flite",