interesting
/
riscv-openocd
mirror of https://github.com/riscv/riscv-openocd.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Remove long-deprecated "target count" and "target number" commands. 

Given that the manual states that these two subcommands are
deprecated and were scheduled to be removed back in 2010,
remove them and the corresponding documentation from the
manual.

Change-Id: Iaac633349d7fcb8b7f964109c7d26dd0cc5fc233
Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
Reviewed-on: http://openocd.zylin.com/1860
Tested-by: jenkins
Reviewed-by: Paul Fertser <fercerpav@gmail.com>
This commit is contained in:
Robert P. J. Day 2014-01-09 11:59:37 -05:00 committed by Paul Fertser
parent 10331d2007
commit 41124ea992
2 changed files with 0 additions and 92 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
doc/openocd.texi
View File

@ -4133,22 +4133,6 @@ are examples; and there are many more.
Several commands let you examine the list of targets:
@deffn Command {target count}
@emph{Note: target numbers are deprecated; don't use them.
They will be removed shortly after August 2010, including this command.
Iterate target using @command{target names}, not by counting.}
Returns the number of targets, @math{N}.
The highest numbered target is @math{N - 1}.
@example
set c [target count]
for @{ set x 0 @} @{ $x < $c @} @{ incr x @} @{
# Assuming you have created this function
print_target_details $x
@}
@end example
@end deffn
@deffn Command {target current}
Returns the name of the current target.
@end deffn
@ -4162,18 +4146,6 @@ foreach t [target names] @{
@end example
@end deffn
@deffn Command {target number} number
@emph{Note: target numbers are deprecated; don't use them.
They will be removed shortly after August 2010, including this command.}
The list of targets is numbered starting at zero.
This command returns the name of the target at index @var{number}.
@example
set thename [target number $x]
puts [format "Target %d is: %s\n" $x $thename]
@end example
@end deffn
@c yep, "target list" would have been better.
@c plus maybe "target setdefault".

64
src/target/target.c
View File

@ -5333,55 +5333,6 @@ static int jim_target_create(Jim_Interp *interp, int argc, Jim_Obj *const *argv)
return target_create(&goi);
}
static int jim_target_number(Jim_Interp *interp, int argc, Jim_Obj *const *argv)
{
Jim_GetOptInfo goi;
Jim_GetOpt_Setup(&goi, interp, argc - 1, argv + 1);
/* It's OK to remove this mechanism sometime after August 2010 or so */
LOG_WARNING("don't use numbers as target identifiers; use names");
if (goi.argc != 1) {
Jim_SetResultFormatted(goi.interp, "usage: target number <number>");
return JIM_ERR;
}
jim_wide w;
int e = Jim_GetOpt_Wide(&goi, &w);
if (e != JIM_OK)
return JIM_ERR;
struct target *target;
for (target = all_targets; NULL != target; target = target->next) {
if (target->target_number != w)
continue;
Jim_SetResultString(goi.interp, target_name(target), -1);
return JIM_OK;
}
{
Jim_Obj *wObj = Jim_NewIntObj(goi.interp, w);
Jim_SetResultFormatted(goi.interp,
"Target: number %#s does not exist", wObj);
Jim_FreeNewObj(interp, wObj);
}
return JIM_ERR;
}
static int jim_target_count(Jim_Interp *interp, int argc, Jim_Obj *const *argv)
{
if (argc != 1) {
Jim_WrongNumArgs(interp, 1, argv, "<no parameters>");
return JIM_ERR;
}
unsigned count = 0;
struct target *target = all_targets;
while (NULL != target) {
target = target->next;
count++;
}
Jim_SetResult(interp, Jim_NewIntObj(interp, count));
return JIM_OK;
}
static const struct command_registration target_subcommand_handlers[] = {
{
.name = "init",
@ -5416,21 +5367,6 @@ static const struct command_registration target_subcommand_handlers[] = {
.jim_handler = jim_target_names,
.help = "Returns the names of all targets as a list of strings",
},
{
.name = "number",
.mode = COMMAND_ANY,
.jim_handler = jim_target_number,
.usage = "number",
.help = "Returns the name of the numbered target "
"(DEPRECATED)",
},
{
.name = "count",
.mode = COMMAND_ANY,
.jim_handler = jim_target_count,
.help = "Returns the number of targets as an integer "
"(DEPRECATED)",
},
{
.name = "smp",
.mode = COMMAND_ANY,