Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
* [rfc/5.1] Document HP/UX build/config problems
@ 2001-10-15 19:54 Andrew Cagney
  2001-10-17 21:31 ` Andrew Cagney
  0 siblings, 1 reply; 2+ messages in thread
From: Andrew Cagney @ 2001-10-15 19:54 UTC (permalink / raw)
  To: gdb-patches

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++;
 >  	}
 >      }


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2001-10-17 21:31 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2001-10-15 19:54 [rfc/5.1] Document HP/UX build/config problems Andrew Cagney
2001-10-17 21:31 ` Andrew Cagney

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox