This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch][rfc] Allow GDB to search for the right libthread_db.so.1


On Friday 01 May 2009 01:19:43, Paul Pluzhnikov wrote:
> Sorry, missed "space before parenth":
> 
> < +  return libpthread_name_p(so->so_name);
> > +  return libpthread_name_p (so->so_name);
> 
> Fix attached.

On Friday 01 May 2009 01:19:43, Paul Pluzhnikov wrote:

> handle = dlopen (library, RTLD_NOW);

I wonder if making this RTLD_LAZY until you found the correct one
wouldn't make sense?

> +static int
> +thread_db_load_search ()

                         ^ (void)

  (this function could be made to use `openat' at some point, but
   gdb is already assumes PATH_MAX is largest path possible elsewhere
   anyway)

I also wonder if `set sysroot' should affect this search path: I think
not, but I'm not 100% sure.

> +int libpthread_name_p (const char *name)
> +{

      ^ function name at column 0, please.


Other that those nits, this version looks good to me.  Everyone else
has already picked on what else could have been picked on.  :-)
AFAICS, this version addressed all of Daniel's concerns.  Daniel,
what do you think?

-- 
Pedro Alves


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]