From a7191e8bd7f4188d608764389d5a73fcf4b0e8da Mon Sep 17 00:00:00 2001 From: Tom Tromey Date: Sat, 18 Jan 2014 19:42:15 -0700 Subject: [PATCH] replace convert_ascii_to_int with hex2bin convert_ascii_to_int is identical to hex2bin. This removes the former. 2014-02-12 Tom Tromey * common/rsp-low.c (convert_ascii_to_int): Remove. * common/rsp-low.h (convert_ascii_to_int): Don't declare. 2014-02-12 Tom Tromey * ax.c (gdb_parse_agent_expr): Use hex2bin, not convert_ascii_to_int. * regcache.c (registers_to_string): Likewise. * remote-utils.c (decode_M_packet): Likewise. * server.c (process_serial_event): Likewise. --- gdb/ChangeLog | 5 +++++ gdb/common/rsp-low.c | 12 ------------ gdb/common/rsp-low.h | 2 -- gdb/gdbserver/ChangeLog | 8 ++++++++ gdb/gdbserver/ax.c | 2 +- gdb/gdbserver/regcache.c | 2 +- gdb/gdbserver/remote-utils.c | 2 +- gdb/gdbserver/server.c | 4 ++-- 8 files changed, 18 insertions(+), 19 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 82e09aa..fb8100a 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2014-02-12 Tom Tromey + * common/rsp-low.c (convert_ascii_to_int): Remove. + * common/rsp-low.h (convert_ascii_to_int): Don't declare. + +2014-02-12 Tom Tromey + * common/rsp-low.h (unhexify): Don't declare. * common/rsp-low.c (unhexify): Remove. diff --git a/gdb/common/rsp-low.c b/gdb/common/rsp-low.c index decd23c..ecaa69a 100644 --- a/gdb/common/rsp-low.c +++ b/gdb/common/rsp-low.c @@ -127,18 +127,6 @@ hex2bin (const char *hex, gdb_byte *bin, int count) return i; } -void -convert_ascii_to_int (const char *from, unsigned char *to, int n) -{ - int nib1, nib2; - while (n--) - { - nib1 = fromhex (*from++); - nib2 = fromhex (*from++); - *to++ = (((nib1 & 0x0f) << 4) & 0xf0) | (nib2 & 0x0f); - } -} - int bin2hex (const gdb_byte *bin, char *hex, int count) { diff --git a/gdb/common/rsp-low.h b/gdb/common/rsp-low.h index b793f15..9edf4dc 100644 --- a/gdb/common/rsp-low.h +++ b/gdb/common/rsp-low.h @@ -36,8 +36,6 @@ extern char *unpack_varlen_hex (char *buff, ULONGEST *result); extern int hex2bin (const char *hex, gdb_byte *bin, int count); -extern void convert_ascii_to_int (const char *from, unsigned char *to, int n); - extern int bin2hex (const gdb_byte *bin, char *hex, int count); /* Convert BUFFER, binary data at least LEN bytes long, into escaped diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index f0d092d..fc105c4 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,5 +1,13 @@ 2014-02-12 Tom Tromey + * ax.c (gdb_parse_agent_expr): Use hex2bin, not + convert_ascii_to_int. + * regcache.c (registers_to_string): Likewise. + * remote-utils.c (decode_M_packet): Likewise. + * server.c (process_serial_event): Likewise. + +2014-02-12 Tom Tromey + * server.c (handle_query, handle_v_run): Use hex2bin, not unhexify. * tracepoint.c (cmd_qtdpsrc, cmd_qtdv, cmd_qtnotes): Likewise. diff --git a/gdb/gdbserver/ax.c b/gdb/gdbserver/ax.c index 1eefaf3..ef659dc 100644 --- a/gdb/gdbserver/ax.c +++ b/gdb/gdbserver/ax.c @@ -105,7 +105,7 @@ gdb_parse_agent_expr (char **actparm) aexpr = xmalloc (sizeof (struct agent_expr)); aexpr->length = xlen; aexpr->bytes = xmalloc (xlen); - convert_ascii_to_int (act, aexpr->bytes, xlen); + hex2bin (act, aexpr->bytes, xlen); *actparm = act + (xlen * 2); return aexpr; } diff --git a/gdb/gdbserver/regcache.c b/gdb/gdbserver/regcache.c index 6e5aa08..bed10b48 100644 --- a/gdb/gdbserver/regcache.c +++ b/gdb/gdbserver/regcache.c @@ -234,7 +234,7 @@ registers_from_string (struct regcache *regcache, char *buf) if (len > tdesc->registers_size * 2) len = tdesc->registers_size * 2; } - convert_ascii_to_int (buf, registers, len / 2); + hex2bin (buf, registers, len / 2); } struct reg * diff --git a/gdb/gdbserver/remote-utils.c b/gdb/gdbserver/remote-utils.c index 3b3f183..4fcafa0 100644 --- a/gdb/gdbserver/remote-utils.c +++ b/gdb/gdbserver/remote-utils.c @@ -1270,7 +1270,7 @@ decode_M_packet (char *from, CORE_ADDR *mem_addr_ptr, unsigned int *len_ptr, if (*to_p == NULL) *to_p = xmalloc (*len_ptr); - convert_ascii_to_int (&from[i++], *to_p, *len_ptr); + hex2bin (&from[i++], *to_p, *len_ptr); } int diff --git a/gdb/gdbserver/server.c b/gdb/gdbserver/server.c index 1ea1bde..a441afd 100644 --- a/gdb/gdbserver/server.c +++ b/gdb/gdbserver/server.c @@ -3624,7 +3624,7 @@ process_serial_event (void) break; case 'C': require_running (own_buf); - convert_ascii_to_int (own_buf + 1, &sig, 1); + hex2bin (own_buf + 1, &sig, 1); if (gdb_signal_to_host_p (sig)) signal = gdb_signal_to_host (sig); else @@ -3633,7 +3633,7 @@ process_serial_event (void) break; case 'S': require_running (own_buf); - convert_ascii_to_int (own_buf + 1, &sig, 1); + hex2bin (own_buf + 1, &sig, 1); if (gdb_signal_to_host_p (sig)) signal = gdb_signal_to_host (sig); else -- 2.7.4