gst-plugins-bad: fieldanalysis: Don't reset the context on DISCONT

Robert Swain robswain at kemper.freedesktop.org
Mon Oct 31 10:20:53 PDT 2011


Module: gst-plugins-bad
Branch: master
Commit: e3b7cb8899b800fbc5d26837403c3d20edd7f137
URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-bad/commit/?id=e3b7cb8899b800fbc5d26837403c3d20edd7f137

Author: Robert Swain <robert.swain at collabora.co.uk>
Date:   Fri Oct 28 11:10:06 2011 +0200

fieldanalysis: Don't reset the context on DISCONT

On discont, the queue of frames was being cleared, but also the
context reset. We only need to clear the queue.

---

 gst/fieldanalysis/gstfieldanalysis.c |   11 ++++++++---
 1 files changed, 8 insertions(+), 3 deletions(-)

diff --git a/gst/fieldanalysis/gstfieldanalysis.c b/gst/fieldanalysis/gstfieldanalysis.c
index 7f8a059..3cfd5c8 100644
--- a/gst/fieldanalysis/gstfieldanalysis.c
+++ b/gst/fieldanalysis/gstfieldanalysis.c
@@ -313,9 +313,8 @@ static guint64 block_score_for_row_5_tap (GstFieldAnalysis * filter,
 static gfloat opposite_parity_windowed_comb (GstFieldAnalysis * filter,
     FieldAnalysisFields * fields);
 
-
 static void
-gst_field_analysis_reset (GstFieldAnalysis * filter)
+gst_field_analysis_empty_queue (GstFieldAnalysis * filter)
 {
   if (filter->frames) {
     guint length = g_queue_get_length (filter->frames);
@@ -327,6 +326,12 @@ gst_field_analysis_reset (GstFieldAnalysis * filter)
       length--;
     }
   }
+}
+
+static void
+gst_field_analysis_reset (GstFieldAnalysis * filter)
+{
+  gst_field_analysis_empty_queue (filter);
   GST_DEBUG_OBJECT (filter, "Resetting context");
   memset (filter->results, 0, 2 * sizeof (FieldAnalysis));
   filter->is_telecine = FALSE;
@@ -1711,7 +1716,7 @@ gst_field_analysis_chain (GstPad * pad, GstBuffer * buf)
       }
     }
 
-    gst_field_analysis_reset (filter);
+    gst_field_analysis_empty_queue (filter);
 
     if (ret != GST_FLOW_OK) {
       GST_DEBUG_OBJECT (filter,



More information about the gstreamer-commits mailing list