diff options
author | David Brownell <dbrownell@users.sourceforge.net> | 2010-04-04 00:38:39 -0700 |
---|---|---|
committer | David Brownell <dbrownell@users.sourceforge.net> | 2010-04-04 00:38:39 -0700 |
commit | 88fcb5a9ef971e54166de7cd16a3b0be20113b82 (patch) | |
tree | a82b4df0999e3d4c72b0d1a0211de47b0495bea0 /src/pld/xilinx_bit.c | |
parent | d37a10da52bc8e6df4e3df2edd02ddbc29fe3bc4 (diff) | |
download | openocd_libswd-88fcb5a9ef971e54166de7cd16a3b0be20113b82.tar.gz openocd_libswd-88fcb5a9ef971e54166de7cd16a3b0be20113b82.tar.bz2 openocd_libswd-88fcb5a9ef971e54166de7cd16a3b0be20113b82.tar.xz openocd_libswd-88fcb5a9ef971e54166de7cd16a3b0be20113b82.zip |
simplify and unconfuse target_run_algorithm()
For some reason there are *two* schemes for interposing logic into
the run_algorithm() code path... One is a standard procedural wapper
around the target method invocation.
the other (superfluous) one hacked the method table by splicing
a second procedural wrapper into the method table. Remove it:
* Rename its slightly-more-featureful wrapper so it becomes
the standard procedural wrapper, leaving its added logic
(where it should have been in the first place.
Also add a paranoia check, to report targets that don't
support algorithms without traversing a NULL pointer, and
tweak its code structure a bit so it's easier to modify.
* Get rid of the superfluous/conusing method table hacks.
This is a net simplification, making it simpler to analyse what's
going on, and then interpose logic . ... by ensuring there's only one
natural place for it to live.
------------
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Diffstat (limited to 'src/pld/xilinx_bit.c')
0 files changed, 0 insertions, 0 deletions