no1wudi commented on code in PR #1608:
URL: https://github.com/apache/nuttx-apps/pull/1608#discussion_r1122555742


##########
system/readline/readline_common.c:
##########
@@ -708,11 +708,6 @@ ssize_t readline_common(FAR struct rl_common_s *vtbl, FAR 
char *buf,
           buf[nch++] = '\n';
           buf[nch]   = '\0';
 
-#ifdef CONFIG_READLINE_ECHO

Review Comment:
   Maybe should rename it to represent its function now (cmd history/backspace 
etc)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to