From 9fce0e5659af2210dffef48f83531fe3f46963cf Mon Sep 17 00:00:00 2001
From: Meco Man <920369182@qq.com>
Date: Wed, 13 Oct 2021 22:38:01 -0400
Subject: [PATCH] [shell] improve finsh_getchar

---
 components/finsh/shell.c | 12 ++++++++----
 components/finsh/shell.h |  2 +-
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/components/finsh/shell.c b/components/finsh/shell.c
index 7b577db6b6..20f1bdef0f 100644
--- a/components/finsh/shell.c
+++ b/components/finsh/shell.c
@@ -142,20 +142,24 @@ void finsh_set_prompt_mode(rt_uint32_t prompt_mode)
     shell->prompt_mode = prompt_mode;
 }
 
-char finsh_getchar(void)
+int finsh_getchar(void)
 {
 #ifdef RT_USING_DEVICE
 #ifdef RT_USING_POSIX
     return getchar();
 #else
     char ch = 0;
+    rt_device_t device;
 
     RT_ASSERT(shell != RT_NULL);
-    if (shell->device == RT_NULL)
+
+    device = shell->device;
+    if (device == RT_NULL)
     {
-        return -RT_ERROR;
+        return -1; /* EOF */
     }
-    while (rt_device_read(shell->device, -1, &ch, 1) != 1)
+
+    while (rt_device_read(device, -1, &ch, 1) != 1)
         rt_sem_take(&shell->rx_sem, RT_WAITING_FOREVER);
 
     return ch;
diff --git a/components/finsh/shell.h b/components/finsh/shell.h
index b671db0849..985d3b3fbd 100644
--- a/components/finsh/shell.h
+++ b/components/finsh/shell.h
@@ -93,7 +93,7 @@ rt_uint32_t finsh_get_echo(void);
 int finsh_system_init(void);
 void finsh_set_device(const char *device_name);
 const char *finsh_get_device(void);
-char finsh_getchar(void);
+int finsh_getchar(void);
 
 rt_uint32_t finsh_get_prompt_mode(void);
 void finsh_set_prompt_mode(rt_uint32_t prompt_mode);
-- 
GitLab