Skip to content

refactor of all zencode internals for reentrancy #631

refactor of all zencode internals for reentrancy

refactor of all zencode internals for reentrancy #631

GitHub Actions / cpplint completed Oct 2, 2023 in 0s

reviewdog [cpplint] report

reported by reviewdog 🐶

Findings (2)

src/zen_error.c|63| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/zen_error.h|31| Lines should be <= 120 characters long [whitespace/line_length] [2]

Filtered Findings (279)

src/api_hash.c|50| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/api_hash.c|52| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/api_hash.c|56| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/api_hash.c|58| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/api_hash.c|62| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/api_hash.c|116| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/api_hash.c|119| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/api_hash.c|126| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/api_hash.c|129| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/base58.c|45| Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size. [runtime/arrays] [1]
src/base58.c|52| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|55| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|59| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|63| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|86| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|94| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|101| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|111| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|121| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|124| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|126| Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size. [runtime/arrays] [1]
src/base58.c|128| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|142| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|143| Empty loop bodies should use {} or continue [whitespace/empty_loop_body] [5]
src/base58.c|144| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|150| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/base58.c|157| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/bip39_english.h|| No #ifndef header guard found, suggested CPP variable is: SRC_BIP39_ENGLISH_H_ [build/header_guard] [5]
src/bitop.h|29| #ifndef header guard has wrong style, please use: SRC_BITOP_H_ [build/header_guard] [5]
src/bitop.h|49| #endif line should be "#endif // SRC_BITOP_H_" [build/header_guard] [5]
src/cli-zenroom.c|49| "sys/types.h" already included at src/cli-zenroom.c:34 [build/include] [4]
src/cli-zenroom.c|72| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/cli-zenroom.c|132| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/cli-zenroom.c|209| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/cli-zenroom.c|384| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/cli-zenroom.c|424| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/cli-zenroom.c|432| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/cli-zenroom.c|470| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/cortex_m.c|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/cortex_m.c|7| Include the directory when naming header files [build/include_subdir] [4]
src/cortex_m.c|12| "stdbool.h" already included at src/cortex_m.c:6 [build/include] [4]
src/cortex_m.c|123| Line contains only semicolon. If this should be an empty statement, use {} instead. [whitespace/semicolon] [5]
src/cortex_m.c|152| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/cortex_m.c|191| Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
src/cortex_m.c|195| Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
src/cortex_m.c|206| Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
src/cortex_m.c|211| Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
src/cortex_m.c|220| Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
src/cortex_m.c|227| Consider using strtok_r(...) instead of strtok(...) for improved thread safety. [runtime/threadsafe_fn] [2]
src/encoding.c|27| Include the directory when naming header files [build/include_subdir] [4]
src/encoding.c|28| Found C system header after other header. Should be: encoding.h, c system, c++ system, other. [build/include_order] [4]
src/encoding.c|52| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/encoding.c|118| Empty loop bodies should use {} or continue [whitespace/empty_loop_body] [5]
src/encoding.c|304| Missing space after ; [whitespace/semicolon] [3]
src/encoding.c|333| Almost always, snprintf is better than strcpy [runtime/printf] [4]
src/encoding.c|420| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/encoding.c|21| src/encoding.c should include its header file src/encoding.h [build/include] [5]
src/encoding.h|21| #ifndef header guard has wrong style, please use: SRC_ENCODING_H_ [build/header_guard] [5]
src/encoding.h|46| #endif line should be "#endif // SRC_ENCODING_H_" [build/header_guard] [5]
src/erlang.c|68| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/lua_functions.c|100| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/lua_functions.h|| No #ifndef header guard found, suggested CPP variable is: SRC_LUA_FUNCTIONS_H_ [build/header_guard] [5]
src/lua_modules.c|184| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/mutt_sprintf.c|95| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|105| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|107| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|110| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|166| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|171| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|173| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|178| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|180| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|227| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|272| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|275| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|287| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|289| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|291| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|299| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|303| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|313| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|317| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|349| Empty loop bodies should use {} or continue [whitespace/empty_loop_body] [5]
src/mutt_sprintf.c|356| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|360| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/mutt_sprintf.c|361| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/mutt_sprintf.c|364| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|365| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|366| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/mutt_sprintf.c|401| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|403| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|406| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|422| Empty loop bodies should use {} or continue [whitespace/empty_loop_body] [5]
src/mutt_sprintf.c|424| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|426| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|428| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|456| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|459| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|461| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|473| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|493| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|508| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|520| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|522| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.c|544| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/mutt_sprintf.h|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/mutt_sprintf.h|5| #ifndef header guard has wrong style, please use: SRC_MUTT_SPRINTF_H_ [build/header_guard] [5]
src/mutt_sprintf.h|14| #endif line should be "#endif // SRC_MUTT_SPRINTF_H_" [build/header_guard] [5]
src/randombytes.c|24| Include the directory when naming header files [build/include_subdir] [4]
src/randombytes.c|72| "stdlib.h" already included at src/randombytes.c:65 [build/include] [4]
src/randombytes.c|90| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/randombytes.c|91| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/randombytes.c|92| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/randombytes.c|98| Missing space after ; [whitespace/semicolon] [3]
src/randombytes.h|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/randombytes.h|1| #ifndef header guard has wrong style, please use: SRC_RANDOMBYTES_H_ [build/header_guard] [5]
src/randombytes.h|21| #endif line should be "#endif // SRC_RANDOMBYTES_H_" [build/header_guard] [5]
src/repl.c|21| src/repl.c should include its header file src/repl.h [build/include] [5]
src/repl.h|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/repl.h|1| #ifndef header guard has wrong style, please use: SRC_REPL_H_ [build/header_guard] [5]
src/repl.h|10| #endif line should be "#endif // SRC_REPL_H_" [build/header_guard] [5]
src/rmd160.c|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/rmd160.c|43| Include the directory when naming header files [build/include_subdir] [4]
src/rmd160.c|86| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/rmd160.c|178| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/rmd160.c|213| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/rmd160.h|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/rmd160.h|36| #ifndef header guard has wrong style, please use: SRC_RMD160_H_ [build/header_guard] [5]
src/rmd160.h|148| #endif line should be "#endif // SRC_RMD160_H_" [build/header_guard] [5]
src/rmd160.h|67| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/rmd160.h|68| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/rmd160.h|70| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/rmd160.h|72| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/segwit_addr.c|26| Include the directory when naming header files [build/include_subdir] [4]
src/segwit_addr.c|156| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/segwit_addr.h|22| #ifndef header guard has wrong style, please use: SRC_SEGWIT_ADDR_H_ [build/header_guard] [5]
src/segwit_addr.h|113| #endif line should be "#endif // SRC_SEGWIT_ADDR_H_" [build/header_guard] [5]
src/stb_c_lexer.h|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/stb_c_lexer.h|| No #ifndef header guard found, suggested CPP variable is: SRC_STB_C_LEXER_H_ [build/header_guard] [5]
src/stb_c_lexer.h|74| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|75| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|81| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|118| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/stb_c_lexer.h|120| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/stb_c_lexer.h|135| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|146| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|148| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|186| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/stb_c_lexer.h|283| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/stb_c_lexer.h|298| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|351| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/stb_c_lexer.h|358| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
src/stb_c_lexer.h|607| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/stb_c_lexer.h|614| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/stb_c_lexer.h|650| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|713| Empty loop bodies should use {} or continue [whitespace/empty_loop_body] [5]
src/stb_c_lexer.h|712| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/stb_c_lexer.h|725| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/stb_c_lexer.h|776| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/stb_c_lexer.h|831| "stdlib.h" already included at src/stb_c_lexer.h:231 [build/include] [4]
src/stb_c_lexer.h|861| Add #include for string [build/include_what_you_use] [4]
src/test.c|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/test.c|2| Include the directory when naming header files [build/include_subdir] [4]
src/test.c|32| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/test.c|38| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/test.c|43| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/zen_big.c|47| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/zen_big.c|133| Almost always, snprintf is better than strcpy [runtime/printf] [4]
src/zen_big.c|146| Almost always, snprintf is better than strcpy [runtime/printf] [4]
src/zen_big.c|317| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_big.c|420| Empty loop bodies should use {} or continue [whitespace/empty_loop_body] [5]
src/zen_big.c|525| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/zen_big.c|541| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_big.c|997| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_big.c|1399| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_big.c|1484| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_big.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_BIG_H_ [build/header_guard] [5]
src/zen_big.h|58| #endif line should be "#endif // SRC_ZEN_BIG_H_" [build/header_guard] [5]
src/zen_big.h|40| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/zen_ecdh.c|453| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_ecdh.c|460| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_ecdh.c|691| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecdh.c|765| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecdh.c|827| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecdh.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_ECDH_H_ [build/header_guard] [5]
src/zen_ecdh.h|61| #endif line should be "#endif // SRC_ZEN_ECDH_H_" [build/header_guard] [5]
src/zen_ecp.c|559| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp.c|808| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp.c|812| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_ecp.c|911| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_ecp.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_ECP_H_ [build/header_guard] [5]
src/zen_ecp.h|59| #endif line should be "#endif // SRC_ZEN_ECP_H_" [build/header_guard] [5]
src/zen_ecp.h|30| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp.h|45| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp2.c|135| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp2.c|450| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp2.c|743| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zen_ecp2.c|814| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp2.c|822| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp2.c|823| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_ecp2.c|867| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_error.h|23| #ifndef header guard has wrong style, please use: SRC_ZEN_ERROR_H_ [build/header_guard] [5]
src/zen_error.h|100| #endif line should be "#endif // SRC_ZEN_ERROR_H_" [build/header_guard] [5]
src/zen_error.h|86| Lines should be <= 120 characters long [whitespace/line_length] [2]
src/zen_float.c|42| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_float.c|50| If you can, use sizeof(dest) instead of 1024 as the 2nd arg to snprintf. [runtime/printf] [3]
src/zen_float.c|81| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/zen_float.c|118| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/zen_float.c|264| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_float.c|302| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_float.c|386| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_float.c|403| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_float.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_FLOAT_H_ [build/header_guard] [5]
src/zen_float.h|34| #endif line should be "#endif // SRC_ZEN_FLOAT_H_" [build/header_guard] [5]
src/zen_fp12.c|43| Almost always, snprintf is better than strcpy [runtime/printf] [4]
src/zen_fp12.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_FP12_H_ [build/header_guard] [5]
src/zen_fp12.h|40| #endif line should be "#endif // SRC_ZEN_FP12_H_" [build/header_guard] [5]
src/zen_hash.c|89| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_hash.c|203| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.c|204| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.c|205| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.c|206| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.c|207| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.c|208| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.c|209| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.c|549| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_hash.c|626| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_hash.c|634| Missing space after ; [whitespace/semicolon] [3]
src/zen_hash.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_HASH_H_ [build/header_guard] [5]
src/zen_hash.h|63| #endif line should be "#endif // SRC_ZEN_HASH_H_" [build/header_guard] [5]
src/zen_io.c|122| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_io.c|244| Missing space after ; [whitespace/semicolon] [3]
src/zen_memory.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_MEMORY_H_ [build/header_guard] [5]
src/zen_memory.h|35| #endif line should be "#endif // SRC_ZEN_MEMORY_H_" [build/header_guard] [5]
src/zen_octet.c|544| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_octet.c|547| Missing space after ; [whitespace/semicolon] [3]
src/zen_octet.c|550| Missing space after ; [whitespace/semicolon] [3]
src/zen_octet.c|574| Missing space after ; [whitespace/semicolon] [3]
src/zen_octet.c|657| Missing space after ; [whitespace/semicolon] [3]
src/zen_octet.c|1104| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_octet.c|1232| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_octet.c|1272| Missing space after ; [whitespace/semicolon] [3]
src/zen_octet.c|1489| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/zen_octet.c|1577| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/zen_octet.c|1617| Missing space after ; [whitespace/semicolon] [3]
src/zen_octet.c|1697| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_octet.c|1804| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_octet.c|1805| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
src/zen_octet.c|1696| Add #include for min [build/include_what_you_use] [4]
src/zen_octet.h|21| #ifndef header guard has wrong style, please use: SRC_ZEN_OCTET_H_ [build/header_guard] [5]
src/zen_octet.h|49| #endif line should be "#endif // SRC_ZEN_OCTET_H_" [build/header_guard] [5]
src/zen_parse.c|39| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_parse.c|43| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/zen_parse.c|44| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/zen_parse.c|62| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/zen_parse.c|63| Use int16/int64/etc, rather than the C type short [runtime/int] [4]
src/zen_parse.c|64| Empty loop bodies should use {} or continue [whitespace/empty_loop_body] [5]
src/zen_random.c|63| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zen_random.c|147| Missing space after ; [whitespace/semicolon] [3]
src/zen_random.c|150| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/zencode-exec.c|94| Almost always, snprintf is better than strcat [runtime/printf] [4]
src/zenroom.c|91| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zenroom.c|156| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zenroom.c|334| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
src/zenroom.c|460| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zenroom.c|475| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zenroom.c|492| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zenroom.c|516| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zenroom.h|21| #ifndef header guard has wrong style, please use: SRC_ZENROOM_H_ [build/header_guard] [5]
src/zenroom.h|164| #endif line should be "#endif // SRC_ZENROOM_H_" [build/header_guard] [5]
src/zenroom_jni.c|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/zenroom_jni.c|1| Include the directory when naming header files [build/include_subdir] [4]
src/zenroom_jni.c|2| Include the directory when naming header files [build/include_subdir] [4]
src/zenroom_jni.c|14| Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
src/zenroom_jni.c|43| Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
src/zenroom_jni.h|| No copyright message found. You should have a line: "Copyright [year] " [legal/copyright] [5]
src/zenroom_jni.h|5| #ifndef header guard has wrong style, please use: SRC_ZENROOM_JNI_H_ [build/header_guard] [5]
src/zenroom_jni.h|21| #endif line should be "#endif // SRC_ZENROOM_JNI_H_" [build/header_guard] [5]

Annotations

Check warning on line 63 in src/zen_error.c

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] src/zen_error.c#L63

 Lines should be <= 120 characters long  [whitespace/line_length] [2]
Raw output
src/zen_error.c:63:  Lines should be <= 120 characters long  [whitespace/line_length] [2]

Check warning on line 31 in src/zen_error.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] src/zen_error.h#L31

 Lines should be <= 120 characters long  [whitespace/line_length] [2]
Raw output
src/zen_error.h:31:  Lines should be <= 120 characters long  [whitespace/line_length] [2]