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: reimplement AIX shared-lib support (v2)


>>>>> "Joel" == Joel Brobecker <brobecker@adacore.com> writes:

Joel> Any comments? Suggestions?

I noticed something minor.

Joel> +  /* The pathname is actually a synthetic filename with the following
Joel> +     form: "/path/to/sharedlib(member.o)" (double-quotes excluded).
Joel> +     split this into archive name and member name.
Joel> +
Joel> +     FIXME: This is a little hacky.  Perhaps we should provide access
Joel> +     to the solib's lm_info here?  */
Joel> +  const int path_len = strlen(pathname);

Space before open paren.

Joel> +  /* Search for the associated parens.  */
Joel> +  len = path_len - 2;
Joel> +  while (len > 1 && pathname[len] != '(')
Joel> +    len--;

I think this can use strrchr.

Tom


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