search

dstroy0 / InputHandler

Arduino input handler
https://dstroy0.github.io/InputHandler/
GNU General Public License v3.0
1 stars 0 forks source link

User command parameters overhaul #15

Closed dstroy0 closed 2 years ago

github-actions[bot] commented 2 years ago

:scroll: Run clang-format on the following files

:speech_balloon: Output from clang-tidy

src/InputHandler.cpp
src/InputHandler.cpp:20:18: warning: [modernize-use-trailing-return-type] > use a trailing return type for this function

```.cpp char *UserInput::NextArgument() ~~~~~~ ^ auto -> char * ```

src/InputHandler.cpp:20:18: warning: [readability-convert-member-functions-to-static] > method 'NextArgument' can be made static

```.cpp char *UserInput::NextArgument() ^ ```

src/InputHandler.cpp:23:5: warning: [bugprone-branch-clone] > if with identical then and else branches

```.cpp if (data_pointers_index < USER_INPUT_MAX_NUMBER_OF_COMMAND_ARGUMENTS && data_pointers_index < rec_num_arg_strings) ^ /github/workspace/src/InputHandler.cpp:27:5: note: else branch starts here else ^ ```

src/InputHandler.cpp:38:13: warning: [cppcoreguidelines-init-variables] > variable 'cmd_count' is not initialized

```.cpp size_t *cmd_count = &commands_count_; ^ = nullptr ```

src/InputHandler.cpp:39:13: warning: [cppcoreguidelines-init-variables] > variable 'arg_count' is not initialized

```.cpp size_t *arg_count = &max_num_user_defined_args; ^ = nullptr ```

src/InputHandler.cpp:59:17: warning: [modernize-use-trailing-return-type] > use a trailing return type for this function

```.cpp bool UserInput::getToken(char *token_buffer, uint8_t *data, size_t len, size_t *data_index) ~~~~ ^ auto -> bool ```

src/InputHandler.cpp:59:17: warning: [readability-convert-member-functions-to-static] > method 'getToken' can be made static

```.cpp bool UserInput::getToken(char *token_buffer, uint8_t *data, size_t len, size_t *data_index) ^ static ```

src/InputHandler.cpp:63:5: warning: [cppcoreguidelines-avoid-c-arrays] > do not declare C-style arrays, use std::array<> instead

```.cpp bool token_flag[2] = {false, false}; // token state machine, point to a token once ^ ```

src/InputHandler.cpp:63:5: warning: [modernize-avoid-c-arrays] > do not declare C-style arrays, use std::array<> instead

```.cpp ```

src/InputHandler.cpp:64:14: warning: [cppcoreguidelines-init-variables] > variable 'data_length' is not initialized

```.cpp uint32_t data_length = (uint32_t)len; ^ = 0 ```

src/InputHandler.cpp:192:17: warning: [modernize-use-trailing-return-type] > use a trailing return type for this function

```.cpp bool UserInput::validateUserInput(uint8_t arg_type, size_t data_pointers_index) ~~~~ ^ auto -> bool ```

src/InputHandler.cpp:192:17: warning: [readability-convert-member-functions-to-static] > method 'validateUserInput' can be made static

```.cpp bool UserInput::validateUserInput(uint8_t arg_type, size_t data_pointers_index) ^ static ```

src/InputHandler.cpp:194:14: warning: [cppcoreguidelines-init-variables] > variable 'strlen_data' is not initialized

```.cpp uint16_t strlen_data = strlen(data_pointers[data_pointers_index]); ^ = 0 ```

src/InputHandler.cpp:195:10: warning: [cppcoreguidelines-init-variables] > variable 'found_negative_sign' is not initialized

```.cpp bool found_negative_sign = ((char)data_pointers[data_pointers_index][0] == _neg_) ? true : false; ^ = 0 ```

src/InputHandler.cpp:201:40: warning: [readability-simplify-boolean-expr] > redundant boolean literal supplied to boolean operator

```.cpp if (found_negative_sign == true) ~~~~~~~~~~~~~~~~~~~~~~~^~~~ found_negative_sign ```

src/InputHandler.cpp:216:13: warning: [bugprone-branch-clone] > if with identical then and else branches

