On Sunday 13 July 2008 00:14:42 Ulrich Weigand wrote:
> Vladimir Prus wrote:
> 
> >  mi_cmd_execute (struct mi_parse *parse)
> >  {
> >    struct cleanup *cleanup = make_cleanup (null_cleanup, NULL);
> > +  char *thread_str;
> > +  char *frame_str;
> > +  int thread;
> > +  int i;
> 
> [snip]
> 
> > +	error (_("Invalid frame id: %s"), frame_str);
> 
> This variable is used uninitialized, which breaks the -Werror
> build for me.
> 
> (Also, it seems the other local variables introduced by this
> patch are never used ...)

Apologies. I've checked in the attached to fix this.

- Volodya