aggregator: code cleanups
authorStefan Sauer <ensonic@users.sf.net>
Thu, 13 Jul 2017 19:55:55 +0000 (21:55 +0200)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 2 Dec 2017 15:10:27 +0000 (15:10 +0000)
Fix comment typos, some copy'n'paste in logging. Add more doc comments.

libs/gst/base/gstaggregator.c

index 315b8b8..11c7f0d 100644 (file)
@@ -356,6 +356,8 @@ static GstFlowReturn gst_aggregator_pad_chain_internal (GstAggregator * self,
  *
  * This method guarantees that @func will be called only once for each
  * sink pad.
+ *
+ * Returns: %FALSE if there are no sinkpads or if @func returned %FALSE
  */
 gboolean
 gst_aggregator_iterate_sinkpads (GstAggregator * self,
@@ -779,9 +781,8 @@ check_events (GstAggregator * self, GstAggregatorPad * pad, gpointer user_data)
       if (klass == NULL)
         klass = GST_AGGREGATOR_GET_CLASS (self);
 
-      GST_LOG_OBJECT (pad, "Processing %" GST_PTR_FORMAT, event);
-
       if (event) {
+        GST_LOG_OBJECT (pad, "Processing %" GST_PTR_FORMAT, event);
         gst_event_ref (event);
         ret = klass->sink_event (self, pad, event);
 
@@ -794,7 +795,7 @@ check_events (GstAggregator * self, GstAggregatorPad * pad, gpointer user_data)
       }
 
       if (query) {
-        GST_LOG_OBJECT (pad, "Processing %" GST_PTR_FORMAT, event);
+        GST_LOG_OBJECT (pad, "Processing %" GST_PTR_FORMAT, query);
         ret = klass->sink_query (self, pad, query);
 
         PAD_LOCK (pad);