Try and play nicer with UTF-8 input sequences

Author: AndreRenaud

embedded_cli.c

@@ -399,7 +399,7 @@ bool embedded_cli_insert_char(struct embedded_cli *cli, char ch)
             cli_putchar(cli, '\n', true);
             break;
         default:
-            if (ch > 0)
+            if (ch < 0 || ch >= 32)
                 embedded_cli_insert_default_char(cli, ch);
         }
     }

examples/posix_demo.c

@@ -95,7 +95,7 @@ int main(void)
             cli_argc = embedded_cli_argc(&cli, &cli_argv);
             printf("Got %d args\n", cli_argc);
             for (int i = 0; i < cli_argc; i++) {
-                printf("Arg %d/%d: '%s'\n", i, cli_argc, cli_argv[i]);
+                printf("Arg %d/%d: [%lu bytes] '%s'\n", i, cli_argc, strlen(cli_argv[i]), cli_argv[i]);
             }
             done = cli_argc >= 1 && (strcmp(cli_argv[0], "quit") == 0);
 

tests/embedded_cli_test.c

@@ -300,6 +300,20 @@ static void test_max_chars(void)
     TEST_ASSERT(cli.buffer[sizeof(cli.buffer) - 2] == 'f');
 }
 
+static void test_utf8(void)
+{
+    struct embedded_cli cli;
+    char **argv;
+    embedded_cli_init(&cli, NULL, NULL, NULL);
+    test_insert_line(&cli, "normal ñ ü 中文 text\n");
+    int argc = embedded_cli_argc(&cli, &argv);
+    TEST_ASSERT(argc == 5);
+    TEST_ASSERT(strcmp(argv[0], "normal") == 0);
+    TEST_ASSERT(strcmp(argv[1], "ñ") == 0);
+    TEST_ASSERT(strcmp(argv[2], "ü") == 0);
+    TEST_ASSERT(strcmp(argv[3], "中文") == 0);
+}
+
 TEST_LIST = {
     {"simple", test_simple},
     {"argc", test_argc},
@@ -317,5 +331,6 @@ TEST_LIST = {
     {"quotes", test_quotes},
     {"too_many_args", test_too_many_args},
     {"max_chars", test_max_chars},
-    {NULL, NULL},
+    {"utf8", test_utf8},
+    {NULL, NULL}
 };