Re: Watchpoint segfaults

[Daniel Jacobowitz <drow@false.org>]

On Sat, Dec 10, 2005 at 10:31:02AM +0200, Eli Zaretskii wrote:
> > Date: Fri, 9 Dec 2005 14:58:11 -0500
> > From: Daniel Jacobowitz <drow@false.org>
> > 
> > I think we want to temporarily disable breakpoints whose expression can't be
> > parsed
> 
> I agree.  But perhaps instead of just disabling it we should display
> some warning the first time we find that we are unable to parse its
> expression.

That sounds like a good idea.  We should also try to re-parse it at any
available opportunity, e.g. when the symbol table changes.  This'll
take a little work to get right.

> > and print it as a string rather than using the expression printer.
> 
> Print it where? in the table displayed by "info breakpoints" or
> equivalent commands?  Or do you have some other situation in mind?

Sorry, yes, I was referring to the "info breakpoints" segfault. 
Basically someone's going to have to go through and audit the uses of
->exp.

-- 
Daniel Jacobowitz
CodeSourcery, LLC