fixed SIZEOF_LONG==4 assumption with ENABLE_MEM_CHECK, from Art Haas
authorTim Janik <timj@gtk.org>
Fri, 19 May 2000 08:18:29 +0000 (08:18 +0000)
committerTim Janik <timj@src.gnome.org>
Fri, 19 May 2000 08:18:29 +0000 (08:18 +0000)
Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>

        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.

        * gslist.c (g_slist_reverse): shut up compiler.

        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.

        * garray.c: remove index>=0 checks for unsigned indices.

        * gmain.c (g_idle_prepare): timeout assignment fix.

        * gtree.c (g_tree_node_rotate_right): shut up compiler.

24 files changed:
ChangeLog
ChangeLog.pre-2-0
ChangeLog.pre-2-10
ChangeLog.pre-2-12
ChangeLog.pre-2-2
ChangeLog.pre-2-4
ChangeLog.pre-2-6
ChangeLog.pre-2-8
garray.c
gcompletion.c
glib/garray.c
glib/gcompletion.c
glib/gmain.c
glib/gmem.c
glib/gscanner.c
glib/gslist.c
glib/gtree.c
gmain.c
gmem.c
gscanner.c
gslist.c
gtree.c
testgdate.c
tests/testgdate.c

index 42d36db..1dd862e 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index 42d36db..1dd862e 100644 (file)
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index 42d36db..1dd862e 100644 (file)
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index 42d36db..1dd862e 100644 (file)
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index 42d36db..1dd862e 100644 (file)
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index 42d36db..1dd862e 100644 (file)
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index 42d36db..1dd862e 100644 (file)
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index 42d36db..1dd862e 100644 (file)
@@ -1,3 +1,18 @@
+Fri May 19 09:00:44 2000  Tim Janik  <timj@gtk.org>
+
+        * gmem.c (g_free): fixed SIZEOF_LONG==4 assumption with
+        ENABLE_MEM_CHECK, from Art Haas <ahaas@neosoft.com>.
+
+        * gslist.c (g_slist_reverse): shut up compiler.
+
+        * gscanner.c (g_scanner_get_token_ll): removed inline assignment.
+
+        * garray.c: remove index>=0 checks for unsigned indices.
+
+        * gmain.c (g_idle_prepare): timeout assignment fix.
+
+        * gtree.c (g_tree_node_rotate_right): shut up compiler.
+
 2000-05-13  Tor Lillqvist  <tml@iki.fi>
 
        * makefile.mingw.in
