Skip to content

[pull] master from magma:master #2834

[pull] master from magma:master

[pull] master from magma:master #2834

This check has been archived and is scheduled for deletion. Learn more about checks retention
GitHub Actions / cpplint completed Aug 23, 2023 in 45s

reviewdog [cpplint] report

reported by reviewdog 🐶

Findings (2778)

lte/gateway/c/connection_tracker/src/EventTracker.cpp|66| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/connection_tracker/src/PacketGenerator.hpp|52| Could not find a newline character at the end of the file. [whitespace/ending_newline] [5]
lte/gateway/c/core/common/assertions.h|38| #ifndef header guard has wrong style, please use: LTE_GATEWAY_C_CORE_COMMON_ASSERTIONS_H_ [build/header_guard] [5]
lte/gateway/c/core/common/assertions.h|126| #endif line should be "#endif // LTE_GATEWAY_C_CORE_COMMON_ASSERTIONS_H_" [build/header_guard] [5]
lte/gateway/c/core/common/assertions.h|51| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
lte/gateway/c/core/common/common_defs.h|38| #ifndef header guard has wrong style, please use: LTE_GATEWAY_C_CORE_COMMON_COMMON_DEFS_H_ [build/header_guard] [5]
lte/gateway/c/core/common/common_defs.h|254| #endif line should be "#endif // LTE_GATEWAY_C_CORE_COMMON_COMMON_DEFS_H_" [build/header_guard] [5]
lte/gateway/c/core/common/common_defs.h|49| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|49| Using C-style cast. Use reinterpret_cast<TyPe*>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|52| Extra space before [ [whitespace/braces] [5]
lte/gateway/c/core/common/common_defs.h|52| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|57| Using C-style cast. Use reinterpret_cast<TyPe*>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|58| Using C-style cast. Use reinterpret_cast<TyPe*>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|65| Using C-style cast. Use reinterpret_cast<TyPe*>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|66| Using C-style cast. Use reinterpret_cast<TyPe*>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|110| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
lte/gateway/c/core/common/common_defs.h|135| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|171| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/common/common_defs.h|240| Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
lte/gateway/c/core/common/common_defs.h|242| Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
lte/gateway/c/core/common/common_defs.h|244| Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
lte/gateway/c/core/oai/common/TLVEncoder.c|36| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/common_utility_funs.cpp|70| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.c|366| If you can, use sizeof(hex_array + i * 3) instead of 3 as the 2nd arg to snprintf. [runtime/printf] [3]
lte/gateway/c/core/oai/common/conversions.h|77| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|78| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|79| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|80| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|85| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|86| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|87| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|92| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|93| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|94| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|95| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|96| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|97| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|98| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|99| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|100| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|101| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|102| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|103| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|104| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|105| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|106| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|107| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|161| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|186| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|193| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|200| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|270| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|388| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|404| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/conversions.h|600| Using C-style cast. Use reinterpret_cast<uint8_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/enum_string.c|72| Using C-style cast. Use reinterpret_cast<enum_to_string_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/enum_string.c|73| Using C-style cast. Use reinterpret_cast<enum_to_string_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/enum_string.h|54| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/enum_string.h|57| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/itti_free_defined_msg.c|157| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/itti_free_defined_msg.c|171| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/itti_free_defined_msg.c|176| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/itti_free_defined_msg.c|243| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/itti_free_defined_msg.c|321| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|213| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|235| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|286| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|291| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|612| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|860| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|866| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|914| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/common/log.c|915| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/common/log.c|1047| Using C-style cast. Use reinterpret_cast<log_queue_item_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1061| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1092| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1097| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1188| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1194| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1216| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/common/log.c|1222| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1228| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1259| Using C-style cast. Use reinterpret_cast<log_queue_item_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1292| Using C-style cast. Use reinterpret_cast<log_queue_item_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1330| Using C-style cast. Use reinterpret_cast<log_queue_item_t**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1347| Using C-style cast. Use reinterpret_cast<shared_log_queue_item_t**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1406| Using C-style cast. Use reinterpret_cast<log_queue_item_t**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.c|1423| Using C-style cast. Use reinterpret_cast<shared_log_queue_item_t**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/log.h|350| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/common/log.h|446| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/common/log.h|451| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/common/log.h|456| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/common/log.h|461| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/common/log.h|507| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/common/pid_file.c|118| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/common/queue.h|502| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|515| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|516| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|523| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|534| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|544| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|545| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|554| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|556| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|565| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|569| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|586| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|593| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/queue.h|596| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/redis_utils/redis_client.hpp|22| Found C system header after C++ system header. Should be: redis_client.h, c system, c++ system, other. [build/include_order] [4]
lte/gateway/c/core/oai/common/redis_utils/redis_client.hpp|23| Found C system header after C++ system header. Should be: redis_client.h, c system, c++ system, other. [build/include_order] [4]
lte/gateway/c/core/oai/common/redis_utils/redis_client.hpp|74| Is this a non-const reference? If so, make const or use a pointer: std::string& str_to_serialize [runtime/references] [2]
lte/gateway/c/core/oai/common/redis_utils/redis_client.hpp|82| Is this a non-const reference? If so, make const or use a pointer: google::protobuf::Message& proto_msg [runtime/references] [2]
lte/gateway/c/core/oai/common/redis_utils/redis_client.hpp|103| Is this a non-const reference? If so, make const or use a pointer: orc8r::RedisState& state_out [runtime/references] [2]
lte/gateway/c/core/oai/common/redis_utils/redis_client.hpp|111| Is this a non-const reference? If so, make const or use a pointer: google::protobuf::Message& proto_msg [runtime/references] [2]
lte/gateway/c/core/oai/common/shared_ts_log.c|235| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|283| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|296| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|310| Using C-style cast. Use reinterpret_cast<shared_log_queue_item_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|316| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|322| Using C-style cast. Use reinterpret_cast<shared_log_queue_item_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|328| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|342| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/shared_ts_log.c|360| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|35| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|36| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|37| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|38| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|39| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|40| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|88| Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/common/state_converter.cpp|176| Almost always, snprintf is better than strcpy [runtime/printf] [4]
lte/gateway/c/core/oai/common/tree.h|142| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|172| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|333| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|353| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|498| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|512| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|522| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|543| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/common/tree.h|604| If an else has a brace on one side, it should have it on both [readability/braces] [5]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|30| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|33| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|55| Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|61| Weird number of spaces at line-start. Are you using a 2-space indent? [whitespace/indent] [3]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|77| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|78| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|79| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|80| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|81| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/TrackingAreaIdentity.h|84| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/amf_config.hpp|133| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/include/amf_securityDef.h|37| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/include/mme_app_desc.hpp|31| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/mme_app_messages_types.hpp|129| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
lte/gateway/c/core/oai/include/mme_app_ue_context.hpp|74| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
lte/gateway/c/core/oai/include/mme_config.hpp|449| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/mme_config.hpp|450| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/mme_config.hpp|451| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/mme_config.hpp|452| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/mme_config.hpp|453| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/nas/securityDef.hpp|54| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/include/ngap_messages_types.h|24| Should have a space between // and comment [whitespace/comments] [4]
lte/gateway/c/core/oai/include/proto_map.hpp|19| Found C++ system header after other header. Should be: proto_map.h, c system, c++ system, other. [build/include_order] [4]
lte/gateway/c/core/oai/include/proto_map.hpp|20| Found C system header after other header. Should be: proto_map.h, c system, c++ system, other. [build/include_order] [4]
lte/gateway/c/core/oai/include/s11_messages_types.hpp|1333| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
lte/gateway/c/core/oai/include/s6a_messages_types.hpp|135| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
lte/gateway/c/core/oai/include/s6a_messages_types.hpp|143| Redundant blank line at the end of a code block should be deleted. [whitespace/blank_line] [3]
lte/gateway/c/core/oai/include/spgw_types.hpp|88| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
lte/gateway/c/core/oai/include/state_converter.hpp|38| Found C system header after C++ system header. Should be: state_converter.h, c system, c++ system, other. [build/include_order] [4]
lte/gateway/c/core/oai/include/state_converter.hpp|39| "functional" already included at /home/runner/work/magma/magma/lte/gateway/c/core/oai/include/state_converter.hpp:36 [build/include] [4]
lte/gateway/c/core/oai/include/state_converter.hpp|95| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/state_converter.hpp|98| Using C-style cast. Use reinterpret_cast<NodeType*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/state_converter.hpp|117| Using C-style cast. Use reinterpret_cast<NodeType*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/state_converter.hpp|191| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/state_converter.hpp|194| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/include/state_manager.hpp|29| Found C++ system header after other header. Should be: state_manager.h, c system, c++ system, other. [build/include_order] [4]
lte/gateway/c/core/oai/include/state_manager.hpp|41| Do not use unnamed namespaces in header files. See https://google-styleguide.googlecode.com/svn/trunk/cppguide.xml#Namespaces for more information. [build/namespaces_headers] [4]
lte/gateway/c/core/oai/include/state_manager.hpp|60| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
lte/gateway/c/core/oai/include/state_manager.hpp|113| Using C-style cast. Use reinterpret_cast<UeContextType*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/state_manager.hpp|124| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/include/state_manager.hpp|252| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
lte/gateway/c/core/oai/include/state_manager.hpp|255| Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
lte/gateway/c/core/oai/lib/3gpp/3gpp_23.003.h|143| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/lib/3gpp/3gpp_23.003.h|144| Extra space before ( in function call [whitespace/parens] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_23.003.h|145| Use operator && instead of and [readability/alt_tokens] [2]
lte/gateway/c/core/oai/lib/3gpp/3gpp_23.003.h|158| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/lib/3gpp/3gpp_23.003.h|159| Extra space before ( in function call [whitespace/parens] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_23.003.h|160| Use operator && instead of and [readability/alt_tokens] [2]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008.h|96| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008.h|101| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|81| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|95| Using C-style cast. Use reinterpret_cast<void*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|211| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|820| Using C-style cast. Use reinterpret_cast<packet_filter_identifier_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|828| Using C-style cast. Use reinterpret_cast<packet_filter_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|836| Using C-style cast. Use reinterpret_cast<packet_filter_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|844| Using C-style cast. Use reinterpret_cast<packet_filter_t*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.008_sm_ies.c|1267| Using C-style cast. Use reinterpret_cast<void**>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.301.h|96| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.301.h|103| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.301.h|106| Complex multi-line /.../-style comment found. Lint may give bogus warnings. Consider replacing these with //-style comments, with #if 0...#endif, or with more clearly structured multi-line comments. [readability/multiline_comment] [5]
lte/gateway/c/core/oai/lib/3gpp/3gpp_24.501.h|109| Use int16/int64/etc, rather than the C type long [runtime/int] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|36| Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|104| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|132| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|138| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|148| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|155| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|155| Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|185| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|186| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|190| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|212| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|213| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|245| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|251| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|295| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|301| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|320| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|320| Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|324| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|377| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|414| Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|424| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|441| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|481| Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|528| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|530| Using C-style cast. Use static_cast<size_t>(...) instead [readability/casting] [4]
lte/gateway/c/core/oai/lib/bstr/bstrlib.c|531| Using C-style cast. Use static_cast(...) instead [readability/casting] [4]
... (Too many findings. Dropped some findings)

Filtered Findings (1612)

... (Too many findings. Dropped some findings)

Annotations

Check warning on line 66 in lte/gateway/c/connection_tracker/src/EventTracker.cpp

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/connection_tracker/src/EventTracker.cpp#L66

 Using C-style cast.  Use reinterpret_cast<void*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/connection_tracker/src/EventTracker.cpp:66:  Using C-style cast.  Use reinterpret_cast<void*>(...) instead  [readability/casting] [4]

Check warning on line 52 in lte/gateway/c/connection_tracker/src/PacketGenerator.hpp

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/connection_tracker/src/PacketGenerator.hpp#L52

 Could not find a newline character at the end of the file.  [whitespace/ending_newline] [5]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/connection_tracker/src/PacketGenerator.hpp:52:  Could not find a newline character at the end of the file.  [whitespace/ending_newline] [5]

Check warning on line 38 in lte/gateway/c/core/common/assertions.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/assertions.h#L38

 #ifndef header guard has wrong style, please use: LTE_GATEWAY_C_CORE_COMMON_ASSERTIONS_H_  [build/header_guard] [5]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/assertions.h:38:  #ifndef header guard has wrong style, please use: LTE_GATEWAY_C_CORE_COMMON_ASSERTIONS_H_  [build/header_guard] [5]

Check warning on line 126 in lte/gateway/c/core/common/assertions.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/assertions.h#L126

 #endif line should be "#endif  // LTE_GATEWAY_C_CORE_COMMON_ASSERTIONS_H_"  [build/header_guard] [5]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/assertions.h:126:  #endif line should be "#endif  // LTE_GATEWAY_C_CORE_COMMON_ASSERTIONS_H_"  [build/header_guard] [5]

Check warning on line 51 in lte/gateway/c/core/common/assertions.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/assertions.h#L51

 Missing username in TODO; it should look like "// TODO(my_username): Stuff."  [readability/todo] [2]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/assertions.h:51:  Missing username in TODO; it should look like "// TODO(my_username): Stuff."  [readability/todo] [2]

Check warning on line 38 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L38

 #ifndef header guard has wrong style, please use: LTE_GATEWAY_C_CORE_COMMON_COMMON_DEFS_H_  [build/header_guard] [5]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:38:  #ifndef header guard has wrong style, please use: LTE_GATEWAY_C_CORE_COMMON_COMMON_DEFS_H_  [build/header_guard] [5]

Check warning on line 254 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L254

 #endif line should be "#endif  // LTE_GATEWAY_C_CORE_COMMON_COMMON_DEFS_H_"  [build/header_guard] [5]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:254:  #endif line should be "#endif  // LTE_GATEWAY_C_CORE_COMMON_COMMON_DEFS_H_"  [build/header_guard] [5]

Check warning on line 49 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L49

 Using C-style cast.  Use static_cast<size_t>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:49:  Using C-style cast.  Use static_cast<size_t>(...) instead  [readability/casting] [4]

Check warning on line 49 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L49

 Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:49:  Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]

Check warning on line 52 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L52

 Extra space before [  [whitespace/braces] [5]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:52:  Extra space before [  [whitespace/braces] [5]

Check warning on line 52 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L52

 Using C-style cast.  Use static_cast<size_t>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:52:  Using C-style cast.  Use static_cast<size_t>(...) instead  [readability/casting] [4]

Check warning on line 57 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L57

 Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:57:  Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]

Check warning on line 58 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L58

 Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:58:  Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]

Check warning on line 65 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L65

 Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:65:  Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]

Check warning on line 66 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L66

 Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:66:  Using C-style cast.  Use reinterpret_cast<TyPe*>(...) instead  [readability/casting] [4]

Check warning on line 110 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L110

 Redundant blank line at the end of a code block should be deleted.  [whitespace/blank_line] [3]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:110:  Redundant blank line at the end of a code block should be deleted.  [whitespace/blank_line] [3]

Check warning on line 135 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L135

 Using C-style cast.  Use reinterpret_cast<uint8_t*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:135:  Using C-style cast.  Use reinterpret_cast<uint8_t*>(...) instead  [readability/casting] [4]

Check warning on line 171 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L171

 Using C-style cast.  Use reinterpret_cast<uint8_t*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:171:  Using C-style cast.  Use reinterpret_cast<uint8_t*>(...) instead  [readability/casting] [4]

Check warning on line 240 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L240

 Weird number of spaces at line-start.  Are you using a 2-space indent?  [whitespace/indent] [3]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:240:  Weird number of spaces at line-start.  Are you using a 2-space indent?  [whitespace/indent] [3]

Check warning on line 242 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L242

 Weird number of spaces at line-start.  Are you using a 2-space indent?  [whitespace/indent] [3]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:242:  Weird number of spaces at line-start.  Are you using a 2-space indent?  [whitespace/indent] [3]

Check warning on line 244 in lte/gateway/c/core/common/common_defs.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/common/common_defs.h#L244

 Weird number of spaces at line-start.  Are you using a 2-space indent?  [whitespace/indent] [3]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/common/common_defs.h:244:  Weird number of spaces at line-start.  Are you using a 2-space indent?  [whitespace/indent] [3]

Check warning on line 36 in lte/gateway/c/core/oai/common/TLVEncoder.c

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/oai/common/TLVEncoder.c#L36

 Using C-style cast.  Use reinterpret_cast<void*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/oai/common/TLVEncoder.c:36:  Using C-style cast.  Use reinterpret_cast<void*>(...) instead  [readability/casting] [4]

Check warning on line 70 in lte/gateway/c/core/oai/common/common_utility_funs.cpp

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/oai/common/common_utility_funs.cpp#L70

 Using C-style cast.  Use reinterpret_cast<void**>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/oai/common/common_utility_funs.cpp:70:  Using C-style cast.  Use reinterpret_cast<void**>(...) instead  [readability/casting] [4]

Check warning on line 366 in lte/gateway/c/core/oai/common/conversions.c

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/oai/common/conversions.c#L366

 If you can, use sizeof(hex_array + i * 3) instead of 3 as the 2nd arg to snprintf.  [runtime/printf] [3]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/oai/common/conversions.c:366:  If you can, use sizeof(hex_array + i * 3) instead of 3 as the 2nd arg to snprintf.  [runtime/printf] [3]

Check warning on line 77 in lte/gateway/c/core/oai/common/conversions.h

See this annotation in the file changed.

@github-actions github-actions / cpplint

[cpplint] lte/gateway/c/core/oai/common/conversions.h#L77

 Using C-style cast.  Use reinterpret_cast<uint8_t*>(...) instead  [readability/casting] [4]
Raw output
/home/runner/work/magma/magma/lte/gateway/c/core/oai/common/conversions.h:77:  Using C-style cast.  Use reinterpret_cast<uint8_t*>(...) instead  [readability/casting] [4]