[rfc] : current_gdbarch replacement in breakpoint.c

Ulrich Weigand uweigand@de.ibm.com
Thu Jan 17 14:30:00 GMT 2008


Markus Deuling wrote:

> -      if ((!within_current_scope || fr == get_current_frame ())
> -          && gdbarch_in_function_epilogue_p (current_gdbarch, read_pc ()))
> -	return WP_VALUE_NOT_CHANGED;
> +      if (!within_current_scope)
> +	{
> +	  fr = get_current_frame ();

This does not look correct to me.  The old code checked for the in-epilogue
situation when FR was the current frame; your code simply removes that check.

Bye,
Ulrich

-- 
  Dr. Ulrich Weigand
  GNU Toolchain for Linux on System z and Cell BE
  Ulrich.Weigand@de.ibm.com



More information about the Gdb-patches mailing list