diff options
| author | Pierre Muller <muller@ics.u-strasbg.fr> | 2011-04-19 07:16:39 +0000 |
|---|---|---|
| committer | Pierre Muller <muller@ics.u-strasbg.fr> | 2011-04-19 07:16:39 +0000 |
| commit | 9381495589a5870b108fb9dce73d77385dc4f9aa (patch) | |
| tree | e934f39f69730bbf47efaf164f62f05f4fb7dc86 /gdb | |
| parent | 1dab1aa9e2d17bca12f7f7cb46076326dab2e576 (diff) | |
| download | gdb-9381495589a5870b108fb9dce73d77385dc4f9aa.tar.gz | |
ARI cleanup.
* xtensa-tdep.c (xtensa_register_type): Use xstrprintf instead of
sprintf. Simplify code and avoid loosing memory.
(xtensa_register_reggroup_p): Extract assignment out of IF clause.
(call0_frame_cache): Remove && operator from end of line.
Diffstat (limited to 'gdb')
| -rw-r--r-- | gdb/ChangeLog | 8 | ||||
| -rw-r--r-- | gdb/xtensa-tdep.c | 16 |
2 files changed, 16 insertions, 8 deletions
diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 2be27b53c42..20006276f91 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,11 @@ +2011-04-19 Pierre Muller <muller@ics.u-strasbg.fr> + + ARI cleanup. + * xtensa-tdep.c (xtensa_register_type): Use xstrprintf instead of + sprintf. Simplify code and avoid loosing memory. + (xtensa_register_reggroup_p): Extract assignment out of IF clause. + (call0_frame_cache): Remove && operator from end of line. + 2011-04-17 Jan Kratochvil <jan.kratochvil@redhat.com> Fix libraries displacement if they change whether they were prelinked. diff --git a/gdb/xtensa-tdep.c b/gdb/xtensa-tdep.c index 24cc79c42bd..518b77b85de 100644 --- a/gdb/xtensa-tdep.c +++ b/gdb/xtensa-tdep.c @@ -318,15 +318,14 @@ xtensa_register_type (struct gdbarch *gdbarch, int regnum) if (tp == NULL) { - char *name = xmalloc (16); + char *name = xstrprintf ("int%d", size * 8); tp = xmalloc (sizeof (struct ctype_cache)); tp->next = tdep->type_entries; tdep->type_entries = tp; tp->size = size; - - sprintf (name, "int%d", size * 8); tp->virtual_type - = arch_integer_type (gdbarch, size * 8, 1, xstrdup (name)); + = arch_integer_type (gdbarch, size * 8, 1, name); + xfree (name); } reg->ctype = tp->virtual_type; @@ -843,7 +842,8 @@ xtensa_register_reggroup_p (struct gdbarch *gdbarch, if (group == restore_reggroup) return (regnum < gdbarch_num_regs (gdbarch) && (reg->flags & SAVE_REST_FLAGS) == SAVE_REST_VALID); - if ((cp_number = xtensa_coprocessor_register_group (group)) >= 0) + cp_number = xtensa_coprocessor_register_group (group); + if (cp_number >= 0) return rg & (xtRegisterGroupCP0 << cp_number); else return 1; @@ -2715,9 +2715,9 @@ call0_frame_cache (struct frame_info *this_frame, too bad. */ int i; - for (i = 0; - (i < C0_NREGS) && - (i == C0_RA || cache->c0.c0_rt[i].fr_reg != C0_RA); + for (i = 0; + (i < C0_NREGS) + && (i == C0_RA || cache->c0.c0_rt[i].fr_reg != C0_RA); ++i); if (i >= C0_NREGS && cache->c0.c0_rt[C0_RA].fr_reg == C0_RA) i = C0_RA; |