Re: small request regarding commits in binutils-gdb.git

[Gary Benson <gbenson at redhat dot com>]

Eli Zaretskii wrote:
> > Date: Wed, 15 Jan 2014 16:12:51 +0400
> > From: Joel Brobecker <brobecker@adacore.com>
> > 
> > It would look like this:
> > 
> > > psymtab cleanup patch 3/3
> > >
> > > This last patch removes "partial" from the names of
> > > expand_partial_symbol_names and map_partial_symbol_filenames.
> > > It also renames expand_partial_symbol_names to match the
> > > struct quick_symbol_functions "method" that it wraps:
> > > expand_symtabs_matching.
> > >
> > > This patch also adds two parameters to expand_symtabs_matching
> > > so that it can fully wrap the underlying quick_symbol_functions method.
> > > This makes it usable in more places.
> > > I thought of having a cover function that still had the same
> > > signature as the old expand_partial_symbol_names function,
> > > but I couldn't think of a good name, and it wasn't clear it was
> > > worth it anyway.
> > >
> > > gdb/ChangeLog:
> > >
> > >     * symfile.h (expand_symtabs_matching): Renamed from
> > >     expand_partial_symbol_names.  Update prototype.
> > >     (map_symbol_filenames): Renamed from map_partial_symbol_filenames.
> > >     * symfile.c (expand_symtabs_matching): Renamed from
> 
> The text before the log entry is just a long way of saying the same
> thing as the log entry, isn't it?  It repeats the info that is
> already in the log entry.  Why does it make sense to repeat all
> that?

The ChangeLog says what.  The text before it says why.

Cheers,
Gary

-- 
http://gbenson.net/