[patch] partial-stab.h

Elena Zannoni ezannoni@cygnus.com
Thu Aug 10 19:36:00 GMT 2000


Andrew Cagney writes:
 > Just FYI,
 > 
 > The check for PST was redundant (a wrapping if() had already checked
 > it).  GCC didn't like it either :-)
 > 
 > 	enjoy,

Oops, sorry, I didn't review it carefully enough.

Thanks for catching it
Elena

 > 		Andrew2000-08-10  Andrew Cagney  <cagney@ops1.cygnus.com>
 > 
 > 	* partial-stab.h (DBX_READ): Eliminate redundant check for null
 > 	``pst''.  Also fixes GCC warning.
 > 
 > Index: partial-stab.h
 > ===================================================================
 > RCS file: /cvs/src/src/gdb/partial-stab.h,v
 > retrieving revision 1.4
 > diff -p -r1.4 partial-stab.h
 > *** partial-stab.h	2000/08/10 18:26:56	1.4
 > --- partial-stab.h	2000/08/11 01:43:48
 > *************** switch (CUR_SYMBOL_TYPE)
 > *** 401,407 ****
 >   	   function relative stabs, or the address of the function's
 >   	   end for old style stabs.  */
 >   	valu = CUR_SYMBOL_VALUE + last_function_start;
 > ! 	if (pst && pst->texthigh == 0 || valu > pst->texthigh)
 >   	  pst->texthigh = valu;
 >   	break;
 >         }
 > --- 401,407 ----
 >   	   function relative stabs, or the address of the function's
 >   	   end for old style stabs.  */
 >   	valu = CUR_SYMBOL_VALUE + last_function_start;
 > ! 	if (pst->texthigh == 0 || valu > pst->texthigh)
 >   	  pst->texthigh = valu;
 >   	break;
 >         }


More information about the Gdb-patches mailing list