diff options
| author | Yao Qi <yao@codesourcery.com> | 2013-10-24 20:42:02 +0800 |
|---|---|---|
| committer | Yao Qi <yao@codesourcery.com> | 2013-10-29 14:36:29 +0800 |
| commit | 487ad57ccfe1434e21774117d20dfd9e23f12afa (patch) | |
| tree | da22685f2b7ee912998a8a96cd5d7eff900872d5 /gdb/objfiles.c | |
| parent | 3c4797ba74e0b7db0db3e34b606ba9c1d0f2baf7 (diff) | |
| download | gdb-487ad57ccfe1434e21774117d20dfd9e23f12afa.zip gdb-487ad57ccfe1434e21774117d20dfd9e23f12afa.tar.gz gdb-487ad57ccfe1434e21774117d20dfd9e23f12afa.tar.bz2 | |
Simplify REGISTRY cleanup usages
In registry.c:registry_clear_data, the registered data is iterated and
invoke each 'free' function with the data passed:
for (registration = data_registry->registrations, i = 0;
i < fields->num_data;
registration = registration->next, i++)
if (fields->data[i] != NULL && registration->data->free != NULL)
adaptor (registration->data->free, container, fields->data[i]);
we can see that data is passed to function 'free' and data is not NULL.
In each usage, we don't have to get the data again through key and
do NULL pointer checking. This patch is to simplify them.
gdb:
2013-10-29 Yao Qi <yao@codesourcery.com>
* auto-load.c (auto_load_pspace_data_cleanup): Get data from
parameter 'arg' instead of from program_space_data.
* objfiles.c (objfiles_pspace_data_cleanup): Likewise.
* solib-darwin.c (darwin_pspace_data_cleanup): Likewise.
* solib-dsbt.c (dsbt_pspace_data_cleanup): Likewise.
* solib-svr4.c (svr4_pspace_data_cleanup): Likewise.
* inflow.c (inflow_inferior_data_cleanup): Get data from
parameter 'arg' instead of inferior_data.
* registry.h: Add comments.
Diffstat (limited to 'gdb/objfiles.c')
| -rw-r--r-- | gdb/objfiles.c | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/gdb/objfiles.c b/gdb/objfiles.c index d1f3121..7029196 100644 --- a/gdb/objfiles.c +++ b/gdb/objfiles.c @@ -85,14 +85,10 @@ static const struct program_space_data *objfiles_pspace_data; static void objfiles_pspace_data_cleanup (struct program_space *pspace, void *arg) { - struct objfile_pspace_info *info; + struct objfile_pspace_info *info = arg; - info = program_space_data (pspace, objfiles_pspace_data); - if (info != NULL) - { - xfree (info->sections); - xfree (info); - } + xfree (info->sections); + xfree (info); } /* Get the current svr4 data. If none is found yet, add it now. This |