On Wed, Jan 20, 2010 at 8:42 PM, David Brownell <[email protected]> wrote:
> On Wednesday 20 January 2010, Řyvind Harboe wrote:
>> arm7_9 fast_memory_access and working area nags added.
>>
>> Signed-off-by: Řyvind Harboe <[email protected]>
>
> OK, but it'd be *much* cleaner if it did
>
>        if (!get_target_reset_nag())
>                return ERROR_OK;
>
>        if (problem)
>                nag(problem)
>        ... rinse, repeat ...

If I can repeat the tests, I'll do like the above. I merged what I tested.

Me paranoid? I have nnneeeever made just "one more trivial change"
after I tested something and then committed. noooo :-)


-- 
Ø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
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to