On Thursday 07 January 2010, Øyvind Harboe wrote:
> On Fri, Jan 8, 2010 at 6:00 AM, Robert Jenssen
> <robertjens...@ozemail.com.au> wrote:
> > Hi David,
> >
> > Well, this also works for me:
> >
> >
> > *** src/server/server.c 2009-12-22 07:52:03.000000000 +1100
> > --- src/server/server.c.new     2010-01-08 15:57:32.000000000 +1100
> > ***************
> > *** 565,571 ****
> >  {
> >        shutdown_openocd = 1;
> >
> > !       return ERROR_COMMAND_CLOSE_CONNECTION;
> >  }
> >
> >  static const struct command_registration server_command_handlers[] = {
> > --- 565,571 ----
> >  {
> >        shutdown_openocd = 1;
> >
> > !       return ERROR_OK;
> >  }
> 
> I believe this approach is essentially sound and also cleaner than
> invoking exit().

It's not a patch format I feel comfortable reading; it's just
changing to ERROR_OK?

If thats the case, then feel free to commit.  My concern is just
that this bug be fixed.

- Dave


> Eventually we might want to introduce a fn rather than modifying a global
> variable, but that doesn't change the essential approach.
> 
> 
> 
> 
> -- 
> Øyvind Harboe
> US toll free 1-866-980-3434 / International +47 51 63 25 00
> http://www.zylin.com/zy1000.html
> ARM7 ARM9 ARM11 XScale Cortex
> JTAG debugger and flash programmer
> 
> 


_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to