There is no need to check g_malloc() returned value as when the
function fails to allocate memory, it terminates the application.
---
 gattrib/src/x_gtksheet.c |    4 ----
 gnetlist/src/s_rename.c  |   14 --------------
 libgeda/src/s_basic.c    |    5 -----
 3 files changed, 0 insertions(+), 23 deletions(-)

diff --git a/gattrib/src/x_gtksheet.c b/gattrib/src/x_gtksheet.c
index 4f9ca8c..275fa62 100644
--- a/gattrib/src/x_gtksheet.c
+++ b/gattrib/src/x_gtksheet.c
@@ -323,10 +323,6 @@ void x_gtksheet_set_cell_text_color(GtkSheet *sheet, gint 
row, gint col,
   
   /* XXXXX  Attempt to set cell color */
   range = g_malloc(sizeof(GtkSheetRange));
-  if (range == NULL) {
-    g_error ("Can't malloc range");
-    return;
-  }
   range->row0 = row;
   range->rowi = row;
   range->col0 = col;
diff --git a/gnetlist/src/s_rename.c b/gnetlist/src/s_rename.c
index 745a84a..15eee0a 100644
--- a/gnetlist/src/s_rename.c
+++ b/gnetlist/src/s_rename.c
@@ -98,11 +98,6 @@ void s_rename_next_set(void)
     SET * new_set;
     
     new_set = g_malloc(sizeof(SET));
-    if (new_set == NULL) 
-    {
-        fprintf(stderr,"Unable to create a new rename set.\n");
-        exit(-1);
-    }
     memset(new_set,0,sizeof(SET));
     if (first_set)
     {
@@ -189,8 +184,6 @@ void s_rename_add(char *src, char *dest)
                        dest, temp->src, temp->dest, src, temp->dest);
 #endif
                 new_rename = g_malloc(sizeof(RENAME));
-               if (new_rename)
-               {
                     new_rename->next = NULL;
                      new_rename->src = g_strdup(src);
                      new_rename->dest = g_strdup(temp->dest);
@@ -204,7 +197,6 @@ void s_rename_add(char *src, char *dest)
                         last_set->last_rename->next = new_rename;
                         last_set->last_rename = new_rename;
                     } 
-                }
             }
             if (temp == last)
             {
@@ -218,15 +210,10 @@ void s_rename_add(char *src, char *dest)
        if (first_set == NULL)
        {
            new_set = g_malloc(sizeof(SET));
-           if (new_set)
-           {
                memset(new_set,0,sizeof(SET));
                first_set = last_set = new_set;
-           }
        }    
         new_rename = g_malloc(sizeof(RENAME));
-        if (new_rename)
-       {
             new_rename->next = NULL;
              new_rename->src = g_strdup(src);
              new_rename->dest = g_strdup(dest);
@@ -239,7 +226,6 @@ void s_rename_add(char *src, char *dest)
                 last_set->last_rename->next = new_rename;
                 last_set->last_rename = new_rename;
             } 
-        }
     }
 }
 
diff --git a/libgeda/src/s_basic.c b/libgeda/src/s_basic.c
index 531ce1a..a733e94 100644
--- a/libgeda/src/s_basic.c
+++ b/libgeda/src/s_basic.c
@@ -142,11 +142,6 @@ OBJECT *s_basic_init_object( char *name )
 
   new_node = (OBJECT *) g_malloc(sizeof(OBJECT));      
 
-  if (new_node == NULL) {
-    fprintf(stderr, "Could not perform malloc; something is broken or increase 
your process limits\n");
-    exit(-1);
-  }
-
   /* setup sid */
   new_node->sid = global_sid++;
   new_node->type = -1;
-- 
1.5.6




_______________________________________________
geda-dev mailing list
[email protected]
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev

Reply via email to