From 1a3401f1be3f6e90a664eea13630725f36822fc9 Mon Sep 17 00:00:00 2001 From: Luis de Bethencourt Date: Thu, 16 Jun 2011 15:38:10 +0200 Subject: [PATCH] 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(). --- gst/goom/gstgoom.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/gst/goom/gstgoom.c b/gst/goom/gstgoom.c index 8b202318ac..78544492ad 100644 --- a/gst/goom/gstgoom.c +++ b/gst/goom/gstgoom.c @@ -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", DEFAULT_FPS_N, DEFAULT_FPS_D); - 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 */