gdb/

	Code cleanup.
	* objfiles.c (allocate_objfile) <objfile->name != NULL>: Remove.
	(free_objfile) <objfile->name != NULL>: Remove the conditional around
	xfree.
	* objfiles.h (struct objfile) <name>: New comment it is never NULL.
	* python/py-auto-load.c (auto_load_new_objfile) <!objfile->name>:
	Remove.
	* python/py-objfile.c (objfpy_get_filename) <obj->objfile->name>
	Remove the conditional.
	* python/py-progspace.c (pspy_get_filename) <objfile->name>: Likewise.

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 8505e8b4432f69c29293b9cf773590de7f9e4184..9f959c176da6c1da1e7347217a48d7d00a456f7e 100644 (file)
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,16 @@
+2010-09-22  Jan Kratochvil  <jan.kratochvil@redhat.com>
+
+       Code cleanup.
+       * objfiles.c (allocate_objfile) <objfile->name != NULL>: Remove.
+       (free_objfile) <objfile->name != NULL>: Remove the conditional around
+       xfree.
+       * objfiles.h (struct objfile) <name>: New comment it is never NULL.
+       * python/py-auto-load.c (auto_load_new_objfile) <!objfile->name>:
+       Remove.
+       * python/py-objfile.c (objfpy_get_filename) <obj->objfile->name>
+       Remove the conditional.
+       * python/py-progspace.c (pspy_get_filename) <objfile->name>: Likewise.
+
 2010-09-22  Jan Kratochvil  <jan.kratochvil@redhat.com>
 
        * main.c: Include objfiles.h.

diff --git a/gdb/objfiles.c b/gdb/objfiles.c
index 7d5186f17882c1fb4efe1ab5b07c194f987c0f39..076827619c8e3599e7e364fd2340c5b86194da20 100644 (file)
--- a/gdb/objfiles.c
+++ b/gdb/objfiles.c
@@ -214,10 +214,6 @@ allocate_objfile (bfd *abfd, int flags)
      region. */
 
   objfile->obfd = gdb_bfd_ref (abfd);
-  if (objfile->name != NULL)
-    {
-      xfree (objfile->name);
-    }
   if (abfd != NULL)
     {
       /* Look up the gdbarch associated with the BFD.  */
@@ -649,10 +645,7 @@ free_objfile (struct objfile *objfile)
 
   /* The last thing we do is free the objfile struct itself. */
 
-  if (objfile->name != NULL)
-    {
-      xfree (objfile->name);
-    }
+  xfree (objfile->name);
   if (objfile->global_psymbols.list)
     xfree (objfile->global_psymbols.list);
   if (objfile->static_psymbols.list)

diff --git a/gdb/objfiles.h b/gdb/objfiles.h
index ec4870b27a72a2891b9834afa119062e04277530..b3e2e26faac695036864eb794ec74ce1a0467aa6 100644 (file)
--- a/gdb/objfiles.h
+++ b/gdb/objfiles.h
@@ -188,8 +188,8 @@ struct objfile
 
     struct objfile *next;
 
-    /* The object file's name, tilde-expanded and absolute.
-       Malloc'd; free it if you free this struct.  */
+    /* The object file's name, tilde-expanded and absolute.  Malloc'd; free it
+       if you free this struct.  This pointer is never NULL.  */
 
     char *name;
 

diff --git a/gdb/python/py-auto-load.c b/gdb/python/py-auto-load.c
index 044d968b57584c7e936951438c7f5f904ea9bf58..538e06d7bcdd2837726ff127b397cf52727c7d10 100644 (file)
--- a/gdb/python/py-auto-load.c
+++ b/gdb/python/py-auto-load.c
@@ -395,8 +395,6 @@ auto_load_new_objfile (struct objfile *objfile)
       clear_section_scripts ();
       return;
     }
-  if (!objfile->name)
-    return;
 
   load_auto_scripts_for_objfile (objfile);
 }

diff --git a/gdb/python/py-objfile.c b/gdb/python/py-objfile.c
index 1aa6a4dabca04331feb132cf4c9ed70423e075f3..147350388643ac0f499c915b3d01388c0f181913 100644 (file)
--- a/gdb/python/py-objfile.c
+++ b/gdb/python/py-objfile.c
@@ -46,7 +46,7 @@ objfpy_get_filename (PyObject *self, void *closure)
 {
   objfile_object *obj = (objfile_object *) self;
 
-  if (obj->objfile && obj->objfile->name)
+  if (obj->objfile)
     return PyString_Decode (obj->objfile->name, strlen (obj->objfile->name),
                            host_charset (), NULL);
   Py_RETURN_NONE;

diff --git a/gdb/python/py-progspace.c b/gdb/python/py-progspace.c
index 87245328c83e4cbca45d161dad0f48fe6b7562f3..2c8a677b7090c4cc966232d76f072048dad996f0 100644 (file)
--- a/gdb/python/py-progspace.c
+++ b/gdb/python/py-progspace.c
@@ -52,7 +52,7 @@ pspy_get_filename (PyObject *self, void *closure)
     {
       struct objfile *objfile = obj->pspace->symfile_object_file;
 
-      if (objfile && objfile->name)
+      if (objfile)
        return PyString_Decode (objfile->name, strlen (objfile->name),
                                host_charset (), NULL);
     }