diff options
-rw-r--r-- | gdb/ChangeLog | 13 | ||||
-rw-r--r-- | gdb/macroexp.c | 10 | ||||
-rw-r--r-- | gdb/macrotab.c | 14 |
3 files changed, 25 insertions, 12 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 82626f3..bfa0c1d 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,16 @@ +2002-05-14 Jim Blandy <jimb@redhat.com> + + * macroexp.c (init_buffer, gather_arguments, expand): Use NULL, not 0. + * macrotab.c (macro_lookup_inclusion, find_definition, + new_macro_table): Same. + + * macroexp.c (currently_rescanning, expand): Use `strcmp () == 0', + not `! strcmp ()'. This is a dubious improvement. + * macrotab.c (macro_lookup_inclusion, find_definition): Same. + + * macrotab.c (macro_lookup_inclusion): Initialize `best_depth', + although it's not necessary, to avoid a warning. + 2002-05-14 Daniel Jacobowitz <drow@mvista.com> * gdbtypes.h: Update accessor macros to use TYPE_MAIN_TYPE. diff --git a/gdb/macroexp.c b/gdb/macroexp.c index cb97f42..061c63a 100644 --- a/gdb/macroexp.c +++ b/gdb/macroexp.c @@ -88,7 +88,7 @@ init_buffer (struct macro_buffer *b, int n) if (n > 0) b->text = (char *) xmalloc (n); else - b->text = 0; + b->text = NULL; b->len = 0; b->shared = 0; b->last_token = -1; @@ -646,7 +646,7 @@ static int currently_rescanning (struct macro_name_list *list, const char *name) { for (; list; list = list->next) - if (! strcmp (name, list->name)) + if (strcmp (name, list->name) == 0) return 1; return 0; @@ -692,7 +692,7 @@ gather_arguments (const char *name, struct macro_buffer *src, int *argc_p) { struct macro_buffer tok; int args_len, args_size; - struct macro_buffer *args = 0; + struct macro_buffer *args = NULL; struct cleanup *back_to = make_cleanup (free_current_contents, &args); /* Does SRC start with an opening paren token? Read from a copy of @@ -928,12 +928,12 @@ expand (const char *id, { struct cleanup *back_to = make_cleanup (null_cleanup, 0); int argc; - struct macro_buffer *argv = 0; + struct macro_buffer *argv = NULL; struct macro_buffer substituted; struct macro_buffer substituted_src; if (def->argc >= 1 - && ! strcmp (def->argv[def->argc - 1], "...")) + && strcmp (def->argv[def->argc - 1], "...") == 0) error ("Varargs macros not implemented yet."); make_cleanup (free_current_contents, &argv); diff --git a/gdb/macrotab.c b/gdb/macrotab.c index d73ec9e..d979a91 100644 --- a/gdb/macrotab.c +++ b/gdb/macrotab.c @@ -479,7 +479,7 @@ struct macro_source_file * macro_lookup_inclusion (struct macro_source_file *source, const char *name) { /* Is SOURCE itself named NAME? */ - if (! strcmp (name, source->filename)) + if (strcmp (name, source->filename) == 0) return source; /* The filename in the source structure is probably a full path, but @@ -493,15 +493,15 @@ macro_lookup_inclusion (struct macro_source_file *source, const char *name) check for a slash here. */ if (name_len < src_name_len && source->filename[src_name_len - name_len - 1] == '/' - && ! strcmp (name, source->filename + src_name_len - name_len)) + && strcmp (name, source->filename + src_name_len - name_len) == 0) return source; } /* It's not us. Try all our children, and return the lowest. */ { struct macro_source_file *child; - struct macro_source_file *best = 0; - int best_depth; + struct macro_source_file *best = NULL; + int best_depth = 0; for (child = source->includes; child; child = child->next_included) { @@ -618,7 +618,7 @@ find_definition (const char *name, query.name = name; query.start_file = file; query.start_line = line; - query.end_file = 0; + query.end_file = NULL; n = splay_tree_lookup (t->definitions, (splay_tree_key) &query); if (! n) @@ -638,7 +638,7 @@ find_definition (const char *name, We just want to search within a given name's definitions. */ struct macro_key *found = (struct macro_key *) pred->key; - if (! strcmp (found->name, name)) + if (strcmp (found->name, name) == 0) n = pred; } } @@ -838,7 +838,7 @@ new_macro_table (struct obstack *obstack, memset (t, 0, sizeof (*t)); t->obstack = obstack; t->bcache = b; - t->main_source = 0; + t->main_source = NULL; t->definitions = (splay_tree_new_with_allocator (macro_tree_compare, ((splay_tree_delete_key_fn) macro_tree_delete_key), |