This is an automated email from Gerrit.

Benjamin Shirley-Quirk (shirleyqu...@gmail.com) just uploaded a new patch set 
to Gerrit, which you can find at http://openocd.zylin.com/6444

-- gerrit

commit 6b3cce941fd1981f42e6ca634503f46fea280390
Author: Benjamin Shirley-Quirk <shirleyqu...@gmail.com>
Date:   Fri Aug 20 13:44:35 2021 +0100

    isspace and isprint take int
    ref https://sourceforge.net/p/openocd/tickets/315/
    Change-Id: I71fcfc28f0cb7d74257edf6ac6a9a6d1fc2b9947
    Signed-off-by: Benjamin Shirley-Quirk <shirleyqu...@gmail.com>

diff --git a/src/helper/log.c b/src/helper/log.c
index caa0a66..49d75d3 100644
--- a/src/helper/log.c
+++ b/src/helper/log.c
@@ -526,7 +526,7 @@ void log_socket_error(const char *socket_desc)
 char *find_nonprint_char(char *buf, unsigned buf_len)
 {
        for (unsigned int i = 0; i < buf_len; i++) {
-               if (!isprint(buf[i]))
+               if (!isprint((int)buf[i]))
                        return buf + i;
        }
        return NULL;
diff --git a/src/server/telnet_server.c b/src/server/telnet_server.c
index 697efa1..33ffbfb 100644
--- a/src/server/telnet_server.c
+++ b/src/server/telnet_server.c
@@ -446,7 +446,7 @@ static void telnet_auto_complete(struct connection 
*connection)
 
        /* user command position in the line, ignore leading spaces */
        size_t usr_cmd_pos = seq_start;
-       while ((usr_cmd_pos < t_con->line_cursor) && 
isspace(t_con->line[usr_cmd_pos]))
+       while ((usr_cmd_pos < t_con->line_cursor) && 
isspace((int)t_con->line[usr_cmd_pos]))
                usr_cmd_pos++;
 
        /* user command length */
@@ -457,8 +457,8 @@ static void telnet_auto_complete(struct connection 
*connection)
        size_t optimized_spaces = 0;
        char query[usr_cmd_len + 1];
        for (size_t i = 0; i < usr_cmd_len; i++) {
-               if ((i < usr_cmd_len - 1) && isspace(t_con->line[usr_cmd_pos + 
i])
-                               && isspace(t_con->line[usr_cmd_pos + i + 1])) {
+               if ((i < usr_cmd_len - 1) && 
isspace((int)t_con->line[usr_cmd_pos + i])
+                               && isspace((int)t_con->line[usr_cmd_pos + i + 
1])) {
                        optimized_spaces++;
                        continue;
                }

-- 

Reply via email to