diff options
| author | Erhan Kurubas <erhan.kurubas@espressif.com> | 2023-04-02 00:26:51 +0200 |
|---|---|---|
| committer | Erhan Kurubas <erhan.kurubas@espressif.com> | 2023-07-20 23:09:06 +0200 |
| commit | e6f30aef80c952b9b60a3e8364f645b7fa287d58 (patch) | |
| tree | 9e7aa8ca85f1bf57577c6d5eadab13101ce639e7 /src | |
| parent | 1a089d8409f159552a35a4ed9e5205586f076a05 (diff) | |
| download | riscv-openocd-e6f30aef80c952b9b60a3e8364f645b7fa287d58.zip riscv-openocd-e6f30aef80c952b9b60a3e8364f645b7fa287d58.tar.gz riscv-openocd-e6f30aef80c952b9b60a3e8364f645b7fa287d58.tar.bz2 | |
src: fix clang15 compiler warnings
Below warnings are fixed.
1- A function declaration without a prototype is deprecated in all
versions of C [-Werror,-Wstrict-prototypes]
2- error: variable set but not used [-Werror,-Wunused-but-set-variable]
Signed-off-by: Erhan Kurubas <erhan.kurubas@espressif.com>
Change-Id: I1cf14b8e5e3e732ebc9cacc4b1cb9009276a8ea9
Reviewed-on: https://review.openocd.org/c/openocd/+/7569
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
Diffstat (limited to 'src')
| -rw-r--r-- | src/server/gdb_server.c | 8 | ||||
| -rw-r--r-- | src/target/arm7_9_common.c | 2 | ||||
| -rw-r--r-- | src/target/target.c | 4 |
3 files changed, 4 insertions, 10 deletions
diff --git a/src/server/gdb_server.c b/src/server/gdb_server.c index 8360ee1..41a47bf 100644 --- a/src/server/gdb_server.c +++ b/src/server/gdb_server.c @@ -3012,7 +3012,8 @@ static int gdb_query_packet(struct connection *connection, return ERROR_OK; } -static bool gdb_handle_vcont_packet(struct connection *connection, const char *packet, int packet_size) +static bool gdb_handle_vcont_packet(struct connection *connection, const char *packet, + __attribute__((unused)) int packet_size) { struct gdb_connection *gdb_connection = connection->priv; struct target *target = get_available_target_from_connection(connection); @@ -3031,7 +3032,6 @@ static bool gdb_handle_vcont_packet(struct connection *connection, const char *p if (parse[0] == ';') { ++parse; - --packet_size; } /* simple case, a continue packet */ @@ -3070,14 +3070,11 @@ static bool gdb_handle_vcont_packet(struct connection *connection, const char *p int current_pc = 1; int64_t thread_id; parse++; - packet_size--; if (parse[0] == ':') { char *endp; parse++; - packet_size--; thread_id = strtoll(parse, &endp, 16); if (endp) { - packet_size -= endp - parse; parse = endp; } } else { @@ -3115,7 +3112,6 @@ static bool gdb_handle_vcont_packet(struct connection *connection, const char *p if (parse[0] == ';') { ++parse; - --packet_size; if (parse[0] == 'c') { parse += 1; diff --git a/src/target/arm7_9_common.c b/src/target/arm7_9_common.c index 8f87d9c..0632290 100644 --- a/src/target/arm7_9_common.c +++ b/src/target/arm7_9_common.c @@ -1553,7 +1553,6 @@ static int arm7_9_restore_context(struct target *target) if (dirty) { uint32_t mask = 0x0; - int num_regs = 0; uint32_t regs[16]; if (mode_change) { @@ -1576,7 +1575,6 @@ static int arm7_9_restore_context(struct target *target) if (reg->dirty) { regs[j] = buf_get_u32(reg->value, 0, 32); mask |= 1 << j; - num_regs++; reg->dirty = false; reg->valid = true; LOG_DEBUG("writing register %i mode %s " diff --git a/src/target/target.c b/src/target/target.c index 581f5ef..6481dfb 100644 --- a/src/target/target.c +++ b/src/target/target.c @@ -1948,13 +1948,13 @@ static int target_call_timer_callbacks_check_time(int checktime) return ERROR_OK; } -int target_call_timer_callbacks() +int target_call_timer_callbacks(void) { return target_call_timer_callbacks_check_time(1); } /* invoke periodic callbacks immediately */ -int target_call_timer_callbacks_now() +int target_call_timer_callbacks_now(void) { return target_call_timer_callbacks_check_time(0); } |