add option to disable the echo

CLI/CLI.c

@@ -30,13 +30,14 @@ CLI_t CLI_init(CLI_charOutFn lineOut, CMDList_t* cmdList)
 	cli.CLI_charOut = lineOut;
 	cli.CMDList = cmdList;
 	cli.CLI_State = CLI_State_Default;
+	cli.echo = true;
 
 	cli.FIFO = FIFOBuffChar_create();
 #ifdef HISTORY
 	cli.History = History_init();
 #endif
 
-	if (cli.CLI_charOut != NULL)
+	if (cli.CLI_charOut != NULL && cli.echo)
 	{
 		CLI_stringOut(&cli, (char*)"> ");
 	}
@@ -128,7 +129,11 @@ void historyPrevius(CLI_t* cli)
 	// write line
 	if ((ret >= 0) && (line != NULL))
 	{
-		CLI_stringOut(cli, line);
+		int i;
+		for (i=0; *(line + i) != '\0'; i++)
+		{
+			CLI_charIn(cli, *(line + i));
+		}
 	}
 }
 void historyNext(CLI_t* cli)
@@ -148,7 +153,10 @@ void historyNext(CLI_t* cli)
 	if ((ret >= 0) && (line != NULL))
 	{
 		int i;
-		CLI_stringOut(cli, line);
+		for (i=0; *(line + i) != '\0'; i++)
+		{
+			CLI_charIn(cli, *(line + i));
+		}
 	}
 }
 #endif
