Rewrite colorspace negotiation.
[ffmpeg.git] / libavfilter / defaults.c
index 50b13dc416efc8b088790725a70f2302a8e0c6f5..ae85b5bf2bd63355a39af5cd2742ec3c12222c2e 100644 (file)
@@ -28,13 +28,6 @@ void avfilter_default_free_video_buffer(AVFilterPic *pic)
     av_free(pic);
 }
 
-AVFilterPicRef *avfilter_next_get_video_buffer(AVFilterLink *link, int perms)
-{
-    if(!link->dst->outputs[0])
-        return NULL;
-    return avfilter_get_video_buffer(link->dst->outputs[0], perms);
-}
-
 /* TODO: set the buffer's priv member to a context structure for the whole
  * filter chain.  This will allow for a buffer pool instead of the constant
  * alloc & free cycle currently implemented. */
@@ -46,10 +39,9 @@ AVFilterPicRef *avfilter_default_get_video_buffer(AVFilterLink *link, int perms)
     ref->pic   = pic;
     ref->w     = link->w;
     ref->h     = link->h;
-    ref->perms = perms;
 
-    /* we always give the destination filter read access by default */
-    avfilter_add_pic_perms(ref, link->dst, AV_PERM_READ);
+    /* make sure the buffer gets read permission or it's useless for output */
+    ref->perms = perms | AV_PERM_READ;
 
     pic->refcount = 1;
     pic->format   = link->format;
@@ -72,7 +64,7 @@ void avfilter_default_start_frame(AVFilterLink *link, AVFilterPicRef *picref)
     if(out) {
         out->outpic      = avfilter_get_video_buffer(out, AV_PERM_WRITE);
         out->outpic->pts = picref->pts;
-        avfilter_start_frame(out, avfilter_ref_pic(out->outpic, out->dst, ~0));
+        avfilter_start_frame(out, avfilter_ref_pic(out->outpic, ~0));
     }
 }
 
@@ -106,8 +98,7 @@ int avfilter_default_config_output_link(AVFilterLink *link)
     } else {
         /* XXX: any non-simple filter which would cause this branch to be taken
          * really should implement its own config_props() for this link. */
-        link->w =
-        link->h = 0;
+        return -1;
     }
 
     return 0;
@@ -124,15 +115,40 @@ int avfilter_default_config_input_link(AVFilterLink *link)
 }
 
 /**
- * default query_formats() implementation for output video links to simplify
- * the implementation of one input one output video filters */
-int *avfilter_default_query_output_formats(AVFilterLink *link)
+ * A helper for query_formats() which sets all links to the same list of
+ * formats. If there are no links hooked to this filter, the list of formats is
+ * freed.
+ *
+ * FIXME: this will need changed for filters with a mix of pad types
+ * (video + audio, etc)
+ */
+void avfilter_set_common_formats(AVFilterContext *ctx, AVFilterFormats *formats)
 {
-    if(link->src->input_count && link->src->inputs[0])
-        return avfilter_make_format_list(1, link->src->inputs[0]->format);
-    else
-        /* XXX: any non-simple filter which would cause this branch to be taken
-         * really should implement its own query_formats() for this link */
-        return avfilter_make_format_list(0);
+    int count = 0, i;
+
+    for(i = 0; i < ctx->input_count; i ++) {
+        if(ctx->inputs[i]) {
+            avfilter_formats_ref(formats, &ctx->inputs[i]->out_formats);
+            count ++;
+        }
+    }
+    for(i = 0; i < ctx->output_count; i ++) {
+        if(ctx->outputs[i]) {
+            avfilter_formats_ref(formats, &ctx->outputs[i]->in_formats);
+            count ++;
+        }
+    }
+
+    if(!count) {
+        av_free(formats->formats);
+        av_free(formats->refs);
+        av_free(formats);
+    }
+}
+
+int avfilter_default_query_formats(AVFilterContext *ctx)
+{
+    avfilter_set_common_formats(ctx, avfilter_all_colorspaces());
+    return 0;
 }