[FFmpeg-cvslog] r22596 - trunk/libavfilter/vf_vflip.c

[koorogi]

Author: koorogi
Date: Fri Mar 19 00:13:04 2010
New Revision: 22596

Log:
The vflip filter does not need to keep ownership of a picture reference.
Avoid creating and releasing useless extra picture references in the
start_frame and end_frame callbacks.

Modified:
   trunk/libavfilter/vf_vflip.c

Modified: trunk/libavfilter/vf_vflip.c
==============================================================================
--- trunk/libavfilter/vf_vflip.c	Fri Mar 19 00:13:02 2010	(r22595)
+++ trunk/libavfilter/vf_vflip.c	Fri Mar 19 00:13:04 2010	(r22596)
@@ -63,19 +63,18 @@ static AVFilterPicRef *get_video_buffer(
 static void start_frame(AVFilterLink *link, AVFilterPicRef *picref)
 {
     FlipContext *flip = link->dst->priv;
-    AVFilterPicRef *ref2 = avfilter_ref_pic(picref, ~0);
     int i;
 
     for (i = 0; i < 4; i ++) {
         int vsub = i == 1 || i == 2 ? flip->vsub : 0;
 
-        if (ref2->data[i]) {
-            ref2->data[i] += ((link->h >> vsub)-1) * ref2->linesize[i];
-            ref2->linesize[i] = -ref2->linesize[i];
+        if (picref->data[i]) {
+            picref->data[i] += ((link->h >> vsub)-1) * picref->linesize[i];
+            picref->linesize[i] = -picref->linesize[i];
         }
     }
 
-    avfilter_start_frame(link->dst->outputs[0], ref2);
+    avfilter_start_frame(link->dst->outputs[0], picref);
 }
 
 static void draw_slice(AVFilterLink *link, int y, int h, int slice_dir)
@@ -96,6 +95,7 @@ AVFilter avfilter_vf_vflip = {
                                     .get_video_buffer = get_video_buffer,
                                     .start_frame      = start_frame,
                                     .draw_slice       = draw_slice,
+                                    .end_frame        = avfilter_null_end_frame,
                                     .config_props     = config_input, },
                                   { .name = NULL}},
     .outputs   = (AVFilterPad[]) {{ .name             = "default",