```.cpp if (found_negative_sign == true) // negative ^ /github/workspace/src/InputHandler.cpp:226:13: note: else branch starts here else // positive ^ ```

src/InputHandler.cpp:216:40: warning: [readability-simplify-boolean-expr] > redundant boolean literal supplied to boolean operator

```.cpp if (found_negative_sign == true) // negative ~~~~~~~~~~~~~~~~~~~~~~~^~~~ found_negative_sign ```

src/InputHandler.cpp:240:21: warning: [cppcoreguidelines-init-variables] > variable 'found_dot' is not initialized

```.cpp uint8_t found_dot = 0; ^ = 0 ```

src/InputHandler.cpp:241:21: warning: [cppcoreguidelines-init-variables] > variable 'num_digits' is not initialized

```.cpp uint8_t num_digits = 0; ^ = 0 ```

src/InputHandler.cpp:242:40: warning: [readability-simplify-boolean-expr] > redundant boolean literal supplied to boolean operator

```.cpp if (found_negative_sign == true) // negative ~~~~~~~~~~~~~~~~~~~~~~~^~~~ found_negative_sign ```

src/InputHandler.cpp:244:25: warning: [cppcoreguidelines-init-variables] > variable 'not_digits' is not initialized

```.cpp uint8_t not_digits = 0; ^ = 0 ```

src/InputHandler.cpp:404:12: warning: [cppcoreguidelines-init-variables] > variable 'data_index' is not initialized

```.cpp size_t data_index = 0; // data iterator ^ = 0 ```

src/InputHandler.cpp:409:25: warning: [cppcoreguidelines-init-variables] > variable 'cmd' is not initialized

```.cpp CommandConstructor* cmd; // command parameters pointer ^ = nullptr ```

src/InputHandler.cpp:419:9: warning: [cppcoreguidelines-avoid-c-arrays] > do not declare C-style arrays, use std::array<> instead

```.cpp bool input_type_match_flag[USER_INPUT_MAX_NUMBER_OF_COMMAND_ARGUMENTS] = {false}; ^ ```

src/InputHandler.cpp:419:9: warning: [modernize-avoid-c-arrays] > do not declare C-style arrays, use std::array<> instead

```.cpp ```

src/InputHandler.cpp:491:40: warning: [readability-simplify-boolean-expr] > redundant boolean literal supplied to boolean operator

```.cpp if (command_matched == true) ~~~~~~~~~~~~~~~~~~~^~~~ command_matched ```

src/InputHandler.cpp:493:30: warning: [cppcoreguidelines-init-variables] > variable 'err_n_args' is not initialized

```.cpp uint16_t err_n_args = (opt.num_args > rec_num_arg_strings) ? rec_num_arg_strings : opt.num_args; ^ = 0 ```

src/InputHandler.cpp:521:63: warning: [readability-simplify-boolean-expr] > redundant boolean literal supplied to boolean operator

```.cpp if (command_matched && all_arguments_valid == false) ~~~~~~~~~~~~~~~~~~~~~~~^~~~~ !all_arguments_valid ```

src/InputHandler.cpp:523:30: warning: [cppcoreguidelines-init-variables] > variable 'err_n_args' is not initialized

```.cpp uint16_t err_n_args = (opt.num_args > rec_num_arg_strings) ? rec_num_arg_strings : opt.num_args; ^ = 0 ```

src/InputHandler.cpp:564:36: warning: [readability-simplify-boolean-expr] > redundant boolean literal supplied to boolean operator

```.cpp if (stream_buffer_allocated == false) ~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~ !stream_buffer_allocated ```

src/InputHandler.cpp:580:11: warning: [cppcoreguidelines-init-variables] > variable 'rc' is not initialized

```.cpp char *rc = (char *)stream_data; ^ = nullptr ```

src/InputHandler.cpp:595:28: warning: [readability-simplify-boolean-expr] > redundant boolean literal supplied to boolean operator

```.cpp if (new_stream_data == true) ~~~~~~~~~~~~~~~~~~~^~~~ new_stream_data ```

src/InputHandler.cpp:609:29: warning: [cppcoreguidelines-init-variables] > variable 'cmd' is not initialized

```.cpp CommandConstructor *cmd; ^ = nullptr ```

src/InputHandler.cpp:614:17: warning: [cppcoreguidelines-init-variables] > variable 'i' is not initialized

