From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 10887 invoked by alias); 6 Jan 2014 17:32:16 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 10779 invoked by uid 89); 6 Jan 2014 17:32:14 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-3.2 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mx1.redhat.com Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 06 Jan 2014 17:32:10 +0000 Received: from int-mx02.intmail.prod.int.phx2.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s06HW9Ks011501 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Mon, 6 Jan 2014 12:32:09 -0500 Received: from barimba.redhat.com (ovpn-113-85.phx2.redhat.com [10.3.113.85]) by int-mx02.intmail.prod.int.phx2.redhat.com (8.13.8/8.13.8) with ESMTP id s06HW6lw000636; Mon, 6 Jan 2014 12:32:08 -0500 From: Tom Tromey To: gdb-patches@sourceware.org Cc: Tom Tromey Subject: [PATCH 3/3] replace XCALLOC with XCNEWVEC or XCNEW Date: Mon, 06 Jan 2014 17:32:00 -0000 Message-Id: <1389029524-19907-4-git-send-email-tromey@redhat.com> In-Reply-To: <1389029524-19907-1-git-send-email-tromey@redhat.com> References: <1389029524-19907-1-git-send-email-tromey@redhat.com> X-SW-Source: 2014-01/txt/msg00073.txt.bz2 This removes XCALLOC and replaces it either with XCNEWVEC, or, if the number of elements being requested was 1, with XCNEW. 2014-01-06 Tom Tromey * defs.h (XCALLOC): Remove. * bcache.c (bcache_xmalloc): Use XCNEW, not XCALLOC. (print_bcache_statistics): Use XCNEWVEC, not XCALLOC. * dwarf2loc.c (allocate_piece_closure): Likewise. * elfread.c (elf_symfile_segments): Likewise. (elf_symfile_segments): Likewise. * gdbtypes.c (copy_type_recursive): Likewise. * i386-tdep.c (i386_gdbarch_init): Use XCNEW, not XCALLOC. * jit.c (jit_frame_sniffer): Use XCNEWVEC, not XCALLOC. * minsyms.c (prim_record_minimal_symbol_full): Use XCNEW, not XCALLOC. * mt-tdep.c (mt_gdbarch_init): Likewise. * opencl-lang.c (allocate_lval_closure): Use XCNEWVEC, not XCALLOC. * psymtab.c (psymbol_compare): Use XCNEW, not XCALLOC. * regcache.c (regcache_xmalloc_1): Use XCNEWVEC, not XCALLOC. * registry.c (registry_alloc_data): Likewise. * rs6000-tdep.c (rs6000_gdbarch_init): Use XCNEW, not XCALLOC. * s390-linux-tdep.c (s390_gdbarch_init): Likewise. * serial.c (serial_fdopen_ops): Likewise. * solib-aix.c (solib_aix_get_section_offsets): Use XCNEWVEC, not XCALLOC. * spu-tdep.c (spu_gdbarch_init): Use XCNEW, not XCALLOC. * symfile.c (default_symfile_segments): Use XCNEW and XCNEWVEC, not XCALLOC. --- gdb/ChangeLog | 28 ++++++++++++++++++++++++++++ gdb/bcache.c | 6 +++--- gdb/defs.h | 5 ----- gdb/dwarf2loc.c | 2 +- gdb/elfread.c | 6 +++--- gdb/gdbtypes.c | 2 +- gdb/i386-tdep.c | 2 +- gdb/jit.c | 4 ++-- gdb/minsyms.c | 2 +- gdb/mt-tdep.c | 2 +- gdb/opencl-lang.c | 2 +- gdb/psymtab.c | 2 +- gdb/regcache.c | 8 ++++---- gdb/registry.c | 2 +- gdb/rs6000-tdep.c | 2 +- gdb/s390-linux-tdep.c | 2 +- gdb/serial.c | 2 +- gdb/solib-aix.c | 2 +- gdb/spu-tdep.c | 2 +- gdb/symfile.c | 6 +++--- 20 files changed, 56 insertions(+), 33 deletions(-) diff --git a/gdb/bcache.c b/gdb/bcache.c index 3d1280a..99e9184 100644 --- a/gdb/bcache.c +++ b/gdb/bcache.c @@ -313,7 +313,7 @@ bcache_xmalloc (unsigned long (*hash_function)(const void *, int length), int length)) { /* Allocate the bcache pre-zeroed. */ - struct bcache *b = XCALLOC (1, struct bcache); + struct bcache *b = XCNEW (struct bcache); if (hash_function) b->hash_function = hash_function; @@ -372,8 +372,8 @@ print_bcache_statistics (struct bcache *c, char *type) lengths, and measure chain lengths. */ { unsigned int b; - int *chain_length = XCALLOC (c->num_buckets + 1, int); - int *entry_size = XCALLOC (c->unique_count + 1, int); + int *chain_length = XCNEWVEC (int, c->num_buckets + 1); + int *entry_size = XCNEWVEC (int, c->unique_count + 1); int stringi = 0; occupied_buckets = 0; diff --git a/gdb/defs.h b/gdb/defs.h index a4770ef..a7ce88a 100644 --- a/gdb/defs.h +++ b/gdb/defs.h @@ -553,11 +553,6 @@ enum val_prettyformat extern int longest_to_int (LONGEST); -/* Utility macros to allocate typed memory. Avoids errors like: - struct foo *foo = xmalloc (sizeof struct bar); and memset (foo, - sizeof (struct foo), 0). */ -#define XCALLOC(NMEMB, TYPE) ((TYPE*) xcalloc ((NMEMB), sizeof (TYPE))) - #include "common-utils.h" /* List of known OS ABIs. If you change this, make sure to update the diff --git a/gdb/dwarf2loc.c b/gdb/dwarf2loc.c index 6185a6b..99a18a9 100644 --- a/gdb/dwarf2loc.c +++ b/gdb/dwarf2loc.c @@ -1421,7 +1421,7 @@ allocate_piece_closure (struct dwarf2_per_cu_data *per_cu, c->per_cu = per_cu; c->n_pieces = n_pieces; c->addr_size = addr_size; - c->pieces = XCALLOC (n_pieces, struct dwarf_expr_piece); + c->pieces = XCNEWVEC (struct dwarf_expr_piece, n_pieces); memcpy (c->pieces, pieces, n_pieces * sizeof (struct dwarf_expr_piece)); for (i = 0; i < n_pieces; ++i) diff --git a/gdb/elfread.c b/gdb/elfread.c index 4f27aff..d43a1f7 100644 --- a/gdb/elfread.c +++ b/gdb/elfread.c @@ -107,8 +107,8 @@ elf_symfile_segments (bfd *abfd) data = XCNEW (struct symfile_segment_data); data->num_segments = num_segments; - data->segment_bases = XCALLOC (num_segments, CORE_ADDR); - data->segment_sizes = XCALLOC (num_segments, CORE_ADDR); + data->segment_bases = XCNEWVEC (CORE_ADDR, num_segments); + data->segment_sizes = XCNEWVEC (CORE_ADDR, num_segments); for (i = 0; i < num_segments; i++) { @@ -117,7 +117,7 @@ elf_symfile_segments (bfd *abfd) } num_sections = bfd_count_sections (abfd); - data->segment_info = XCALLOC (num_sections, int); + data->segment_info = XCNEWVEC (int, num_sections); for (i = 0, sect = abfd->sections; sect != NULL; i++, sect = sect->next) { diff --git a/gdb/gdbtypes.c b/gdb/gdbtypes.c index e02b66c..042c17d 100644 --- a/gdb/gdbtypes.c +++ b/gdb/gdbtypes.c @@ -3700,7 +3700,7 @@ copy_type_recursive (struct objfile *objfile, int i, nfields; nfields = TYPE_NFIELDS (type); - TYPE_FIELDS (new_type) = XCALLOC (nfields, struct field); + TYPE_FIELDS (new_type) = XCNEWVEC (struct field, nfields); for (i = 0; i < nfields; i++) { TYPE_FIELD_ARTIFICIAL (new_type, i) = diff --git a/gdb/i386-tdep.c b/gdb/i386-tdep.c index ce44ffd..588683fd 100644 --- a/gdb/i386-tdep.c +++ b/gdb/i386-tdep.c @@ -7792,7 +7792,7 @@ i386_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) return arches->gdbarch; /* Allocate space for the new architecture. */ - tdep = XCALLOC (1, struct gdbarch_tdep); + tdep = XCNEW (struct gdbarch_tdep); gdbarch = gdbarch_alloc (&info, tdep); /* General-purpose registers. */ diff --git a/gdb/jit.c b/gdb/jit.c index 59823a5..a73ed64 100644 --- a/gdb/jit.c +++ b/gdb/jit.c @@ -1184,8 +1184,8 @@ jit_frame_sniffer (const struct frame_unwind *self, *cache = XCNEW (struct jit_unwind_private); priv_data = *cache; priv_data->registers = - XCALLOC (gdbarch_num_regs (get_frame_arch (this_frame)), - struct gdb_reg_value *); + XCNEWVEC (struct gdb_reg_value *, + gdbarch_num_regs (get_frame_arch (this_frame))); priv_data->this_frame = this_frame; callbacks.priv_data = priv_data; diff --git a/gdb/minsyms.c b/gdb/minsyms.c index ee9575c..c7ae981 100644 --- a/gdb/minsyms.c +++ b/gdb/minsyms.c @@ -935,7 +935,7 @@ prim_record_minimal_symbol_full (const char *name, int name_len, int copy_name, if (msym_bunch_index == BUNCH_SIZE) { - new = XCALLOC (1, struct msym_bunch); + new = XCNEW (struct msym_bunch); msym_bunch_index = 0; new->next = msym_bunch; msym_bunch = new; diff --git a/gdb/mt-tdep.c b/gdb/mt-tdep.c index 2072528..e3786e8 100644 --- a/gdb/mt-tdep.c +++ b/gdb/mt-tdep.c @@ -1149,7 +1149,7 @@ mt_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) /* None found, create a new architecture from the information provided. */ - tdep = XCALLOC (1, struct gdbarch_tdep); + tdep = XCNEW (struct gdbarch_tdep); gdbarch = gdbarch_alloc (&info, tdep); set_gdbarch_float_format (gdbarch, floatformats_ieee_single); diff --git a/gdb/opencl-lang.c b/gdb/opencl-lang.c index 4327fe6..be6e08d 100644 --- a/gdb/opencl-lang.c +++ b/gdb/opencl-lang.c @@ -161,7 +161,7 @@ allocate_lval_closure (int *indices, int n, struct value *val) c->refc = 1; c->n = n; - c->indices = XCALLOC (n, int); + c->indices = XCNEWVEC (int, n); memcpy (c->indices, indices, n * sizeof (int)); value_incref (val); /* Increment the reference counter of the value. */ c->val = val; diff --git a/gdb/psymtab.c b/gdb/psymtab.c index 34e833c..56fe409 100644 --- a/gdb/psymtab.c +++ b/gdb/psymtab.c @@ -1530,7 +1530,7 @@ psymbol_compare (const void *addr1, const void *addr2, int length) struct psymbol_bcache * psymbol_bcache_init (void) { - struct psymbol_bcache *bcache = XCALLOC (1, struct psymbol_bcache); + struct psymbol_bcache *bcache = XCNEW (struct psymbol_bcache); bcache->bcache = bcache_xmalloc (psymbol_hash, psymbol_compare); return bcache; } diff --git a/gdb/regcache.c b/gdb/regcache.c index ea52d45..2e9314c 100644 --- a/gdb/regcache.c +++ b/gdb/regcache.c @@ -224,16 +224,16 @@ regcache_xmalloc_1 (struct gdbarch *gdbarch, struct address_space *aspace, if (readonly_p) { regcache->registers - = XCALLOC (descr->sizeof_cooked_registers, gdb_byte); + = XCNEWVEC (gdb_byte, descr->sizeof_cooked_registers); regcache->register_status - = XCALLOC (descr->sizeof_cooked_register_status, signed char); + = XCNEWVEC (signed char, descr->sizeof_cooked_register_status); } else { regcache->registers - = XCALLOC (descr->sizeof_raw_registers, gdb_byte); + = XCNEWVEC (gdb_byte, descr->sizeof_raw_registers); regcache->register_status - = XCALLOC (descr->sizeof_raw_register_status, signed char); + = XCNEWVEC (signed char, descr->sizeof_raw_register_status); } regcache->aspace = aspace; regcache->ptid = minus_one_ptid; diff --git a/gdb/registry.c b/gdb/registry.c index ed614b9..2fd6033 100644 --- a/gdb/registry.c +++ b/gdb/registry.c @@ -51,7 +51,7 @@ registry_alloc_data (struct registry_data_registry *registry, { gdb_assert (fields->data == NULL); fields->num_data = registry->num_registrations; - fields->data = XCALLOC (fields->num_data, void *); + fields->data = XCNEWVEC (void *, fields->num_data); } void diff --git a/gdb/rs6000-tdep.c b/gdb/rs6000-tdep.c index 0008f84..d7416e5 100644 --- a/gdb/rs6000-tdep.c +++ b/gdb/rs6000-tdep.c @@ -3954,7 +3954,7 @@ rs6000_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) - "set arch" trust blindly - GDB startup useless but harmless */ - tdep = XCALLOC (1, struct gdbarch_tdep); + tdep = XCNEW (struct gdbarch_tdep); tdep->wordsize = wordsize; tdep->soft_float = soft_float; tdep->vector_abi = vector_abi; diff --git a/gdb/s390-linux-tdep.c b/gdb/s390-linux-tdep.c index 69befcf..b75c86a 100644 --- a/gdb/s390-linux-tdep.c +++ b/gdb/s390-linux-tdep.c @@ -3196,7 +3196,7 @@ s390_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) } /* Otherwise create a new gdbarch for the specified machine type. */ - tdep = XCALLOC (1, struct gdbarch_tdep); + tdep = XCNEW (struct gdbarch_tdep); tdep->abi = tdep_abi; gdbarch = gdbarch_alloc (&info, tdep); diff --git a/gdb/serial.c b/gdb/serial.c index c87875f..2b1eced 100644 --- a/gdb/serial.c +++ b/gdb/serial.c @@ -263,7 +263,7 @@ serial_fdopen_ops (const int fd, const struct serial_ops *ops) if (!ops) return NULL; - scb = XCALLOC (1, struct serial); + scb = XCNEW (struct serial); scb->ops = ops; diff --git a/gdb/solib-aix.c b/gdb/solib-aix.c index 8f335f6..78d2753 100644 --- a/gdb/solib-aix.c +++ b/gdb/solib-aix.c @@ -470,7 +470,7 @@ solib_aix_get_section_offsets (struct objfile *objfile, bfd *abfd = objfile->obfd; int i; - offsets = XCALLOC (objfile->num_sections, struct section_offsets); + offsets = XCNEWVEC (struct section_offsets, objfile->num_sections); /* .text */ diff --git a/gdb/spu-tdep.c b/gdb/spu-tdep.c index 4d745d9..f096a2e 100644 --- a/gdb/spu-tdep.c +++ b/gdb/spu-tdep.c @@ -2658,7 +2658,7 @@ spu_gdbarch_init (struct gdbarch_info info, struct gdbarch_list *arches) } /* None found, so create a new architecture. */ - tdep = XCALLOC (1, struct gdbarch_tdep); + tdep = XCNEW (struct gdbarch_tdep); tdep->id = id; gdbarch = gdbarch_alloc (&info, tdep); diff --git a/gdb/symfile.c b/gdb/symfile.c index 75e71cb..dcdfd51 100644 --- a/gdb/symfile.c +++ b/gdb/symfile.c @@ -802,11 +802,11 @@ default_symfile_segments (bfd *abfd) data = XCNEW (struct symfile_segment_data); data->num_segments = 1; - data->segment_bases = XCALLOC (1, CORE_ADDR); - data->segment_sizes = XCALLOC (1, CORE_ADDR); + data->segment_bases = XCNEW (CORE_ADDR); + data->segment_sizes = XCNEW (CORE_ADDR); num_sections = bfd_count_sections (abfd); - data->segment_info = XCALLOC (num_sections, int); + data->segment_info = XCNEWVEC (int, num_sections); for (i = 0, sect = abfd->sections; sect != NULL; i++, sect = sect->next) { -- 1.8.1.4