fixed SIZEOF_LONG==4 assumption with ENABLE_MEM_CHECK, from Art Haas

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.
This commit is contained in:
Tim Janik 2000-05-19 08:18:13 +00:00 committed by Tim Janik
parent 43acfbf619
commit b3278400bc
24 changed files with 158 additions and 60 deletions

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View 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-18 Sebastian Wilhelmi <wilhelmi@ira.uka.de>
* configure.in: Use strlen instead of exit in the test for

View File

@ -170,7 +170,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 (array->data + array->elt_size * index,
@ -194,7 +194,7 @@ g_array_remove_index_fast (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 (array->data + array->elt_size * index,
@ -340,7 +340,7 @@ g_ptr_array_remove_index (GPtrArray* 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);
result = array->pdata[index];
@ -364,7 +364,7 @@ g_ptr_array_remove_index_fast (GPtrArray* 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);
result = array->pdata[index];

View 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)

View File

@ -170,7 +170,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 (array->data + array->elt_size * index,
@ -194,7 +194,7 @@ g_array_remove_index_fast (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 (array->data + array->elt_size * index,
@ -340,7 +340,7 @@ g_ptr_array_remove_index (GPtrArray* 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);
result = array->pdata[index];
@ -364,7 +364,7 @@ g_ptr_array_remove_index_fast (GPtrArray* 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);
result = array->pdata[index];

View 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)

View File

@ -1342,7 +1342,8 @@ g_idle_prepare (gpointer source_data,
gint *timeout,
gpointer user_data)
{
timeout = 0;
*timeout = 0;
return TRUE;
}

View 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 */
}

View File

@ -1613,9 +1613,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])
{
@ -1625,7 +1624,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 &&

View File

@ -348,25 +348,19 @@ g_slist_copy (GSList *list)
GSList*
g_slist_reverse (GSList *list)
{
GSList *tmp;
GSList *prev;
GSList *last;
last = NULL;
prev = NULL;
GSList *prev = NULL;
while (list)
{
last = list;
GSList *next = list->next;
tmp = list->next;
list->next = prev;
prev = list;
list = tmp;
list = next;
}
return last;
return prev;
}
GSList*

View File

@ -682,12 +682,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;

View File

@ -1342,7 +1342,8 @@ g_idle_prepare (gpointer source_data,
gint *timeout,
gpointer user_data)
{
timeout = 0;
*timeout = 0;
return TRUE;
}

6
gmem.c
View 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 */
}

View File

@ -1613,9 +1613,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])
{
@ -1625,7 +1624,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 &&

View File

@ -348,25 +348,19 @@ g_slist_copy (GSList *list)
GSList*
g_slist_reverse (GSList *list)
{
GSList *tmp;
GSList *prev;
GSList *last;
last = NULL;
prev = NULL;
GSList *prev = NULL;
while (list)
{
last = list;
GSList *next = list->next;
tmp = list->next;
list->next = prev;
prev = list;
list = tmp;
list = next;
}
return last;
return prev;
}
GSList*

View File

@ -682,12 +682,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;

View 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...");

View 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...");