cibmodules
Unit Tests
No unit tests written.
Clang Formatting
| File | Status |
|---|---|
| plugins/CIBModule.cpp | ❌ Needs formatting |
| plugins/CIBModule.hpp | ❌ Needs formatting |
| src/CIBModuleIssues.hpp | ❌ Needs formatting |
| src/cib_utilities.cpp | ❌ Needs formatting |
| unittest/Placeholder_test.cxx | ✅ Already formatted |
plugins/CIBModule.cpp
Errors
- Line 307: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 38: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 46: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 47: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 48: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 49: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 583: An unsigned integer appears to be used here.
- Line 628: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 106: Add #include
for shared_ptr<>
Warnings
- Line 38: macro 'CIB_DUNEDAQ' used to declare a constant; consider using a 'constexpr' constant
- Line 46: macro 'TLVL_ENTER_EXIT_METHODS' used to declare a constant; consider using a 'constexpr' constant
- Line 47: macro 'TLVL_CIB_INFO' used to declare a constant; consider using a 'constexpr' constant
- Line 48: macro 'TLVL_CIB_DEBUG' used to declare a constant; consider using a 'constexpr' constant
- Line 49: macro 'TLVL_CIB_TRACE' used to declare a constant; consider using a 'constexpr' constant
- Line 98: Call to virtual method 'CIBModule::do_stop' during destruction bypasses virtual dispatch
- Line 445: the value returned by this function should not be disregarded; neglecting it may lead to errors
- Line 458: the value returned by this function should not be disregarded; neglecting it may lead to errors
- Line 502: Value stored to 'connection_closed' is never read
- Line 658: the value returned by this function should not be disregarded; neglecting it may lead to errors
- Line 668: the value returned by this function should not be disregarded; neglecting it may lead to errors
- Line 729: do not declare C-style arrays, use std::array<> instead
- Line 730: variable 'rawtime' is not initialized
- Line 872: variable 'total_counts' is not initialized
- Line 873: variable 'number_of_counts' is not initialized
- Line 19: nested namespaces can be concatenated
- Line 13: nested namespaces can be concatenated
plugins/CIBModule.hpp
Errors
- Line 75: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 169: An unsigned integer appears to be used here.
src/CIBModuleIssues.hpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 1: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 67: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 67: #endif line should be "#endif // CIBMODULES_SRC_CIBMODULEISSUES_HPP_"
src/cib_utilities.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 13: An unsigned integer appears to be used here.
- Line 18: An unsigned integer appears to be used here.
- Line 23: An unsigned integer appears to be used here.
- Line 29: An unsigned integer appears to be used here.
- Line 32: An unsigned integer appears to be used here.
- Line 67: An unsigned integer appears to be used here.
- Line 68: An unsigned integer appears to be used here.
- Line 69: An unsigned integer appears to be used here.
- Line 79: An unsigned integer appears to be used here.
Warnings
- Line 19: nested namespaces can be concatenated
- Line 5: nested namespaces can be concatenated
- Line 47: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int32_t' (aka 'int') is implementation-defined
- Line 48: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int32_t' (aka 'int') is implementation-defined
- Line 55: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int32_t' (aka 'int') is implementation-defined
- Line 9: macro 'CIB_DUNEDAQ' used to declare a constant; consider using a 'constexpr' constant
- Line 13: nested namespaces can be concatenated