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 c85bca961947e3029716125b367231051d016b4b..d84e78dc063b6df5655a5c9172107e4ca8e6a2e5 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,10 @@
+Tue Sep  8 19:42:58 1998  Stan Shebs  <shebs@andros.cygnus.com>
+
+       * 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  <shebs@andros.cygnus.com>
 
        * README: Update remote debugging and testsuite info.

diff --git a/gdb/symfile.c b/gdb/symfile.c
index 5d4c85856d9c549fbaa7df27d27643c2ea2d3699..6b00ed941d384d2e16123680db97619d01b08a60 100644 (file)
--- 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