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/jtag/ft2232.c | |
parent | a585bdf7269ce5c861c83ee3294ba1f074e9c877 (diff) |
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/jtag/ft2232.c')
-rw-r--r-- | src/jtag/ft2232.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/jtag/ft2232.c b/src/jtag/ft2232.c index 6d233569..39036bcb 100644 --- a/src/jtag/ft2232.c +++ b/src/jtag/ft2232.c @@ -2874,8 +2874,7 @@ COMMAND_HANDLER(ft2232_handle_vid_pid_command) argc -= 1; } - int i; - int retval = ERROR_OK; + unsigned i; for (i = 0; i < argc; i += 2) { COMMAND_PARSE_NUMBER(u16, args[i], ft2232_vid[i >> 1]); @@ -2888,7 +2887,7 @@ COMMAND_HANDLER(ft2232_handle_vid_pid_command) */ ft2232_vid[i >> 1] = ft2232_pid[i >> 1] = 0; - return retval; + return ERROR_OK; } COMMAND_HANDLER(ft2232_handle_latency_command) |