flxlibs
Unit Tests
No unit tests written.
Errors in Documentation Links
docs/FELIX-assets.md
- [ERROR] https://cernbox.cern.ch/index.php/s/o1u6HLtpS6yC0OZ/download Connection failed. Check network connectivity and firewall settings
- [ERROR] https://cernbox.cern.ch/index.php/s/tUSn0gjehHfvca7/download Connection failed. Check network connectivity and firewall settings
- [ERROR] https://espace.cern.ch/ATLAS-FELIX/Shared%20Documents/Bitfiles_development/FLX712_FULLMODE_24CH_CLKSELECT_GIT_phase2-FLX-1368_PEXInitForEPYC_rm-5.0_1070_201113_11_29.tar.gz Connection failed. Check network connectivity and firewall settings
docs/Integration-tests.md
- [ERROR] http://np04-srv-30:3336/ Connection failed. Check network connectivity and firewall settings
Clang Formatting
| File | Status |
|---|---|
| apps/emu_confgen.cxx | ✅ Already formatted |
| include/flxlibs/AvailableParserOperations.hpp | ❌ Needs formatting |
| include/flxlibs/felixcardcontroller/Nljs.hpp | ❌ Needs formatting |
| include/flxlibs/felixcardcontroller/Structs.hpp | ❌ Needs formatting |
| plugins/FelixCardControllerModule.cpp | ❌ Needs formatting |
| plugins/FelixCardControllerModule.hpp | ❌ Needs formatting |
| plugins/FelixReaderModule.cpp | ❌ Needs formatting |
| plugins/FelixReaderModule.hpp | ❌ Needs formatting |
| src/CardControllerWrapper.cpp | ❌ Needs formatting |
| src/CardControllerWrapper.hpp | ❌ Needs formatting |
| src/CardWrapper.cpp | ❌ Needs formatting |
| src/CardWrapper.hpp | ❌ Needs formatting |
| src/CreateElink.hpp | ❌ Needs formatting |
| src/DefaultParserImpl.cpp | ❌ Needs formatting |
| src/DefaultParserImpl.hpp | ❌ Needs formatting |
| src/ElinkConcept.hpp | ❌ Needs formatting |
| src/ElinkModel.hpp | ❌ Needs formatting |
| src/FelixDefinitions.hpp | ✅ Already formatted |
| src/FelixIssues.hpp | ❌ Needs formatting |
| src/FelixStatistics.hpp | ✅ Already formatted |
| test/apps/test_cardwrapper_app.cxx | ❌ Needs formatting |
| test/apps/test_elink_to_file_app.cxx | ✅ Already formatted |
| test/apps/test_elink_to_heap_app.cxx | ❌ Needs formatting |
| test/apps/test_elinkhandler_app.cxx | ❌ Needs formatting |
| test/apps/test_tp_elinkhandler_app.cxx | ❌ Needs formatting |
apps/emu_confgen.cxx
Warnings
- Line 63: do not use pointer arithmetic
- Line 65: do not use pointer arithmetic
- Line 101: do not use pointer arithmetic
- Line 112: do not use pointer arithmetic
- Line 113: do not use pointer arithmetic
- Line 137: do not use pointer arithmetic
- Line 216: do not use pointer arithmetic
- Line 252: the variable 'arg' is copy-constructed from a const reference but is only used as const reference; consider making it a const reference
- Line 334: do not use array subscript when the index is not an integer constant expression
include/flxlibs/AvailableParserOperations.hpp
Errors
- Line 168: An unsigned integer appears to be used here.
plugins/FelixCardControllerModule.cpp
Errors
- Line 86: An unsigned integer appears to be used here.
- Line 104: An unsigned integer appears to be used here.
Warnings
- Line 46: nested namespaces can be concatenated
- Line 115: loop variable is copied but only used as const reference; consider making it a const reference
- Line 127: loop variable is copied but only used as const reference; consider making it a const reference
- Line 138: loop variable is copied but only used as const reference; consider making it a const reference
- Line 150: loop variable is copied but only used as const reference; consider making it a const reference
plugins/FelixCardControllerModule.hpp
Errors
- Line 9: #ifndef header guard has wrong style, please use: FLXLIBS_PLUGINS_FELIXCARDCONTROLLERMODULE_HPP_
- Line 62: #endif line should be "#endif // FLXLIBS_PLUGINS_FELIXCARDCONTROLLERMODULE_HPP_"
- Line 56: An unsigned integer appears to be used here.
- Line 56: Add #include
plugins/FelixReaderModule.cpp
Errors
- Line 100: An unsigned integer appears to be used here.
- Line 86: Add #include
Warnings
- Line 26: nested namespaces can be concatenated
- Line 36: do not use pointer arithmetic
- Line 54: do not use pointer arithmetic
- Line 66: unused parameter 'timeout'
- Line 83: do not use pointer arithmetic
- Line 83: do not use pointer arithmetic
- Line 84: do not use pointer arithmetic
- Line 99: unused parameter 'timeout'
- Line 123: unused parameter 'timeout'
- Line 137: use auto when initializing with new to avoid duplicating the type name
- Line 141: do not use pointer arithmetic
- Line 142: do not use pointer arithmetic
- Line 146: do not use pointer arithmetic
- Line 160: unused parameter 'timeout'
- Line 170: do not use pointer arithmetic
- Line 171: do not use pointer arithmetic
- Line 175: do not use pointer arithmetic
- Line 189: unused parameter 'timeout'
- Line 206: unused parameter 'timeout'
- Line 214: do not manage memory manually; consider a container or a smart pointer
- Line 218: do not use pointer arithmetic
- Line 218: do not use pointer arithmetic
- Line 219: do not use pointer arithmetic
- Line 232: unused parameter 'timeout'
- Line 236: do not manage memory manually; consider a container or a smart pointer
- Line 250: unused parameter 'timeout'
- Line 50: nested namespaces can be concatenated
- Line 111: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 120: narrowing conversion from 'mapped_type' (aka 'unsigned int') to signed type 'key_type' (aka 'int') is implementation-defined
- Line 131: do not use const_cast to remove const qualifier
- Line 135: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 136: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 186: 'push_back' is called inside a loop; consider pre-allocating the container capacity before the loop
- Line 192: narrowing conversion from 'value_type' (aka 'unsigned int') to signed type 'key_type' (aka 'int') is implementation-defined
- Line 194: narrowing conversion from 'value_type' (aka 'unsigned int') to signed type 'key_type' (aka 'int') is implementation-defined
- Line 194: narrowing conversion from 'value_type' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 194: narrowing conversion from 'value_type' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 195: narrowing conversion from 'value_type' (aka 'unsigned int') to signed type 'key_type' (aka 'int') is implementation-defined
- Line 29: Excessive padding in 'class dunedaq::flxlibs::CardWrapper' (47 padding bytes, where 7 is optimal). Optimal fields order: m_margin_blocks, m_block_threshold, m_poll_time, m_flx_card, m_dma_memory_size, m_virt_addr, m_phys_addr, m_current_addr, m_destination, m_links_enabled, m_card_id_str, m_info_str, m_handle_block_addr, m_card_mutex, m_dma_processor, m_cmem_handle, m_read_index, m_run_marker, m_configured, m_card_id, m_logical_unit, m_dma_id, m_interrupt_mode, m_numa_id, m_run_lock, m_block_addr_handler_available, consider reordering the fields or adding explicit padding members
- Line 52: prefer a lambda to std::bind
- Line 108: initialization of 'm_dma_processor_name' with static storage duration may throw an exception that cannot be caught
- Line 36: function 'createElinkModel' defined in a header file; function definitions in header files can lead to ODR violations
- Line 38: the variable 'datatype' is copy-constructed from a const reference but is only used as const reference; consider making it a const reference
- Line 24: nested namespaces can be concatenated
- Line 56: 'init' overrides a member function but is not marked 'override'
- Line 61: 'conf' overrides a member function but is not marked 'override'
- Line 75: 'start' overrides a member function but is not marked 'override'
- Line 87: 'stop' overrides a member function but is not marked 'override'
- Line 106: 'queue_in_block_address' overrides a member function but is not marked 'override'
- Line 146: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 147: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 197: initialization of 'm_parser_thread_name' with static storage duration may throw an exception that cannot be caught
- Line 204: do not use const_cast to remove const qualifier
plugins/FelixReaderModule.hpp
Errors
- Line 8: #ifndef header guard has wrong style, please use: FLXLIBS_PLUGINS_FELIXREADERMODULE_HPP_
- Line 84: #endif line should be "#endif // FLXLIBS_PLUGINS_FELIXREADERMODULE_HPP_"
src/CardControllerWrapper.cpp
Errors
- Line 40: An unsigned integer appears to be used here.
- Line 89: An unsigned integer appears to be used here.
- Line 110: If an else has a brace on one side, it should have it on both
- Line 200: An unsigned integer appears to be used here.
- Line 229: An unsigned integer appears to be used here.
- Line 40: Add #include
for vector<> - Line 239: Add #include
for move
Warnings
- Line 37: nested namespaces can be concatenated
- Line 80: narrowing conversion from 'u_int' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 209: FIXME: Horrible remapping workaround. Temporary fix only!
- Line 235: FIXME: Horrible remapping workaround. Temporary fix only!
- Line 13: nested namespaces can be concatenated
- Line 17: macro 'MAX_IRQ' used to declare a constant; consider using a 'constexpr' constant
- Line 20: macro 'IRQ_WRAP_AROUND_FROM_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 21: macro 'IRQ_WRAP_AROUND_TO_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 22: macro 'IRQ_DATA_AVAILABLE' used to declare a constant; consider using a 'constexpr' constant
- Line 23: macro 'IRQ_FIFO_FULL_FROM_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 24: macro 'IRQ_PROG_FULL_TO_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 25: macro 'IRQ_FIFO_FULL_TO_HOST' used to declare a constant; consider using a 'constexpr' constant
src/CardControllerWrapper.hpp
Errors
- Line 24: Namespace should be terminated with '// namespace appmodel '
- Line 33: An unsigned integer appears to be used here.
- Line 42: An unsigned integer appears to be used here.
- Line 49: An unsigned integer appears to be used here.
- Line 61: An unsigned integer appears to be used here.
- Line 67: Add #include
for vector<>
src/CardWrapper.cpp
Errors
- Line 36: Add #include
for vector<>
Warnings
- Line 33: nested namespaces can be concatenated
- Line 49: field 'm_handle_block_addr' will be initialized after field 'm_links_enabled'
- Line 159: use auto when initializing with a cast to avoid duplicating the type name
- Line 159: C-style casts are discouraged; use static_cast
- Line 190: variable 'handle' is not initialized
- Line 193: do not use const_cast to remove const qualifier
- Line 276: do not use array subscript when the index is not an integer constant expression
- Line 29: Excessive padding in 'class dunedaq::flxlibs::CardWrapper' (47 padding bytes, where 7 is optimal). Optimal fields order: m_margin_blocks, m_block_threshold, m_poll_time, m_flx_card, m_dma_memory_size, m_virt_addr, m_phys_addr, m_current_addr, m_destination, m_links_enabled, m_card_id_str, m_info_str, m_handle_block_addr, m_card_mutex, m_dma_processor, m_cmem_handle, m_read_index, m_run_marker, m_configured, m_card_id, m_logical_unit, m_dma_id, m_interrupt_mode, m_numa_id, m_run_lock, m_block_addr_handler_available, consider reordering the fields or adding explicit padding members
- Line 52: prefer a lambda to std::bind
- Line 108: initialization of 'm_dma_processor_name' with static storage duration may throw an exception that cannot be caught
- Line 13: nested namespaces can be concatenated
- Line 17: macro 'MAX_IRQ' used to declare a constant; consider using a 'constexpr' constant
- Line 20: macro 'IRQ_WRAP_AROUND_FROM_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 21: macro 'IRQ_WRAP_AROUND_TO_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 22: macro 'IRQ_DATA_AVAILABLE' used to declare a constant; consider using a 'constexpr' constant
- Line 23: macro 'IRQ_FIFO_FULL_FROM_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 24: macro 'IRQ_PROG_FULL_TO_HOST' used to declare a constant; consider using a 'constexpr' constant
- Line 25: macro 'IRQ_FIFO_FULL_TO_HOST' used to declare a constant; consider using a 'constexpr' constant
src/CreateElink.hpp
Errors
- Line 39: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
src/DefaultParserImpl.cpp
Warnings
- Line 18: nested namespaces can be concatenated
src/ElinkModel.hpp
Errors
- Line 172: Add #include
for move
src/FelixIssues.hpp
Errors
- Line 12: Found other header after system header. Should be: FelixIssues.hh, other non-system headers, system headers.
test/apps/test_cardwrapper_app.cxx
Warnings
- Line 52: prefer a lambda to std::bind
- Line 108: initialization of 'm_dma_processor_name' with static storage duration may throw an exception that cannot be caught
- Line 55: do not use const_cast to remove const qualifier
test/apps/test_elink_to_file_app.cxx
Warnings
- Line 26: nested namespaces can be concatenated
- Line 36: do not use pointer arithmetic
- Line 54: do not use pointer arithmetic
- Line 66: unused parameter 'timeout'
- Line 83: do not use pointer arithmetic
- Line 83: do not use pointer arithmetic
- Line 84: do not use pointer arithmetic
- Line 99: unused parameter 'timeout'
- Line 123: unused parameter 'timeout'
- Line 137: use auto when initializing with new to avoid duplicating the type name
- Line 141: do not use pointer arithmetic
- Line 142: do not use pointer arithmetic
- Line 146: do not use pointer arithmetic
- Line 160: unused parameter 'timeout'
- Line 170: do not use pointer arithmetic
- Line 171: do not use pointer arithmetic
- Line 175: do not use pointer arithmetic
- Line 189: unused parameter 'timeout'
- Line 206: unused parameter 'timeout'
- Line 214: do not manage memory manually; consider a container or a smart pointer
- Line 218: do not use pointer arithmetic
- Line 218: do not use pointer arithmetic
- Line 219: do not use pointer arithmetic
- Line 232: unused parameter 'timeout'
- Line 236: do not manage memory manually; consider a container or a smart pointer
- Line 250: unused parameter 'timeout'
- Line 52: prefer a lambda to std::bind
- Line 108: initialization of 'm_dma_processor_name' with static storage duration may throw an exception that cannot be caught
- Line 24: nested namespaces can be concatenated
- Line 56: 'init' overrides a member function but is not marked 'override'
- Line 61: 'conf' overrides a member function but is not marked 'override'
- Line 75: 'start' overrides a member function but is not marked 'override'
- Line 87: 'stop' overrides a member function but is not marked 'override'
- Line 106: 'queue_in_block_address' overrides a member function but is not marked 'override'
- Line 146: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 147: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 197: initialization of 'm_parser_thread_name' with static storage duration may throw an exception that cannot be caught
- Line 204: do not use const_cast to remove const qualifier
- Line 37: do not declare C-style arrays, use std::array<> instead
- Line 58: do not use pointer arithmetic
- Line 59: do not use pointer arithmetic
- Line 63: do not use pointer arithmetic
- Line 65: std::move of the variable 'payload' of the trivially-copyable type 'USER_PAYLOAD_STRUCT' has no effect
- Line 82: do not use const_cast to remove const qualifier
- Line 90: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 93: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 131: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 132: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 133: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 134: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 135: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 136: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 137: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 138: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 139: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 140: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 141: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 141: narrowing conversion from 'unsigned int' to signed type 'int' is implementation-defined
- Line 141: narrowing conversion from 'unsigned int' to signed type 'int' is implementation-defined
- Line 142: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 142: narrowing conversion from 'unsigned int' to signed type 'int' is implementation-defined
- Line 142: narrowing conversion from 'unsigned int' to signed type 'int' is implementation-defined
- Line 143: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 144: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
test/apps/test_elink_to_heap_app.cxx
Warnings
- Line 26: nested namespaces can be concatenated
- Line 36: do not use pointer arithmetic
- Line 54: do not use pointer arithmetic
- Line 66: unused parameter 'timeout'
- Line 83: do not use pointer arithmetic
- Line 83: do not use pointer arithmetic
- Line 84: do not use pointer arithmetic
- Line 99: unused parameter 'timeout'
- Line 123: unused parameter 'timeout'
- Line 137: use auto when initializing with new to avoid duplicating the type name
- Line 141: do not use pointer arithmetic
- Line 142: do not use pointer arithmetic
- Line 146: do not use pointer arithmetic
- Line 160: unused parameter 'timeout'
- Line 170: do not use pointer arithmetic
- Line 171: do not use pointer arithmetic
- Line 175: do not use pointer arithmetic
- Line 189: unused parameter 'timeout'
- Line 206: unused parameter 'timeout'
- Line 214: do not manage memory manually; consider a container or a smart pointer
- Line 218: do not use pointer arithmetic
- Line 218: do not use pointer arithmetic
- Line 219: do not use pointer arithmetic
- Line 232: unused parameter 'timeout'
- Line 236: do not manage memory manually; consider a container or a smart pointer
- Line 250: unused parameter 'timeout'
- Line 52: prefer a lambda to std::bind
- Line 108: initialization of 'm_dma_processor_name' with static storage duration may throw an exception that cannot be caught
- Line 24: nested namespaces can be concatenated
- Line 56: 'init' overrides a member function but is not marked 'override'
- Line 61: 'conf' overrides a member function but is not marked 'override'
- Line 75: 'start' overrides a member function but is not marked 'override'
- Line 87: 'stop' overrides a member function but is not marked 'override'
- Line 106: 'queue_in_block_address' overrides a member function but is not marked 'override'
- Line 146: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 147: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 197: initialization of 'm_parser_thread_name' with static storage duration may throw an exception that cannot be caught
- Line 204: do not use const_cast to remove const qualifier
- Line 57: do not manage memory manually; consider a container or a smart pointer
- Line 61: do not use pointer arithmetic
- Line 61: do not use pointer arithmetic
- Line 62: do not use pointer arithmetic
- Line 79: do not use const_cast to remove const qualifier
- Line 87: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 90: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 121: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 122: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 123: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 124: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 125: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 126: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 126: narrowing conversion from 'unsigned int' to signed type 'int' is implementation-defined
- Line 126: narrowing conversion from 'unsigned int' to signed type 'int' is implementation-defined
- Line 127: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 150: variable 'popped' set but not used
test/apps/test_elinkhandler_app.cxx
Warnings
- Line 26: nested namespaces can be concatenated
- Line 36: do not use pointer arithmetic
- Line 54: do not use pointer arithmetic
- Line 66: unused parameter 'timeout'
- Line 83: do not use pointer arithmetic
- Line 83: do not use pointer arithmetic
- Line 84: do not use pointer arithmetic
- Line 99: unused parameter 'timeout'
- Line 123: unused parameter 'timeout'
- Line 137: use auto when initializing with new to avoid duplicating the type name
- Line 141: do not use pointer arithmetic
- Line 142: do not use pointer arithmetic
- Line 146: do not use pointer arithmetic
- Line 160: unused parameter 'timeout'
- Line 170: do not use pointer arithmetic
- Line 171: do not use pointer arithmetic
- Line 175: do not use pointer arithmetic
- Line 189: unused parameter 'timeout'
- Line 206: unused parameter 'timeout'
- Line 214: do not manage memory manually; consider a container or a smart pointer
- Line 218: do not use pointer arithmetic
- Line 218: do not use pointer arithmetic
- Line 219: do not use pointer arithmetic
- Line 232: unused parameter 'timeout'
- Line 236: do not manage memory manually; consider a container or a smart pointer
- Line 250: unused parameter 'timeout'
- Line 52: prefer a lambda to std::bind
- Line 108: initialization of 'm_dma_processor_name' with static storage duration may throw an exception that cannot be caught
- Line 38: the variable 'datatype' is copy-constructed from a const reference but is only used as const reference; consider making it a const reference
- Line 24: nested namespaces can be concatenated
- Line 56: 'init' overrides a member function but is not marked 'override'
- Line 61: 'conf' overrides a member function but is not marked 'override'
- Line 75: 'start' overrides a member function but is not marked 'override'
- Line 87: 'stop' overrides a member function but is not marked 'override'
- Line 106: 'queue_in_block_address' overrides a member function but is not marked 'override'
- Line 146: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 147: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 197: initialization of 'm_parser_thread_name' with static storage duration may throw an exception that cannot be caught
- Line 204: do not use const_cast to remove const qualifier
- Line 78: do not use const_cast to remove const qualifier
- Line 82: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
- Line 83: narrowing conversion from 'unsigned int' to signed type 'key_type' (aka 'int') is implementation-defined
test/apps/test_tp_elinkhandler_app.cxx
Errors
- Line 140: An unsigned integer appears to be used here.
- Line 141: An unsigned integer appears to be used here.
- Line 153: An unsigned integer appears to be used here.
- Line 169: An unsigned integer appears to be used here.
- Line 267: Small and focused functions are preferred: main() has 162 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 165: Add #include
for move
Warnings
- Line 26: nested namespaces can be concatenated
- Line 36: do not use pointer arithmetic
- Line 54: do not use pointer arithmetic
- Line 66: unused parameter 'timeout'
- Line 83: do not use pointer arithmetic
- Line 83: do not use pointer arithmetic
- Line 84: do not use pointer arithmetic
- Line 99: unused parameter 'timeout'
- Line 123: unused parameter 'timeout'
- Line 137: use auto when initializing with new to avoid duplicating the type name
- Line 141: do not use pointer arithmetic
- Line 142: do not use pointer arithmetic
- Line 146: do not use pointer arithmetic
- Line 160: unused parameter 'timeout'
- Line 170: do not use pointer arithmetic
- Line 171: do not use pointer arithmetic
- Line 175: do not use pointer arithmetic
- Line 189: unused parameter 'timeout'
- Line 206: unused parameter 'timeout'
- Line 214: do not manage memory manually; consider a container or a smart pointer
- Line 218: do not use pointer arithmetic
- Line 218: do not use pointer arithmetic
- Line 219: do not use pointer arithmetic
- Line 232: unused parameter 'timeout'
- Line 236: do not manage memory manually; consider a container or a smart pointer
- Line 250: unused parameter 'timeout'
- Line 52: prefer a lambda to std::bind
- Line 108: initialization of 'm_dma_processor_name' with static storage duration may throw an exception that cannot be caught
- Line 38: the variable 'datatype' is copy-constructed from a const reference but is only used as const reference; consider making it a const reference
- Line 24: nested namespaces can be concatenated
- Line 56: 'init' overrides a member function but is not marked 'override'
- Line 61: 'conf' overrides a member function but is not marked 'override'
- Line 75: 'start' overrides a member function but is not marked 'override'
- Line 87: 'stop' overrides a member function but is not marked 'override'
- Line 106: 'queue_in_block_address' overrides a member function but is not marked 'override'
- Line 146: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 147: narrowing conversion from '::uint64_t' (aka 'unsigned long') to 'double'
- Line 197: initialization of 'm_parser_thread_name' with static storage duration may throw an exception that cannot be caught
- Line 204: do not use const_cast to remove const qualifier
- Line 69: do not use pointer arithmetic
- Line 70: do not use pointer arithmetic
- Line 125: do not use pointer arithmetic
- Line 154: do not manage memory manually; consider a container or a smart pointer
- Line 157: do not manage memory manually; consider a container or a smart pointer
- Line 162: do not use pointer arithmetic
- Line 162: do not use pointer arithmetic
- Line 163: do not use pointer arithmetic
- Line 169: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
- Line 191: do not use const_cast to remove const qualifier