```.cpp uint8_t i = 1; ^ = 0 ```

src/InputHandler.cpp:626:17: warning: [readability-convert-member-functions-to-static] > method 'escapeCharactersSoTheyPrint' can be made static

```.cpp void UserInput::escapeCharactersSoTheyPrint(const char *input, char *output) ^ ```

src/InputHandler.cpp:628:14: warning: [cppcoreguidelines-init-variables] > variable 'len' is not initialized

```.cpp uint16_t len = strlen(input); ^ = 0 ```

src/InputHandler.cpp:665:17: warning: [modernize-use-trailing-return-type] > use a trailing return type for this function

```.cpp char UserInput::combineControlCharacters(char input) ~~~~ ^ auto -> char ```

src/InputHandler.cpp:665:17: warning: [readability-convert-member-functions-to-static] > method 'combineControlCharacters' can be made static

```.cpp char UserInput::combineControlCharacters(char input) ^ ```

src/InputHandler.cpp:694:9: warning: [cppcoreguidelines-avoid-c-arrays] > do not declare C-style arrays, use std::array<> instead

```.cpp char temp_settings[3][13] = {'\0'}; ^ ```

src/InputHandler.cpp:694:9: warning: [modernize-avoid-c-arrays] > do not declare C-style arrays, use std::array<> instead

```.cpp ```

src/InputHandler.cpp:694:31: warning: [cppcoreguidelines-avoid-magic-numbers] > 13 is a magic number; consider replacing it with a named constant

```.cpp char temp_settings[3][13] = {'\0'}; ^ ```

src/InputHandler.cpp:694:31: warning: [readability-magic-numbers] > 13 is a magic number; consider replacing it with a named constant

```.cpp ```

src/InputHandler.cpp:695:59: warning: [cppcoreguidelines-pro-bounds-array-to-pointer-decay] > do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead

```.cpp inputprocess->escapeCharactersSoTheyPrint(_term_, temp_settings[0]); ^ ```

src/InputHandler.cpp:696:60: warning: [cppcoreguidelines-pro-bounds-array-to-pointer-decay] > do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead

```.cpp inputprocess->escapeCharactersSoTheyPrint(_delim_, temp_settings[1]); ^ ```

src/InputHandler.cpp:697:66: warning: [cppcoreguidelines-pro-bounds-array-to-pointer-decay] > do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead

```.cpp inputprocess->escapeCharactersSoTheyPrint(_c_str_delim_, temp_settings[2]); ^ ```

src/InputHandler.cpp:716:17: warning: [modernize-use-trailing-return-type] > use a trailing return type for this function

```.cpp bool UserInput::OutputIsAvailable() ~~~~ ^ auto -> bool ```

src/InputHandler.cpp:716:17: warning: [readability-make-member-function-const] > method 'OutputIsAvailable' can be made const

```.cpp bool UserInput::OutputIsAvailable() ^ const ```

src/InputHandler.cpp:721:17: warning: [modernize-use-trailing-return-type] > use a trailing return type for this function

```.cpp bool UserInput::OutputIsEnabled() ~~~~ ^ auto -> bool ```

src/InputHandler.cpp:721:17: warning: [readability-make-member-function-const] > method 'OutputIsEnabled' can be made const

```.cpp bool UserInput::OutputIsEnabled() ^ const ```

src/InputHandler.cpp:749:20: warning: [modernize-use-trailing-return-type] > use a trailing return type for this function

```.cpp uint8_t UserInput::getArgType(CommandParameters &opt, size_t index) ^ ```

src/config/InputHandler_config.h:21:10: error: [clang-diagnostic-error] > 'Arduino.h' file not found

```.h #include ^ ```

src/InputHandler.h
src/config/InputHandler_config.h:21:10: error: [clang-diagnostic-error] > 'Arduino.h' file not found

```.h #include ^ ```

src/config/InputHandler_config.h
src/config/InputHandler_config.h:21:10: error: [clang-diagnostic-error] > 'Arduino.h' file not found

```.h #include ^ ```

src/config/InputHandler_config.h:56:24: warning: [bugprone-macro-parentheses] > macro replacement list should be enclosed in parentheses

```.h #define UI_DEREFERENCE & ^ () ```