From: Stan Shebs Date: Wed, 9 Sep 1998 02:48:11 +0000 (+0000) Subject: Cleanup opportunity discovered while updating internals manual. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=1f926de145e7fd8211c58e4ae9d08916c16657e4;p=binutils-gdb.git Cleanup opportunity discovered while updating internals manual. * symtab.h (struct symtab): Remove EXTRA_SYMTAB_INFO hook, not currently used. * symfile.c (allocate_symtab): Deprecate use of INIT_EXTRA_SYMTAB_INFO here. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index c85bca96194..d84e78dc063 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,10 @@ +Tue Sep 8 19:42:58 1998 Stan Shebs + + * symtab.h (struct symtab): Remove EXTRA_SYMTAB_INFO hook, + not currently used. + * symfile.c (allocate_symtab): Deprecate use of + INIT_EXTRA_SYMTAB_INFO here. + Fri Sep 4 15:33:25 1998 Stan Shebs * README: Update remote debugging and testsuite info. diff --git a/gdb/symfile.c b/gdb/symfile.c index 5d4c85856d9..6b00ed941d3 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -656,13 +656,16 @@ symbol_file_add (name, from_tty, addr, mainline, mapped, readnow) /* We either created a new mapped symbol table, mapped an existing symbol table file which has not had initial symbol reading performed, or need to read an unmapped symbol table. */ - if (pre_add_symbol_hook) - pre_add_symbol_hook (name); if (from_tty || info_verbose) { - printf_filtered ("Reading symbols from %s...", name); - wrap_here (""); - gdb_flush (gdb_stdout); + if (pre_add_symbol_hook) + pre_add_symbol_hook (name); + else + { + printf_filtered ("Reading symbols from %s...", name); + wrap_here (""); + gdb_flush (gdb_stdout); + } } syms_from_objfile (objfile, addr, mainline, from_tty); } @@ -689,12 +692,15 @@ symbol_file_add (name, from_tty, addr, mainline, mapped, readnow) } } - if (post_add_symbol_hook) - post_add_symbol_hook (); if (from_tty || info_verbose) { - printf_filtered ("done.\n"); - gdb_flush (gdb_stdout); + if (post_add_symbol_hook) + post_add_symbol_hook (); + else + { + printf_filtered ("done.\n"); + gdb_flush (gdb_stdout); + } } new_symfile_objfile (objfile, mainline, from_tty); @@ -1199,11 +1205,12 @@ add_symbol_file_command (args, from_tty) } /* FIXME-32x64: Assumes text_addr fits in a long. */ - if (!query ("add symbol table from file \"%s\" at text_addr = %s?\n", - name, local_hex_string ((unsigned long)text_addr))) + if ((from_tty) + && (!query ("add symbol table from file \"%s\" at text_addr = %s?\n", + name, local_hex_string ((unsigned long)text_addr)))) error ("Not confirmed."); - symbol_file_add (name, 0, text_addr, 0, mapped, readnow); + symbol_file_add (name, from_tty, text_addr, 0, mapped, readnow); /* Getting new symbols may change our opinion about what is frameless. */ @@ -1458,7 +1465,6 @@ deduce_language_from_filename (filename) symtab->dirname symtab->free_code symtab->free_ptr - initialize any EXTRA_SYMTAB_INFO possibly free_named_symtabs (symtab->filename); */ @@ -1485,6 +1491,10 @@ allocate_symtab (filename, objfile) symtab -> next = objfile -> symtabs; objfile -> symtabs = symtab; + /* FIXME: This should go away. It is only defined for the Z8000, + and the Z8000 definition of this macro doesn't have anything to + do with the now-nonexistent EXTRA_SYMTAB_INFO macro, it's just + here for convenience. */ #ifdef INIT_EXTRA_SYMTAB_INFO INIT_EXTRA_SYMTAB_INFO (symtab); #endif