From: Andrew Cagney <ac131313@cygnus.com>
To: gdb-patches@sources.redhat.com
Subject: [rfc/5.1] Document HP/UX build/config problems
Date: Mon, 15 Oct 2001 19:54:00 -0000 [thread overview]
Message-ID: <3BCBA134.8070407@cygnus.com> (raw)
Hello,
The attached updates the README file so that it mentions the HP/UX problems.
Comments?
Andrew
From ezannoni@cygnus.com Mon Oct 15 20:10:00 2001
From: Elena Zannoni <ezannoni@cygnus.com>
To: jason-swarelist@molenda.com
Cc: gdb-patches@sources.redhat.com
Subject: Re: [RFA] bug in symtab.c:lookup_block_symbol()'s search method
Date: Mon, 15 Oct 2001 20:10:00 -0000
Message-id: <15307.42699.960252.786487@krustylu.cygnus.com>
References: <3BCA0ED9.E26EDF21@ne.mediaone.net>
X-SW-Source: 2001-10/msg00230.html
Content-length: 1657
Jason Molenda writes:
>
>
> This patch fixes a bug introduced in October, 2000. Discussion and
> history
> are here:
> http://sources.redhat.com/ml/gdb-patches/2001-09/msg00076.html
>
> Analysis of performance impact is here:
> http://sources.redhat.com/ml/gdb-patches/2001-09/msg00084.html
>
> This patch should be approved for both the mainline and the 5.1 branch.
> It adds no new testsuite failures.
>
After pondering on this for a while and discussing it with others in
the gdb group, I am inclined to accept this patch into the main line
but not 5.1.
The possibility that there is an identifier that start with a
isspace() char is quite remote. And as Jason points out the rest
of the code in the function wouldn't be able to deal with it either.
Jason, if you can check this in, please go ahead.
Looking at this function I also noticed an outdated comment. Will fix
in a follow up patch.
Elena
> Jason
>
> 2001-09-07 Jason Molenda (jmolenda@apple.com)
>
> * symtab.c (lookup_block_symbol): Break out of linear search
> if we're past the range of possible matches.
>
> Index: symtab.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/symtab.c,v
> retrieving revision 1.42
> diff -u -p -r1.42 symtab.c
> --- symtab.c 2001/07/07 17:19:50 1.42
> +++ symtab.c 2001/09/09 14:17:25
> @@ -1249,6 +1249,10 @@ lookup_block_symbol (register const stru
> {
> return sym;
> }
> + if (SYMBOL_SOURCE_NAME (sym)[0] > name[0])
> + {
> + break;
> + }
> bot++;
> }
> }
next reply other threads:[~2001-10-15 19:54 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-10-15 19:54 Andrew Cagney [this message]
2001-10-17 21:31 ` Andrew Cagney
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=3BCBA134.8070407@cygnus.com \
--to=ac131313@cygnus.com \
--cc=gdb-patches@sources.redhat.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