@@ -161,7 +169,10 @@ int tryExecute(CLI_t* cli, FIFOBuffChar_t* fifo)
 	if (*line == '\0')
 	{ // empty line
 		ret = 1;
-		CLI_stringOut(cli, (char*)"> ");
+		if (cli->echo)
+		{
+			CLI_stringOut(cli, (char*)"> ");
+		}
 	}
 	else
 	{
@@ -173,9 +184,9 @@ int tryExecute(CLI_t* cli, FIFOBuffChar_t* fifo)
 #ifdef HISTORY
 		History_put(cli->History, line);
 #endif
-		int ret = (*(cmd->fn))(line);
+		int ret = (*(cmd->fn))(line, cli);
 
-		if (ret != INT_MIN)
+		if (ret != INT_MIN && cli->echo)
 		{
 			CLI_stringOut(cli, (char*)"> ");
 		}
@@ -184,8 +195,12 @@ int tryExecute(CLI_t* cli, FIFOBuffChar_t* fifo)
 	else if (ret == 0)
 	{
 		char err[100];
-		sprintf(&err[0], "command not found: %s\n> ", line);
+		sprintf(&err[0], "command not found: %s\n", line);
 		CLI_stringOut(cli, &err[0]);
+		if (cli->echo)
+		{
+			CLI_stringOut(cli, (char*)"> ");
+		}
 		ret = -1;
 
 #ifdef HISTORY
@@ -215,7 +230,10 @@ bool CLI_charIn(CLI_t* cli, char c)
 			if ((C >= ' ') && (C <= '~')) // see ascii table
 			{
 				FIFOBuffChar_put(cli->FIFO, C); // save char in buffer
-				CLI_charOut_save(cli, c);
+				if (cli->echo)
+				{
+					CLI_charOut_save(cli, c);
+				}
 			}
 			else
 			{
@@ -224,7 +242,10 @@ bool CLI_charIn(CLI_t* cli, char c)
 					case '\n':
 					case '\r':
 						// echo to terminal
-						CLI_charOut_save(cli, c);
+						if (cli->echo)
+						{
+							CLI_charOut_save(cli, c);
+						}
 						// copy buffer and create new buffer
 						FIFOBuffChar_t* fifo = cli->FIFO;
 						cli->FIFO = FIFOBuffChar_create();
@@ -238,7 +259,10 @@ bool CLI_charIn(CLI_t* cli, char c)
 					case 127: // delete (backspace)
 						if (FIFOBuffChar_pop(cli->FIFO))
 						{ // pop something of the buffer
-							CLI_stringOut(cli, (char*)"\x1b[D \x1b[D"); // "<left arrow><space><left arrow>"
+							if (cli->echo)
+							{
+								CLI_stringOut(cli, (char*)"\x1b[D \x1b[D"); // "<left arrow><space><left arrow>"
+							}
 						}
 						break;
 
@@ -247,8 +271,11 @@ bool CLI_charIn(CLI_t* cli, char c)
 						break;
 
 					default:
-						sprintf(&str[0], "\ninvlid char: '%c' - (%i)\n", c, c);
+						if (cli->echo)
-						CLI_stringOut(cli, &str[0]);
+						{
+							sprintf(&str[0], "\ninvlid char: '%c' - (%i)\n", c, c);
+							CLI_stringOut(cli, &str[0]);
+						}
 						break;
 				}
 			}
@@ -278,11 +305,17 @@ bool CLI_charIn(CLI_t* cli, char c)
 					historyNext(cli);
 					break;
 				case 'C': // arrow right
-					CLI_stringOut(cli, (char*)"(key: arrow right)");
+					if (cli->echo)
+					{
+						CLI_stringOut(cli, (char*)"(key: arrow right)");
+					}
 					cli->CLI_State = CLI_State_Default;
 					break;
 				case 'D': // arrow left
-					CLI_stringOut(cli, (char*)"(key: arrow left)");
+					if (cli->echo)
+					{
+						CLI_stringOut(cli, (char*)"(key: arrow left)");
+					}
 					cli->CLI_State = CLI_State_Default;
 					break;
 #endif

CLI/CLI.h

@@ -4,7 +4,6 @@
 #include <stdbool.h>
 
 #include <config.h>
-#define HISTORY
 
 #include "../CMDList/CMDList.h"
 #include "../FIFOBuff/FIFOBuffChar.h"
@@ -27,6 +26,7 @@ typedef struct CLI {
    History_t* History;
 #endif
 	CLI_State_t CLI_State;
+	bool echo;
 } CLI_t;
 
 // initilize and register the lineout print function
@@ -39,6 +39,8 @@ extern void CLI_PrintHistory(CLI_t* cli);
 
 // to recive a single caracter
 bool CLI_charIn(CLI_t* cli, char c);
+void CLI_charOut_save(CLI_t* cli, char ch);
+void CLI_stringOut(CLI_t* cli, char* str);
 
 
 #endif

CMDList/CMDList.h

@@ -3,7 +3,7 @@
 
 typedef struct CMD_t {
 	char* cmd;
-	int (*fn)(char* line);
+	int (*fn)(char* line, void* cli);
 } CMD_t;
 
 typedef struct CMDList_s {

CMakeLists.txt

@@ -1,18 +1,10 @@
 
-# FIFO Buffer
+idf_component_register(
-add_library(FIFOBuffChar FIFOBuff/FIFOBuffChar.c)
+	COMPONENT_ALIAS "cli"
-
+	SRCS
-# CMD List
+		CLI/CLI.c
-add_library(CMDList CMDList/CMDList.c)
+		FIFOBuff/FIFOBuffChar.c
-add_library(CMDListPrint CMDList/printList.c)
+		CMDList/CMDList.c
-
+		History/History.c
-# History
+	INCLUDE_DIRS "./"
-add_library(history History/History.c)
+	PRIV_INCLUDE_DIRS "../../src")
-
-# CLI
-add_library(CLI CLI/CLI.c)
-target_link_libraries(CLI CMDList FIFOBuffChar history)
-
-add_library(CLI_History CLI/CLI.c)
-target_link_libraries(CLI_History CMDList FIFOBuffChar history)
-target_compile_definitions(CLI_History PRIVATE HISTORY)

History/History.c

@@ -123,7 +123,7 @@ int History_getPrev(History_t* history, char** line)
 		*line = NULL;
 		retCode = -1;
 	}
-	return 0;
+	return retCode;
 }
 
 int History_getNext(History_t* history, char** line)
@@ -144,7 +144,7 @@ int History_getNext(History_t* history, char** line)
 		*line = NULL;
 		retCode = -1;
 	}
-	return 0;
+	return retCode;
 }
 
 int History_getFullHistory(History_t* history, char*** list)