audioaggregator: remove buffer!=NULL check
authorStefan Sauer <ensonic@users.sf.net>
Thu, 5 Oct 2017 15:54:34 +0000 (17:54 +0200)
committerStefan Sauer <ensonic@users.sf.net>
Thu, 5 Oct 2017 16:12:29 +0000 (18:12 +0200)
Acording to the logic this cannot happen (we already check this before). So
add a assert like we do above and remove the check. This make it clearer that
we check for the offset range.
Also remove a dead assignment since we reassign this a few lines below.

gst-libs/gst/audio/gstaudioaggregator.c

index fab852d..3447d08 100644 (file)
@@ -1287,11 +1287,11 @@ gst_audio_aggregator_aggregate (GstAggregator * agg, gboolean timeout)
       }
     }
 
+    g_assert (pad->priv->buffer);
 
     if (pad->priv->output_offset >= aagg->priv->offset
-        && pad->priv->output_offset <
-        aagg->priv->offset + blocksize && pad->priv->buffer) {
-      gboolean drop_buf = FALSE;
+        && pad->priv->output_offset < aagg->priv->offset + blocksize) {
+      gboolean drop_buf;
 
       GST_LOG_OBJECT (aggpad, "Mixing buffer for current offset");
       drop_buf = !gst_audio_aggregator_mix_buffer (aagg, pad, pad->priv->buffer,