I tested it OK in VSProg, but in OpenOCD, no reply from the bug reporter.

> The patch reads a bit oddly -- why would any type of parsing
> code *add* characters instead of taking them out of the buffer??
It simply add a space as a delimiter.

Below is the report:
> Looking at svf.c, the code wrongly assumes that there is always a space
delimiter e.g. TDI<space>(123) when it can be TDI(123).
When implementing svf.c, my Altera sample SVF file do have a space between
TDI and (123).
So the svf.c code require a space between TDI and (123) to parse it
corrently.
Of course, a rewrite of the parser is welcome.
2010/1/19 Øyvind Harboe <oyvind.har...@zylin.com>

>  On Mon, Jan 18, 2010 at 10:13 PM, David Brownell <davi...@pacbell.net>
> wrote:
> > On Sunday 17 January 2010, Øyvind Harboe wrote:
> >> On Sun, Jan 17, 2010 at 9:56 PM, simon qian <
> simonqian.open...@gmail.com> wrote:
> >> > see http://forum.sparkfun.com/viewtopic.php?p=90983#90983.
> >> > Can anybody try this?
> >>
> >> If you can confirm that this solves the problem, I would want
> >> a comment that explains why this is necessary.
> >>
> >> If somebody goes in and rewrites that code later on, they need
> >> to know that there are compatibility(?) problems out there to
> >> deal with.
> >
> > The patch reads a bit oddly -- why would any type of parsing
> > code *add* characters instead of taking them out of the buffer??
>
> Because modifying the input is easier than parsing it correctly?
>
> If nobody can confirm that this fixes the problem, then there isn't
> a reason to clean up the patch though...
>
> --
> Ø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
>



-- 
Best Regards, SimonQian
http://www.SimonQian.com
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to