index e90dcd4..749cd2f 100644 (file)
--- a/garray.c
+++ b/garray.c
@@ -209,7 +209,7 @@ g_array_remove_index (GArray* farray,
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   if (index != array->len - 1)
     g_memmove (g_array_elt_pos (array, index),
@@ -229,13 +229,13 @@ g_array_remove_index (GArray* farray,
 
 GArray*
 g_array_remove_index_fast (GArray* farray,
-                          guint index)
+                          guint   index)
 {
   GRealArray* array = (GRealArray*) farray;
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   if (index != array->len - 1)
     g_memmove (g_array_elt_pos (array, index), 
@@ -403,14 +403,14 @@ g_ptr_array_set_size  (GPtrArray   *farray,
 
 gpointer
 g_ptr_array_remove_index (GPtrArray* farray,
-                         guint index)
+                         guint      index)
 {
   GRealPtrArray* array = (GRealPtrArray*) farray;
   gpointer result;
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   result = array->pdata[index];
   
@@ -429,14 +429,14 @@ g_ptr_array_remove_index (GPtrArray* farray,
 
 gpointer
 g_ptr_array_remove_index_fast (GPtrArray* farray,
-                              guint index)
+                              guint      index)
 {
   GRealPtrArray* array = (GRealPtrArray*) farray;
   gpointer result;
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   result = array->pdata[index];
   
index 68617c1..9bc4c9b 100644 (file)
@@ -123,7 +123,7 @@ completion_check_cache (GCompletion* cmp,
   register gint len;
   register gint i;
   register gint plen;
-  gchar* postfix=NULL;
+  gchar* postfix;
   gchar* s;
   
   if (!new_prefix)
index e90dcd4..749cd2f 100644 (file)
@@ -209,7 +209,7 @@ g_array_remove_index (GArray* farray,
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   if (index != array->len - 1)
     g_memmove (g_array_elt_pos (array, index),
@@ -229,13 +229,13 @@ g_array_remove_index (GArray* farray,
 
 GArray*
 g_array_remove_index_fast (GArray* farray,
-                          guint index)
+                          guint   index)
 {
   GRealArray* array = (GRealArray*) farray;
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   if (index != array->len - 1)
     g_memmove (g_array_elt_pos (array, index), 
@@ -403,14 +403,14 @@ g_ptr_array_set_size  (GPtrArray   *farray,
 
 gpointer
 g_ptr_array_remove_index (GPtrArray* farray,
-                         guint index)
+                         guint      index)
 {
   GRealPtrArray* array = (GRealPtrArray*) farray;
   gpointer result;
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   result = array->pdata[index];
   
@@ -429,14 +429,14 @@ g_ptr_array_remove_index (GPtrArray* farray,
 
 gpointer
 g_ptr_array_remove_index_fast (GPtrArray* farray,
-                              guint index)
+                              guint      index)
 {
   GRealPtrArray* array = (GRealPtrArray*) farray;
   gpointer result;
 
   g_return_val_if_fail (array, NULL);
 
-  g_return_val_if_fail (index >= 0 && index < array->len, NULL);
+  g_return_val_if_fail (index < array->len, NULL);
 
   result = array->pdata[index];
   
index 68617c1..9bc4c9b 100644 (file)
@@ -123,7 +123,7 @@ completion_check_cache (GCompletion* cmp,
   register gint len;
   register gint i;
   register gint plen;
-  gchar* postfix=NULL;
+  gchar* postfix;
   gchar* s;
   
   if (!new_prefix)
index e7161c4..155ccbd 100644 (file)
@@ -1352,7 +1352,8 @@ g_idle_prepare  (gpointer  source_data,
                 gint     *timeout,
                 gpointer  user_data)
 {
-  timeout = 0;
+  *timeout = 0;
+
   return TRUE;
 }
 
index 4b9dfd9..41704bc 100644 (file)
@@ -402,11 +402,11 @@ g_free (gpointer mem)
 #ifdef ENABLE_MEM_CHECK
       t = (gulong*) ((guchar*) mem - SIZEOF_LONG);
       if (*t >= 1)
-       g_warning ("freeing previously freed memory\n");
+        g_warning ("freeing previously freed (%lu times) memory\n", *t);
       *t += 1;
       mem = t;
       
-      memset ((guchar*) mem + 8, 0, size);
+      memset ((guchar*) mem + 2 * SIZEOF_LONG, 0, size);
 #else /* ENABLE_MEM_CHECK */
       free (mem);
 #endif /* ENABLE_MEM_CHECK */
@@ -456,7 +456,7 @@ g_mem_check (gpointer mem)
   t = (gulong*) ((guchar*) mem - SIZEOF_LONG - SIZEOF_LONG);
   
   if (*t >= 1)
-    g_warning ("mem: 0x%08x has been freed %lu times\n", (gulong) mem, *t);
+    g_warning ("mem: 0x%08lx has been freed %lu times\n", (gulong) mem, *t);
 #endif /* ENABLE_MEM_CHECK */
 }
 
index 55d5fb2..55c311d 100644 (file)
@@ -1614,9 +1614,8 @@ g_scanner_get_token_ll    (GScanner       *scanner,
              token = G_TOKEN_COMMENT_SINGLE;
              in_comment_single = TRUE;
              gstring = g_string_new ("");
-             while ((ch = g_scanner_get_char (scanner,
-                                              line_p,
-                                              position_p)) != 0)
+             ch = g_scanner_get_char (scanner, line_p, position_p);
+             while (ch != 0)
                {
                  if (ch == config->cpair_comment_single[1])
                    {
@@ -1626,7 +1625,7 @@ g_scanner_get_token_ll    (GScanner       *scanner,
                    }
                  
                  gstring = g_string_append_c (gstring, ch);
-                 ch = 0;
+                 ch = g_scanner_get_char (scanner, line_p, position_p);
                }
            }
          else if (config->scan_identifier && ch &&
index cc8e906..341d6bd 100644 (file)
@@ -390,11 +390,11 @@ GSList*
 g_slist_reverse (GSList *list)
 {
   GSList *prev = NULL;
-  GSList *next = NULL;
   
   while (list)
     {
-      next = list->next;
+      GSList *next = list->next;
+
       list->next = prev;
       
       prev = list;
index fa40b77..229bdf5 100644 (file)
@@ -693,12 +693,10 @@ static GTreeNode*
 g_tree_node_rotate_right (GTreeNode *node)
 {
   GTreeNode *left;
-  GTreeNode *right;
   gint a_bal;
   gint b_bal;
 
   left = node->left;
-  right = node->right;
 
   node->left = left->right;
   left->right = node;
diff --git a/gmain.c b/gmain.c
index e7161c4..155ccbd 100644 (file)
--- a/gmain.c
+++ b/gmain.c
@@ -1352,7 +1352,8 @@ g_idle_prepare  (gpointer  source_data,
                 gint     *timeout,
                 gpointer  user_data)
 {
-  timeout = 0;
+  *timeout = 0;
+
   return TRUE;
 }
 
diff --git a/gmem.c b/gmem.c
index 4b9dfd9..41704bc 100644 (file)
--- a/gmem.c
+++ b/gmem.c
@@ -402,11 +402,11 @@ g_free (gpointer mem)
 #ifdef ENABLE_MEM_CHECK
       t = (gulong*) ((guchar*) mem - SIZEOF_LONG);
       if (*t >= 1)
-       g_warning ("freeing previously freed memory\n");
+        g_warning ("freeing previously freed (%lu times) memory\n", *t);
       *t += 1;
       mem = t;
       
-      memset ((guchar*) mem + 8, 0, size);
+      memset ((guchar*) mem + 2 * SIZEOF_LONG, 0, size);
 #else /* ENABLE_MEM_CHECK */
       free (mem);
 #endif /* ENABLE_MEM_CHECK */
@@ -456,7 +456,7 @@ g_mem_check (gpointer mem)
   t = (gulong*) ((guchar*) mem - SIZEOF_LONG - SIZEOF_LONG);
   
   if (*t >= 1)
-    g_warning ("mem: 0x%08x has been freed %lu times\n", (gulong) mem, *t);
+    g_warning ("mem: 0x%08lx has been freed %lu times\n", (gulong) mem, *t);
 #endif /* ENABLE_MEM_CHECK */
 }
 
index 55d5fb2..55c311d 100644 (file)
@@ -1614,9 +1614,8 @@ g_scanner_get_token_ll    (GScanner       *scanner,
              token = G_TOKEN_COMMENT_SINGLE;
              in_comment_single = TRUE;
              gstring = g_string_new ("");
-             while ((ch = g_scanner_get_char (scanner,
-                                              line_p,
-                                              position_p)) != 0)
+             ch = g_scanner_get_char (scanner, line_p, position_p);
+             while (ch != 0)
                {
                  if (ch == config->cpair_comment_single[1])
                    {
@@ -1626,7 +1625,7 @@ g_scanner_get_token_ll    (GScanner       *scanner,
                    }
                  
                  gstring = g_string_append_c (gstring, ch);
-                 ch = 0;
+                 ch = g_scanner_get_char (scanner, line_p, position_p);
                }
            }
          else if (config->scan_identifier && ch &&
index cc8e906..341d6bd 100644 (file)
--- a/gslist.c
+++ b/gslist.c
@@ -390,11 +390,11 @@ GSList*
 g_slist_reverse (GSList *list)
 {
   GSList *prev = NULL;
-  GSList *next = NULL;
   
   while (list)
     {
-      next = list->next;
+      GSList *next = list->next;
+
       list->next = prev;
       
       prev = list;
diff --git a/gtree.c b/gtree.c
index fa40b77..229bdf5 100644 (file)
--- a/gtree.c
+++ b/gtree.c
@@ -693,12 +693,10 @@ static GTreeNode*
 g_tree_node_rotate_right (GTreeNode *node)
 {
   GTreeNode *left;
-  GTreeNode *right;
   gint a_bal;
   gint b_bal;
 
   left = node->left;
-  right = node->right;
 
   node->left = left->right;
   left->right = node;
index 25e8b15..f74e00a 100644 (file)
@@ -47,7 +47,6 @@ int main(int argc, char** argv)
   GDateDay day;
   gchar buf[101];
   gchar* loc;
-
   /* Try to get all the leap year cases. */
   GDateYear check_years[] = { 
     1, 2, 3, 4, 5, 6, 7, 8, 9, 10,
@@ -63,11 +62,9 @@ int main(int argc, char** argv)
     2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012,
     3000, 3001, 3002, 3998, 3999, 4000, 4001, 4002, 4003
   };
-                           
   guint n_check_years = sizeof(check_years)/sizeof(GDateYear);
-
-  guint i = 0;
-  gboolean discontinuity = FALSE;
+  guint i;
+  gboolean discontinuity;
 
   g_print("checking GDate...");
   
index 25e8b15..f74e00a 100644 (file)
@@ -47,7 +47,6 @@ int main(int argc, char** argv)
   GDateDay day;
   gchar buf[101];
   gchar* loc;
-
   /* Try to get all the leap year cases. */
   GDateYear check_years[] = { 
     1, 2, 3, 4, 5, 6, 7, 8, 9, 10,
@@ -63,11 +62,9 @@ int main(int argc, char** argv)
     2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012,
     3000, 3001, 3002, 3998, 3999, 4000, 4001, 4002, 4003
   };
-                           
   guint n_check_years = sizeof(check_years)/sizeof(GDateYear);
-
-  guint i = 0;
-  gboolean discontinuity = FALSE;
+  guint i;
+  gboolean discontinuity;
 
   g_print("checking GDate...");