[glib] Remove bits of dead code identified by coverage tests



commit 1fe4429318b147f275e730ef5a69cb94a0a25062
Author: Matthias Clasen <mclasen redhat com>
Date:   Mon Oct 3 23:54:08 2011 -0400

    Remove bits of dead code identified by coverage tests
    
    These lines could were not hit by our tests, and examination
    of the code reveals that they can't ever be hit.

 glib/glist.c  |   16 ++++++----------
 glib/gslist.c |   12 ++----------
 2 files changed, 8 insertions(+), 20 deletions(-)
---
diff --git a/glib/glist.c b/glib/glist.c
index f75e2af..91794d9 100644
--- a/glib/glist.c
+++ b/glib/glist.c
@@ -310,28 +310,24 @@ g_list_insert (GList	*list,
 {
   GList *new_list;
   GList *tmp_list;
-  
+
   if (position < 0)
     return g_list_append (list, data);
   else if (position == 0)
     return g_list_prepend (list, data);
-  
+
   tmp_list = g_list_nth (list, position);
   if (!tmp_list)
     return g_list_append (list, data);
-  
+
   new_list = _g_list_alloc ();
   new_list->data = data;
   new_list->prev = tmp_list->prev;
-  if (tmp_list->prev)
-    tmp_list->prev->next = new_list;
+  tmp_list->prev->next = new_list;
   new_list->next = tmp_list;
   tmp_list->prev = new_list;
-  
-  if (tmp_list == list)
-    return new_list;
-  else
-    return list;
+
+  return list;
 }
 
 /**
diff --git a/glib/gslist.c b/glib/gslist.c
index 6a11669..e99971e 100644
--- a/glib/gslist.c
+++ b/glib/gslist.c
@@ -316,16 +316,8 @@ g_slist_insert (GSList   *list,
       tmp_list = tmp_list->next;
     }
 
-  if (prev_list)
-    {
-      new_list->next = prev_list->next;
-      prev_list->next = new_list;
-    }
-  else
-    {
-      new_list->next = list;
-      list = new_list;
-    }
+  new_list->next = prev_list->next;
+  prev_list->next = new_list;
 
   return list;
 }



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]