diff options
| author | Zachary T Welch <zw@superlucidity.net> | 2009-11-10 00:02:18 -0800 |
|---|---|---|
| committer | Zachary T Welch <zw@superlucidity.net> | 2009-11-13 10:55:27 -0800 |
| commit | d22270e0ed291d3b08fd03a25181b279ca5e0911 (patch) | |
| tree | e9467fb7e4a05043d18eb747d0b4bfd89511abf1 /src/target/arm9tdmi.c | |
| parent | a585bdf7269ce5c861c83ee3294ba1f074e9c877 (diff) | |
| download | openocd+libswd-d22270e0ed291d3b08fd03a25181b279ca5e0911.tar.gz openocd+libswd-d22270e0ed291d3b08fd03a25181b279ca5e0911.tar.bz2 openocd+libswd-d22270e0ed291d3b08fd03a25181b279ca5e0911.tar.xz openocd+libswd-d22270e0ed291d3b08fd03a25181b279ca5e0911.zip | |
command_handler_t: make argc unsigned
The number of command arguments will always be 0 or more, so use
the right type in handlers. This has a cascading effect up through
the layers, but the new COMMAND_HANDLER macros prevented total chaos.
Diffstat (limited to 'src/target/arm9tdmi.c')
| -rw-r--r-- | src/target/arm9tdmi.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/target/arm9tdmi.c b/src/target/arm9tdmi.c index 112ec2a4..416fe794 100644 --- a/src/target/arm9tdmi.c +++ b/src/target/arm9tdmi.c @@ -862,7 +862,6 @@ COMMAND_HANDLER(handle_arm9tdmi_catch_vectors_command) struct arm7_9_common_s *arm7_9 = target_to_arm7_9(target); reg_t *vector_catch; uint32_t vector_catch_value; - int i, j; /* it's uncommon, but some ARM7 chips can support this */ if (arm7_9->common_magic != ARM7_9_COMMON_MAGIC @@ -894,9 +893,10 @@ COMMAND_HANDLER(handle_arm9tdmi_catch_vectors_command) } else { - for (i = 0; i < argc; i++) + for (unsigned i = 0; i < argc; i++) { /* go through list of vectors */ + unsigned j; for (j = 0; arm9tdmi_vectors[j].name; j++) { if (strcmp(args[i], arm9tdmi_vectors[j].name) == 0) @@ -927,7 +927,7 @@ COMMAND_HANDLER(handle_arm9tdmi_catch_vectors_command) } /* output current settings */ - for (i = 0; arm9tdmi_vectors[i].name; i++) { + for (unsigned i = 0; arm9tdmi_vectors[i].name; i++) { command_print(cmd_ctx, "%s: %s", arm9tdmi_vectors[i].name, (vector_catch_value & arm9tdmi_vectors[i].value) ? "catch" : "don't catch"); |