From 96cf0c8d651ccc7d996fe002824d146868ebda1a Mon Sep 17 00:00:00 2001 From: prozessorkern Date: Sat, 28 Mar 2020 12:08:01 +0100 Subject: [PATCH] added busy command to the example + declared option parser options as const --- .vscode/launch.json | 2 +- api/shellmatta.h | 12 ++--- example/main.c | 104 +++++++++++++++++++++++++++++-------------- src/shellmatta_opt.c | 28 ++++++------ src/shellmatta_opt.h | 12 ++--- 5 files changed, 98 insertions(+), 60 deletions(-) diff --git a/.vscode/launch.json b/.vscode/launch.json index 83b5550..66e525d 100644 --- a/.vscode/launch.json +++ b/.vscode/launch.json @@ -9,7 +9,7 @@ "type": "cppdbg", "request": "launch", "program": "${workspaceFolder}/output/example/example", - "args": ["/dev/pts/3"], + "args": ["/dev/pts/4"], "stopAtEntry": false, "cwd": "${workspaceFolder}", "environment": [], diff --git a/api/shellmatta.h b/api/shellmatta.h index f9a7394..11c0e52 100644 --- a/api/shellmatta.h +++ b/api/shellmatta.h @@ -192,16 +192,16 @@ shellmatta_retCode_t shellmatta_read( shellmatta_handle_t handle, uint32_t *length); shellmatta_retCode_t shellmatta_opt( shellmatta_handle_t handle, - char *optionString, + const char *optionString, char *option, char **argument, uint32_t *argLen); -shellmatta_retCode_t shellmatta_opt_long( shellmatta_handle_t handle, - shellmatta_opt_long_t *longOptions, - char *option, - char **argument, - uint32_t *argLen); +shellmatta_retCode_t shellmatta_opt_long( shellmatta_handle_t handle, + const shellmatta_opt_long_t *longOptions, + char *option, + char **argument, + uint32_t *argLen); #ifndef SHELLMATTA_STRIP_PRINTF shellmatta_retCode_t shellmatta_printf( shellmatta_handle_t handle, diff --git a/example/main.c b/example/main.c index 1f37a43..0ae3d60 100644 --- a/example/main.c +++ b/example/main.c @@ -1,19 +1,23 @@ /* - * main.c + * Copyright (c) 2019 Stefan Strobel * - * Created on: Jun 10, 2019 - * Author: stefan + * This Source Code Form is subject to the terms of the Mozilla Public + * License, v. 2.0. If a copy of the MPL was not distributed with this + * file, You can obtain one at https://mozilla.org/MPL/2.0/. */ +/** + * @file main.c + * @brief main module to demonstrate use of the shellmatta. + * @author Stefan Strobel + */ #include "shellmatta.h" #include #include -#include #include #include #include -#include #include #include @@ -22,24 +26,6 @@ static bool exitRequest = false; int f; shellmatta_handle_t handle; -void set_blocking (int fd, int should_block) -{ - struct termios tty; - memset (&tty, 0, sizeof tty); - if (tcgetattr (fd, &tty) != 0) - { - printf ("error %d from tggetattr", errno); - return; - } - - tty.c_cc[VMIN] = should_block ? 1 : 0; - tty.c_cc[VTIME] = 5; // 0.5 seconds read timeout - - if (tcsetattr (fd, TCSANOW, &tty) != 0) - printf ("error %d setting term attributes", errno); -} - - static shellmatta_retCode_t doSomething(shellmatta_handle_t handle, const char *arguments, uint32_t length) { shellmatta_printf(handle, "%s - length: %u", arguments, length); @@ -101,23 +87,47 @@ shellmatta_cmd_t emptyCommand = {"empty", NULL, NULL, NULL, empty, NULL}; static shellmatta_retCode_t reset(shellmatta_handle_t handle, const char *arguments, uint32_t length) { + shellmatta_retCode_t ret; (void)arguments; (void)length; + char option; + char *argument; + uint32_t argLen; + bool printPrompt = false; - if(0 == strncmp(arguments, "prompt", length)) + static const shellmatta_opt_long_t options[] = { - shellmatta_resetShell(handle, true); - } - else + {"prompt", 'p', SHELLMATTA_OPT_ARG_REQUIRED}, + {NULL, '\0', SHELLMATTA_OPT_ARG_NONE} + }; + + ret = shellmatta_opt_long(handle, options, &option, &argument, &argLen); + while(SHELLMATTA_OK == ret) { - shellmatta_resetShell(handle, false); + switch(option) + { + case 'p': + if(NULL != argument) + { + if(0 == strncmp("true", argument, 4u)) + { + printPrompt = true; + } + } + break; + default: + shellmatta_printf(handle, "Unknown option: %c\r\n", option); + break; + } + ret = shellmatta_opt_long(handle, options, &option, &argument, &argLen); } + shellmatta_resetShell(handle, printPrompt); shellmatta_configure(handle, SHELLMATTA_MODE_INSERT, true, '\r'); return SHELLMATTA_OK; } -shellmatta_cmd_t resetCommand = {"reset", NULL, "resets the shellmatta instance", "reset [prompt]", reset, NULL}; +shellmatta_cmd_t resetCommand = {"reset", NULL, "resets the shellmatta instance", "reset [--prompt true/false]", reset, NULL}; static shellmatta_retCode_t continuous(shellmatta_handle_t handle, const char *arguments, uint32_t length) { @@ -143,6 +153,28 @@ static shellmatta_retCode_t continuous(shellmatta_handle_t handle, const char *a } shellmatta_cmd_t continuousCommand = {"continuous", "cont", "prints continously all input bytes", "continuous", continuous, NULL}; +static shellmatta_retCode_t busy(shellmatta_handle_t handle, const char *arguments, uint32_t length) +{ + (void)arguments; + (void)length; + static uint32_t callCnt = 0u; + shellmatta_retCode_t ret = SHELLMATTA_BUSY; + + if(callCnt < 10u) + { + callCnt ++; + shellmatta_printf(handle, "%s - length %u - callCnt %u\r\n", arguments, length, callCnt); + } + else + { + callCnt = 0u; + ret = SHELLMATTA_OK; + } + + return ret; +} +shellmatta_cmd_t busyCommand = {"busy", NULL, NULL, NULL, busy, NULL}; + shellmatta_retCode_t writeFct(const char* data, uint32_t length) { @@ -171,8 +203,6 @@ int main(int argc, char **argv) return f; } - set_blocking (f, 1); - shellmatta_doInit( &instance, &handle, buffer, @@ -189,18 +219,26 @@ int main(int argc, char **argv) shellmatta_addCmd(handle, &emptyCommand); shellmatta_addCmd(handle, &resetCommand); shellmatta_addCmd(handle, &continuousCommand); + shellmatta_addCmd(handle, &busyCommand); while(exitRequest == false) { char c; - + shellmatta_retCode_t ret; int res = 0; res = read (f, &c, 1); fprintf(stdout, "0x%02x \n", c); fflush(stdout); - shellmatta_processData(handle, &c, res); + do + { + ret = shellmatta_processData(handle, &c, res); + if(SHELLMATTA_BUSY == ret) + { + sleep(1); + } + } while(SHELLMATTA_BUSY == ret); } close(f); diff --git a/src/shellmatta_opt.c b/src/shellmatta_opt.c index ad86ddd..5f740c9 100644 --- a/src/shellmatta_opt.c +++ b/src/shellmatta_opt.c @@ -124,7 +124,7 @@ static char peekNextHunk(shellmatta_instance_t *inst) * #SHELLMATTA_ERROR - format error or option unknown */ static shellmatta_retCode_t parseShortOpt( shellmatta_instance_t *inst, - char *optionString, + const char *optionString, char *option, shellmatta_opt_argtype_t *argtype) { @@ -186,7 +186,7 @@ static shellmatta_retCode_t parseShortOpt( shellmatta_instance_t *inst, * #SHELLMATTA_ERROR - format error or option unknown */ static shellmatta_retCode_t parseLongOpt( shellmatta_instance_t *inst, - shellmatta_opt_long_t *longOptions, + const shellmatta_opt_long_t *longOptions, char *option, shellmatta_opt_argtype_t *argtype) { @@ -253,12 +253,12 @@ static shellmatta_retCode_t parseLongOpt( shellmatta_instance_t *inst, * @return errorcode #SHELLMATTA_OK - no error - keep on calling * #SHELLMATTA_ERROR - error occured - e.g. argument missing */ -static shellmatta_retCode_t shellmatta_opt_int( shellmatta_handle_t handle, - char *optionString, - shellmatta_opt_long_t *longOptions, - char *option, - char **argument, - uint32_t *argLen) +static shellmatta_retCode_t shellmatta_opt_int( shellmatta_handle_t handle, + const char *optionString, + const shellmatta_opt_long_t *longOptions, + char *option, + char **argument, + uint32_t *argLen) { shellmatta_retCode_t ret = SHELLMATTA_USE_FAULT; shellmatta_instance_t *inst = (shellmatta_instance_t*)handle; @@ -370,7 +370,7 @@ static shellmatta_retCode_t shellmatta_opt_int( shellmatta_handle_t handle, * #SHELLMATTA_ERROR - error occured - e.g. argument missing */ shellmatta_retCode_t shellmatta_opt( shellmatta_handle_t handle, - char *optionString, + const char *optionString, char *option, char **argument, uint32_t *argLen) @@ -391,11 +391,11 @@ shellmatta_retCode_t shellmatta_opt( shellmatta_handle_t handle, * @param[out] argument pointer to store the argument string to (can be NULL) * @param[out] argLen pointer to store the argument lengh to (can be NULL) */ -shellmatta_retCode_t shellmatta_opt_long( shellmatta_handle_t handle, - shellmatta_opt_long_t *longOptions, - char *option, - char **argument, - uint32_t *argLen) +shellmatta_retCode_t shellmatta_opt_long( shellmatta_handle_t handle, + const shellmatta_opt_long_t *longOptions, + char *option, + char **argument, + uint32_t *argLen) { return shellmatta_opt_int( handle, NULL, diff --git a/src/shellmatta_opt.h b/src/shellmatta_opt.h index df1c554..852f05b 100644 --- a/src/shellmatta_opt.h +++ b/src/shellmatta_opt.h @@ -23,16 +23,16 @@ #include shellmatta_retCode_t shellmatta_opt( shellmatta_handle_t handle, - char *optionString, + const char *optionString, char *option, char **argument, uint32_t *argLen); -shellmatta_retCode_t shellmatta_opt_long( shellmatta_handle_t handle, - shellmatta_opt_long_t *longOptions, - char *option, - char **argument, - uint32_t *argLen); +shellmatta_retCode_t shellmatta_opt_long( shellmatta_handle_t handle, + const shellmatta_opt_long_t *longOptions, + char *option, + char **argument, + uint32_t *argLen); shellmatta_retCode_t shellmatta_opt_init( shellmatta_instance_t *inst, uint32_t argStart);