Commit 1a3401f1 authored by Luis de Bethencourt's avatar Luis de Bethencourt Committed by Tim-Philipp Müller

goom: fix unused-but-set-compiler warnings

Remove unnecessary res variables, core checks existance
and type of these fields for us already via the template
caps, and we know that these fields exist because we've
fixated them before in _negotiate().
parent 3d6955d6
......@@ -186,12 +186,11 @@ static gboolean
gst_goom_sink_setcaps (GstGoom * goom, GstCaps * caps)
GstStructure *structure;
gboolean res;
structure = gst_caps_get_structure (caps, 0);
res = gst_structure_get_int (structure, "channels", &goom->channels);
res &= gst_structure_get_int (structure, "rate", &goom->rate);
gst_structure_get_int (structure, "channels", &goom->channels);
gst_structure_get_int (structure, "rate", &goom->rate);
goom->bps = goom->channels * sizeof (gint16);
......@@ -238,7 +237,6 @@ error:
static gboolean
gst_goom_src_negotiate (GstGoom * goom)
gboolean res;
GstCaps *othercaps, *target;
GstStructure *structure;
GstCaps *templ;
......@@ -271,7 +269,7 @@ gst_goom_src_negotiate (GstGoom * goom)
gst_structure_fixate_field_nearest_fraction (structure, "framerate",
res = gst_goom_src_setcaps (goom, target);
gst_goom_src_setcaps (goom, target);
/* try to get a bufferpool now */
/* find a pool for the negotiated caps now */
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment