2004-08-10 Andrew Cagney <cagney@gnu.org>

	* defs.h (xmfree): Delete.
	* utils.c (xmfree): Delete function.
	(xfree): Inline calls to xmfree and mfree.
	* symmisc.c (free_symtab_block, free_symtab): Use xfree.
	* symfile.c (reread_symbols, init_psymbol_list): Ditto.
	* source.c (forget_cached_source_info, find_and_open_source): Ditto.
	* somread.c (som_symfile_finish): Ditto.
	* objfiles.c (allocate_objfile, free_objfile):
	* nlmread.c (nlm_symfile_finish): Ditto.
	* hpread.c (hpread_symfile_finish): Ditto.
	* elfread.c (free_elfinfo, elf_symfile_finish): Ditto.
	* dbxread.c (dbx_symfile_finish, free_bincl_list): Ditto.
	* coffread.c (coff_symfile_finish): Ditto.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 9192d67f802cf2d8e3516bdd0bffd9344ef3b111..9043fffb8c5204c8d4f23114a18bbba44fd0ccf7 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,5 +1,19 @@
 2004-08-10  Andrew Cagney  <cagney@gnu.org>
 
+       * defs.h (xmfree): Delete.
+       * utils.c (xmfree): Delete function.
+       (xfree): Inline calls to xmfree and mfree.
+       * symmisc.c (free_symtab_block, free_symtab): Use xfree.
+       * symfile.c (reread_symbols, init_psymbol_list): Ditto.
+       * source.c (forget_cached_source_info, find_and_open_source): Ditto.
+       * somread.c (som_symfile_finish): Ditto.
+       * objfiles.c (allocate_objfile, free_objfile): 
+       * nlmread.c (nlm_symfile_finish): Ditto.
+       * hpread.c (hpread_symfile_finish): Ditto.
+       * elfread.c (free_elfinfo, elf_symfile_finish): Ditto.
+       * dbxread.c (dbx_symfile_finish, free_bincl_list): Ditto.
+       * coffread.c (coff_symfile_finish): Ditto.
+
        * defs.h (xmcalloc): Delete declaration.
        * utils.c (xmcalloc): Delete.
        (xcalloc): Inline calls to xmcalloc and mcalloc.

diff --git a/gdb/coffread.c b/gdb/coffread.c
index 1c31d0f54559098cc4b2a03e22ec16149dee31f3..2f6790e2f5f6620849f02d20535e3673cb9ce6f9 100644 (file)
--- a/gdb/coffread.c
+++ b/gdb/coffread.c
@@ -661,7 +661,7 @@ coff_symfile_finish (struct objfile *objfile)
 {
   if (objfile->sym_private != NULL)
     {
-      xmfree (objfile->md, objfile->sym_private);
+      xfree (objfile->sym_private);
     }
 
   /* Let stabs reader clean up */

diff --git a/gdb/dbxread.c b/gdb/dbxread.c
index 28b0d6bb9e3886d7b6740c0deaf293a37ff80e7c..e537744338d3296231721297c7841ea56f6f740f 100644 (file)
--- a/gdb/dbxread.c
+++ b/gdb/dbxread.c
@@ -748,7 +748,7 @@ dbx_symfile_finish (struct objfile *objfile)
            }
          xfree (hfiles);
        }
-      xmfree (objfile->md, objfile->sym_stab_info);
+      xfree (objfile->sym_stab_info);
     }
   free_header_files ();
 }
@@ -937,7 +937,7 @@ find_corresponding_bincl_psymtab (char *name, int instance)
 static void
 free_bincl_list (struct objfile *objfile)
 {
-  xmfree (objfile->md, bincl_list);
+  xfree (bincl_list);
   bincls_allocated = 0;
 }
 

diff --git a/gdb/defs.h b/gdb/defs.h
index c91fd6b7ca4e3eeac2e2269dcafb3aec112927ed..3f6e7207a1678ada32ee2e35747440590ab2505a 100644 (file)
--- a/gdb/defs.h
+++ b/gdb/defs.h
@@ -873,10 +873,6 @@ extern char *msavestring (void *, const char *, size_t);
 
 extern char *mstrsave (void *, const char *);
 
-/* Robust versions of same.  Throw an internal error when no memory,
-   guard against stray NULL arguments. */
-extern void xmfree (void *md, void *ptr);
-
 /* xmalloc(), xrealloc() and xcalloc() have already been declared in
    "libiberty.h". */
 extern void xfree (void *);

diff --git a/gdb/elfread.c b/gdb/elfread.c
index a9f4e177104777e11322d0f00105acb604076c54..3aa2f66eec41ac563d8eb93fe2decf501cd3618b 100644 (file)
--- a/gdb/elfread.c
+++ b/gdb/elfread.c
@@ -608,7 +608,7 @@ free_elfinfo (void *objp)
   while (ssi)
     {
       nssi = ssi->next;
-      xmfree (objfile->md, ssi);
+      xfree (ssi);
       ssi = nssi;
     }
 
@@ -639,7 +639,7 @@ elf_symfile_finish (struct objfile *objfile)
 {
   if (objfile->sym_stab_info != NULL)
     {
-      xmfree (objfile->md, objfile->sym_stab_info);
+      xfree (objfile->sym_stab_info);
     }
 }
 

diff --git a/gdb/hpread.c b/gdb/hpread.c
index 70070d8fe74d5d166d888d8f54a66411489f09f3..de1670aba07c52a894a9abcb73c1b26f9f712621 100644 (file)
--- a/gdb/hpread.c
+++ b/gdb/hpread.c
@@ -2286,7 +2286,7 @@ hpread_symfile_finish (struct objfile *objfile)
 {
   if (objfile->sym_private != NULL)
     {
-      xmfree (objfile->md, objfile->sym_private);
+      xfree (objfile->sym_private);
     }
 }
 \f

diff --git a/gdb/nlmread.c b/gdb/nlmread.c
index 4e9c87d3ff0de7e1b4e4163cefe16c4028a2e284..e75486e241d3f3dfe113a2acba3b5d4f875e6b0b 100644 (file)
--- a/gdb/nlmread.c
+++ b/gdb/nlmread.c
@@ -224,7 +224,7 @@ nlm_symfile_finish (struct objfile *objfile)
 {
   if (objfile->sym_private != NULL)
     {
-      xmfree (objfile->md, objfile->sym_private);
+      xfree (objfile->sym_private);
     }
 }
 

diff --git a/gdb/objfiles.c b/gdb/objfiles.c
index 48d695456df9931e87b2e907e73ead141e0720da..cc6a1e1c71477f75e97443f1a61a9ba1bc3fc93a 100644 (file)
--- a/gdb/objfiles.c
+++ b/gdb/objfiles.c
@@ -180,7 +180,7 @@ allocate_objfile (bfd *abfd, int flags)
   objfile->obfd = abfd;
   if (objfile->name != NULL)
     {
-      xmfree (objfile->md, objfile->name);
+      xfree (objfile->name);
     }
   if (abfd != NULL)
     {
@@ -443,19 +443,19 @@ free_objfile (struct objfile *objfile)
   objfile_free_data (objfile);
   if (objfile->name != NULL)
     {
-      xmfree (objfile->md, objfile->name);
+      xfree (objfile->name);
     }
   if (objfile->global_psymbols.list)
-    xmfree (objfile->md, objfile->global_psymbols.list);
+    xfree (objfile->global_psymbols.list);
   if (objfile->static_psymbols.list)
-    xmfree (objfile->md, objfile->static_psymbols.list);
+    xfree (objfile->static_psymbols.list);
   /* Free the obstacks for non-reusable objfiles */
   bcache_xfree (objfile->psymbol_cache);
   bcache_xfree (objfile->macro_cache);
   if (objfile->demangled_names_hash)
     htab_delete (objfile->demangled_names_hash);
   obstack_free (&objfile->objfile_obstack, 0);
-  xmfree (objfile->md, objfile);
+  xfree (objfile);
   objfile = NULL;
 }
 

diff --git a/gdb/somread.c b/gdb/somread.c
index 13b52092a2010be2fd03a6c9d73cf89ed5b98d84..560eb887c84c56f413357f71d4f90a81fd16299a 100644 (file)
--- a/gdb/somread.c
+++ b/gdb/somread.c
@@ -404,7 +404,7 @@ som_symfile_finish (struct objfile *objfile)
 {
   if (objfile->sym_stab_info != NULL)
     {
-      xmfree (objfile->md, objfile->sym_stab_info);
+      xfree (objfile->sym_stab_info);
     }
   hpread_symfile_finish (objfile);
 }

diff --git a/gdb/source.c b/gdb/source.c
index a55bd33aca945888262fb0d5d66691d8267892b2..b1e9c387473f3b047379dc71b89515cc644fb091 100644 (file)
--- a/gdb/source.c
+++ b/gdb/source.c
@@ -325,12 +325,12 @@ forget_cached_source_info (void)
        {
          if (s->line_charpos != NULL)
            {
-             xmfree (objfile->md, s->line_charpos);
+             xfree (s->line_charpos);
              s->line_charpos = NULL;
            }
          if (s->fullname != NULL)
            {
-             xmfree (objfile->md, s->fullname);
+             xfree (s->fullname);
              s->fullname = NULL;
            }
        }
@@ -851,7 +851,7 @@ find_and_open_source (struct objfile *objfile,
       if (result >= 0)
        return result;
       /* Didn't work -- free old one, try again. */
-      xmfree (objfile->md, *fullname);
+      xfree (*fullname);
       *fullname = NULL;
     }
 

diff --git a/gdb/symfile.c b/gdb/symfile.c
index f203ba0767848350199d798f16b86f6526e144da..d0ec08337f6fd22fd04de7247c72e83dc51ee23d 100644 (file)
--- a/gdb/symfile.c
+++ b/gdb/symfile.c
@@ -1862,11 +1862,11 @@ reread_symbols (void)
              /* FIXME: Do we have to free a whole linked list, or is this
                 enough?  */
              if (objfile->global_psymbols.list)
-               xmfree (objfile->md, objfile->global_psymbols.list);
+               xfree (objfile->global_psymbols.list);
              memset (&objfile->global_psymbols, 0,
                      sizeof (objfile->global_psymbols));
              if (objfile->static_psymbols.list)
-               xmfree (objfile->md, objfile->static_psymbols.list);
+               xfree (objfile->static_psymbols.list);
              memset (&objfile->static_psymbols, 0,
                      sizeof (objfile->static_psymbols));
 
@@ -2690,11 +2690,11 @@ init_psymbol_list (struct objfile *objfile, int total_symbols)
 
   if (objfile->global_psymbols.list)
     {
-      xmfree (objfile->md, objfile->global_psymbols.list);
+      xfree (objfile->global_psymbols.list);
     }
   if (objfile->static_psymbols.list)
     {
-      xmfree (objfile->md, objfile->static_psymbols.list);
+      xfree (objfile->static_psymbols.list);
     }
 
   /* Current best guess is that approximately a twentieth

diff --git a/gdb/symmisc.c b/gdb/symmisc.c
index 1d8cdc7f51c94006fee81c19c9abec772d88efa6..45e2a44ecd7eed3df325f602ebcd9f305931eeb0 100644 (file)
--- a/gdb/symmisc.c
+++ b/gdb/symmisc.c
@@ -99,12 +99,12 @@ free_symtab_block (struct objfile *objfile, struct block *b)
 
   ALL_BLOCK_SYMBOLS (b, iter, sym)
     {
-      xmfree (objfile->md, DEPRECATED_SYMBOL_NAME (sym));
-      xmfree (objfile->md, sym);
+      xfree (DEPRECATED_SYMBOL_NAME (sym));
+      xfree (sym);
     }
 
   dict_free (BLOCK_DICT (b));
-  xmfree (objfile->md, b);
+  xfree (b);
 }
 
 /* Free all the storage associated with the struct symtab <- S.
@@ -138,7 +138,7 @@ free_symtab (struct symtab *s)
       for (i = 0; i < n; i++)
        free_symtab_block (s->objfile, BLOCKVECTOR_BLOCK (bv, i));
       /* Free the blockvector itself.  */
-      xmfree (s->objfile->md, bv);
+      xfree (bv);
       /* Also free the linetable.  */
 
     case free_linetable:
@@ -146,7 +146,7 @@ free_symtab (struct symtab *s)
          or by some other symtab, except for our linetable.
          Free that now.  */
       if (LINETABLE (s))
-       xmfree (s->objfile->md, LINETABLE (s));
+       xfree (LINETABLE (s));
       break;
     }
 
@@ -156,12 +156,12 @@ free_symtab (struct symtab *s)
 
   /* Free source-related stuff */
   if (s->line_charpos != NULL)
-    xmfree (s->objfile->md, s->line_charpos);
+    xfree (s->line_charpos);
   if (s->fullname != NULL)
-    xmfree (s->objfile->md, s->fullname);
+    xfree (s->fullname);
   if (s->debugformat != NULL)
-    xmfree (s->objfile->md, s->debugformat);
-  xmfree (s->objfile->md, s);
+    xfree (s->debugformat);
+  xfree (s);
 }
 
 void

diff --git a/gdb/utils.c b/gdb/utils.c
index e903aa86c71c7c03eb50da21410fc902eb3f6287..25fca5ae604b1f38f208d80de2a844cbe25526a5 100644 (file)
--- a/gdb/utils.c
+++ b/gdb/utils.c
@@ -1037,13 +1037,6 @@ nomem (long size)
     }
 }
 
-void
-xmfree (void *md, void *ptr)
-{
-  if (ptr != NULL)
-    mfree (md, ptr);
-}
-
 /* The xmalloc() (libiberty.h) family of memory management routines.
 
    These are like the ISO-C malloc() family except that they implement
@@ -1113,7 +1106,8 @@ xcalloc (size_t number, size_t size)
 void
 xfree (void *ptr)
 {
-  xmfree (NULL, ptr);
+  if (ptr != NULL)
+    free (ptr);                /* OK: free */
 }
 \f