This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH 08/18] poison "mutable"
- From: Jan Kratochvil <jan dot kratochvil at redhat dot com>
- To: Ondrej Oprala <ooprala at redhat dot com>
- Cc: gdb-patches at sourceware dot org, Tom Tromey <tromey at redhat dot com>
- Date: Sun, 13 Oct 2013 19:16:08 +0200
- Subject: Re: [PATCH 08/18] poison "mutable"
- Authentication-results: sourceware.org; auth=none
- References: <1381339053-14519-1-git-send-email-ooprala at redhat dot com> <1381339053-14519-9-git-send-email-ooprala at redhat dot com> <20131013170025 dot GD12719 at host2 dot jankratochvil dot net>
On Sun, 13 Oct 2013 19:00:25 +0200, Jan Kratochvil wrote:
> On Wed, 09 Oct 2013 19:17:23 +0200, Ondrej Oprala wrote:
> > --- a/gdb/ChangeLog
> > +++ b/gdb/ChangeLog
> > @@ -17,6 +17,10 @@
> >
> > 2013-10-09 Tom Tromey <tromey@redhat.com>
> >
> > + * addrmap.c (mutable_map) : Rename from mutable.
>
> Use tab + parenthesis content is incorrect, as described elsewhere.
> And s/ : /: /.
>
>
> > +
> > +2013-10-09 Tom Tromey <tromey@redhat.com>
> > +
> > * mi/mi-cmd-var.c (is_explicit) : Rename from explicit.
> > * varobj.c (is_explicit) : Likewise.
> > * varobj.h (is_explicit) : Likewise.
> > diff --git a/gdb/addrmap.c b/gdb/addrmap.c
> > index 348fd43..ccd125c 100644
> > --- a/gdb/addrmap.c
> > +++ b/gdb/addrmap.c
> > @@ -429,16 +429,16 @@ splay_foreach_copy (splay_tree_node n, void *closure)
> > static struct addrmap *
> > addrmap_mutable_create_fixed (struct addrmap *self, struct obstack *obstack)
> > {
> > - struct addrmap_mutable *mutable = (struct addrmap_mutable *) self;
> > + struct addrmap_mutable *mutable_map = (struct addrmap_mutable *) self;
> > struct addrmap_fixed *fixed;
> > size_t num_transitions;
> >
> > /* Count the number of transitions in the tree. */
> > num_transitions = 0;
> > - splay_tree_foreach (mutable->tree, splay_foreach_count, &num_transitions);
> > + splay_tree_foreach (mutable_map->tree, splay_foreach_count, &num_transitions);
>
> Line exceeds 80 columns.
It does not, my mistake.
> >
> > /* Include an extra entry for the transition at zero (which fixed
> > - maps have, but mutable maps do not.) */
> > + maps have, but mutable_map maps do not.) */
> > num_transitions++;
> >
> > fixed = obstack_alloc (obstack,
> > @@ -452,7 +452,7 @@ addrmap_mutable_create_fixed (struct addrmap *self, struct obstack *obstack)
> >
> > /* Copy all entries from the splay tree to the array, in order
> > of increasing address. */
> > - splay_tree_foreach (mutable->tree, splay_foreach_copy, fixed);
> > + splay_tree_foreach (mutable_map->tree, splay_foreach_copy, fixed);
> >
> > /* We should have filled the array. */
> > gdb_assert (fixed->num_transitions == num_transitions);
> > @@ -496,12 +496,12 @@ static int
> > addrmap_mutable_foreach (struct addrmap *self, addrmap_foreach_fn fn,
> > void *data)
> > {
> > - struct addrmap_mutable *mutable = (struct addrmap_mutable *) self;
> > + struct addrmap_mutable *mutable_map = (struct addrmap_mutable *) self;
> > struct mutable_foreach_data foreach_data;
> >
> > foreach_data.fn = fn;
> > foreach_data.data = data;
> > - return splay_tree_foreach (mutable->tree, addrmap_mutable_foreach_worker,
> > + return splay_tree_foreach (mutable_map->tree, addrmap_mutable_foreach_worker,
>
> Line exceeds 80 columns.
It does not, my mistake.
> > &foreach_data);
> > }
> >
> > --
> > 1.8.3.1
Therefore the patch itself is OK.
>
>
> Thanks,
> Jan