This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch libiberty include]: Add additional helper functions for directory-separator searching
2011/3/8 Pedro Alves <pedro@codesourcery.com>:
> On Tuesday 08 March 2011 12:48:11, Kai Tietz wrote:
>
>> Well, a better example is elfstab_offset_sections() in elfread.c.
>
> ?/* The ELF symbol info doesn't include path names, so strip the path
> ? ? (if any) from the psymtab filename. ?*/
> ?while (0 != (p = strchr (filename, '/')))
> ? ?filename = p + 1;
>
> Looks like its looking for the last path separator, so
> it might as well use filename_dirrchr instead.
True, see patch I've posted about filename_cmp. I replaced it there by
a strrchr search.
>> Another is in find_file_and_directory() in dwarf2read.c file.
>
> Workaround for Irix. ?Certainly that '/' should not depend
> on the host gdb is running on.
Right. But well, I was asked if strchr is used in combination with
paths. And so I've shown. If those uses could be rewritten is a
different story and might be true.
Kai