From: Ian Lance Taylor <ian@zembu.com>
To: geoffk@ozemail.com.au
Cc: binutils@sourceware.cygnus.com, law@cygnus.com
Subject: Re: elf32.em
Date: Wed, 27 Oct 1999 16:48:00 -0000 [thread overview]
Message-ID: <19991027234806.10154.qmail@daffy.airs.com> (raw)
In-Reply-To: <199910270848.SAA01584@gluttony.geoffk.wattle.id.au>
Date: Wed, 27 Oct 1999 18:48:30 +1000
From: Geoff Keating <geoffk@ozemail.com.au>
In elf32.em, there's this code:
FIXME: This approach--using bfd_elf_set_dt_needed_name--is not
very pretty. I haven't been able to think of anything that is
pretty, though. */
if (bfd_check_format (entry->the_bfd, bfd_object)
&& (entry->the_bfd->flags & DYNAMIC) != 0)
{
char *filname, *needed_name;
ASSERT (entry->is_archive && entry->search_dirs_flag);
/* Rather than duplicating the logic above. Just use the
filename we recorded earlier.o
First strip off everything before the last '/'. */
filename = strrchr (entry->filename, '/');
Shouldn't the variable be called 'filename'?
Actually, I think the uses of it should be filname, to avoid
shadowing. Or we should just remove the declaration of filname, and
use the existing filename local.
Jeff, I'm not sure it's a good idea to try for the .sl extension on
every system type, given that it is only meaningful on HP/UX. If
there is a long search path, it is a waste of time. It is also
incorrect in principle. You can test for a particular emulation using
#ifdef TARGET_IS_xxx.
Ian
next prev parent reply other threads:[~1999-10-27 16:48 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
1999-10-27 16:33 elf32.em Geoff Keating
1999-10-27 16:48 ` Ian Lance Taylor [this message]
1999-10-27 23:27 ` elf32.em Jeffrey A Law
1999-11-03 7:49 ` elf32.em Jeffrey A Law
1999-11-03 7:49 ` elf32.em Jeffrey A Law
1999-11-03 8:24 ` elf32.em Jeffrey A Law
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=19991027234806.10154.qmail@daffy.airs.com \
--to=ian@zembu.com \
--cc=binutils@sourceware.cygnus.com \
--cc=geoffk@ozemail.com.au \
--cc=law@cygnus.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).