From: Emmanuele Bassi <ebassi@linux.intel.com>
Date: Thu, 11 Feb 2010 15:08:19 +0000 (+0000)
Subject: analysis: ClutterAnimator/1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=667222c301ca0f19c570938055208e564e4d02ef;p=profile%2Fivi%2Fclutter.git

analysis: ClutterAnimator/1

Do not de-reference a void*; use a temporary variable -- after
checking the contents of the pointer. This actually simplifies
the readability and avoids pulling a Lisp with the parentheses.
---

diff --git a/clutter/clutter-animator.c b/clutter/clutter-animator.c
index 31b9c61..d9d98ec 100644
--- a/clutter/clutter-animator.c
+++ b/clutter/clutter-animator.c
@@ -551,6 +551,7 @@ static gfloat
 list_try_get_rel (GList *list,
                   gint   count)
 {
+  ClutterAnimatorKey *key;
   GList *iter = list;
   GList *best = list;
 
@@ -575,7 +576,14 @@ list_try_get_rel (GList *list,
         }
     }
 
-  return g_value_get_float (&(((ClutterAnimatorKey *)best->data)->value));
+  if (best != NULL && best->data != NULL)
+    {
+      key = best->data;
+
+      return g_value_get_float (&(key->value));
+    }
+
+  return 0;
 }
 
 static void