uhallibs

Unit Tests

No unit tests written.

Clang Formatting

File	Status
include/uhallibs/ProtocolAxi4Lite.hpp	❌ Needs formatting
include/uhallibs/ProtocolFlx.hpp	❌ Needs formatting
include/uhallibs/formatters.hpp	❌ Needs formatting
include/uhallibs/ipc/RobustMutex.hpp	❌ Needs formatting
include/uhallibs/ipc/SharedMemObject.hpp	❌ Needs formatting
src/ProtocolAxi4Lite.cpp	❌ Needs formatting
src/ProtocolFlx.cpp	❌ Needs formatting
src/RobustMutex.cpp	❌ Needs formatting
src/formatters.cpp	❌ Needs formatting
test/apps/test-axi4l-if.cxx	❌ Needs formatting
test/apps/test-pcie-axil.cxx	❌ Needs formatting
test/apps/test-uhal-axi4lite.cxx	❌ Needs formatting
test/apps/test-wuip.cxx	❌ Needs formatting
test/apps/toy-tohost-dma-test.cxx	❌ Needs formatting

include/uhallibs/ProtocolAxi4Lite.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 230: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 42: #ifndef header guard has wrong style, please use: UHALLIBS_INCLUDE_UHALLIBS_PROTOCOLAXI4LITE_HPP_
• Line 230: #endif line should be "#endif // UHALLIBS_INCLUDE_UHALLIBS_PROTOCOLAXI4LITE_HPP_"
• Line 56: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 57: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 58: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 59: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 60: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 61: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 66: Namespace should be terminated with '// namespace uhal '
• Line 116: An unsigned integer appears to be used here.
• Line 117: An unsigned integer appears to be used here.
• Line 119: An unsigned integer appears to be used here.
• Line 121: An unsigned integer appears to be used here.
• Line 124: An unsigned integer appears to be used here.
• Line 125: An unsigned integer appears to be used here.
• Line 137: An unsigned integer appears to be used here.
• Line 190: An unsigned integer appears to be used here.
• Line 196: An unsigned integer appears to be used here.
• Line 218: An unsigned integer appears to be used here.
• Line 222: An unsigned integer appears to be used here.
• Line 228: Namespace should be terminated with '// namespace uhallibs '

include/uhallibs/ProtocolFlx.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 255: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 42: #ifndef header guard has wrong style, please use: UHALLIBS_INCLUDE_UHALLIBS_PROTOCOLFLX_HPP_
• Line 255: #endif line should be "#endif // UHALLIBS_INCLUDE_UHALLIBS_PROTOCOLFLX_HPP_"
• Line 56: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 57: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 58: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 60: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 62: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 63: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 68: Namespace should be terminated with '// namespace boost '
• Line 73: Namespace should be terminated with '// namespace uhal '
• Line 104: Namespace should be terminated with '// namespace exception '
• Line 123: An unsigned integer appears to be used here.
• Line 125: An unsigned integer appears to be used here.
• Line 166: Access specifier "public:" appears after one (or both) of "private:" and/or "protected:", not before
• Line 177: Access specifier "private" has already appeared in class Flx
• Line 205: An unsigned integer appears to be used here.
• Line 211: An unsigned integer appears to be used here.
• Line 234: An unsigned integer appears to be used here.
• Line 239: An unsigned integer appears to be used here.
• Line 252: Namespace should be terminated with '// namespace uhallibs '
• Line 242: Add #include for shared_ptr<>

include/uhallibs/formatters.hpp

Errors

• Line 0: The standard copyright message wasn't found.
• Line 32: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 1: #ifndef header guard has wrong style, please use: UHALLIBS_INCLUDE_UHALLIBS_FORMATTERS_HPP_
• Line 32: #endif line should be "#endif // UHALLIBS_INCLUDE_UHALLIBS_FORMATTERS_HPP_"
• Line 13: An unsigned integer appears to be used here.
• Line 14: An unsigned integer appears to be used here.
• Line 14: Single-parameter constructors should be marked explicit.
• Line 17: An unsigned integer appears to be used here.
• Line 23: An unsigned integer appears to be used here.
• Line 26: An unsigned integer appears to be used here.
• Line 30: Namespace should be terminated with '// namespace uhallibs '

include/uhallibs/ipc/RobustMutex.hpp

Errors

• Line 0: The standard copyright message wasn't found.
• Line 48: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 2: #ifndef header guard has wrong style, please use: UHALLIBS_INCLUDE_UHALLIBS_IPC_ROBUSTMUTEX_HPP_
• Line 48: #endif line should be "#endif // UHALLIBS_INCLUDE_UHALLIBS_IPC_ROBUSTMUTEX_HPP_"
• Line 7: Found other header after system header. Should be: RobustMutex.hh, other non-system headers, system headers.
• Line 8: Found other header after system header. Should be: RobustMutex.hh, other non-system headers, system headers.
• Line 29: An unsigned integer appears to be used here.
• Line 41: An unsigned integer appears to be used here.
• Line 46: Namespace should be terminated with '// namespace uhallibs '

include/uhallibs/ipc/SharedMemObject.hpp

Errors

• Line 0: The standard copyright message wasn't found.
• Line 55: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 1: #ifndef header guard has wrong style, please use: UHALLIBS_INCLUDE_UHALLIBS_IPC_SHAREDMEMOBJECT_HPP_
• Line 55: #endif line should be "#endif // UHALLIBS_INCLUDE_UHALLIBS_IPC_SHAREDMEMOBJECT_HPP_"
• Line 16: Single-parameter constructors should be marked explicit.
• Line 53: Namespace should be terminated with '// namespace uhallibs '

src/ProtocolAxi4Lite.cpp

Errors

• Line 0: The standard copyright message wasn't found.
• Line 9: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 11: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 12: Found other header after system header. Should be: ProtocolAxi4Lite.hh, other non-system headers, system headers.
• Line 52: Using C-style cast. Use one of the C++ cast operators instead
• Line 52: An unsigned integer appears to be used here.
• Line 71: Use of NULL #define found; prefer using the nullptr keyword
• Line 74: If an else has a brace on one side, it should have it on both
• Line 76: Use of NULL #define found; prefer using the nullptr keyword
• Line 81: Using C-style cast. Use one of the C++ cast operators instead
• Line 82: Use of NULL #define found; prefer using the nullptr keyword
• Line 92: An unsigned integer appears to be used here.
• Line 93: An unsigned integer appears to be used here.
• Line 101: An unsigned integer appears to be used here.
• Line 102: Use of reinterpret_cast can be dangerous. If there's no way to avoid its use add " // NOLINT" to the end of this line so this script will ignore it next time
• Line 102: An unsigned integer appears to be used here.
• Line 106: An unsigned integer appears to be used here.
• Line 111: An unsigned integer appears to be used here.
• Line 113: Use of reinterpret_cast can be dangerous. If there's no way to avoid its use add " // NOLINT" to the end of this line so this script will ignore it next time
• Line 113: An unsigned integer appears to be used here.
• Line 119: An unsigned integer appears to be used here.
• Line 120: An unsigned integer appears to be used here.
• Line 139: Use of reinterpret_cast can be dangerous. If there's no way to avoid its use add " // NOLINT" to the end of this line so this script will ignore it next time
• Line 139: An unsigned integer appears to be used here.
• Line 164: If an else has a brace on one side, it should have it on both
• Line 254: An unsigned integer appears to be used here.
• Line 263: An unsigned integer appears to be used here.
• Line 282: Use operator && instead of and
• Line 282: Use operator ! instead of not
• Line 290: An unsigned integer appears to be used here.
• Line 296: Use operator || instead of or
• Line 299: Use operator || instead of or
• Line 306: An unsigned integer appears to be used here.
• Line 325: Use operator ! instead of not
• Line 334: Use operator || instead of or
• Line 341: An unsigned integer appears to be used here.
• Line 342: An unsigned integer appears to be used here.
• Line 343: Use of reinterpret_cast can be dangerous. If there's no way to avoid its use add " // NOLINT" to the end of this line so this script will ignore it next time
• Line 343: An unsigned integer appears to be used here.
• Line 362: An unsigned integer appears to be used here.
• Line 364: An unsigned integer appears to be used here.
• Line 370: An unsigned integer appears to be used here.
• Line 399: An unsigned integer appears to be used here.
• Line 402: An unsigned integer appears to be used here.
• Line 406: An unsigned integer appears to be used here.
• Line 409: An unsigned integer appears to be used here.
• Line 421: An unsigned integer appears to be used here.
• Line 428: Use of NULL #define found; prefer using the nullptr keyword
• Line 440: Use of NULL #define found; prefer using the nullptr keyword
• Line 444: Namespace should be terminated with '// namespace uhallibs '
• Line 191: Add #include for string
• Line 402: Add #include for vector<>
• Line 409: Add #include for pair<>
• Line 396: Add #include for shared_ptr<>
• Line 409: Add #include for deque<>
• Line 421: Add #include for min

Warnings

• Line 43: declaration uses identifier '_DUNEDAQ_UHALLIBS_PROTOCOLAXI4LITE_HPP_', which is a reserved identifier
• Line 10: nested namespaces can be concatenated
• Line 7: nested namespaces can be concatenated
• Line 33: use nullptr
• Line 34: do not manage memory manually; use RAII
• Line 52: do not use C-style cast to convert between unrelated types
• Line 52: C-style casts are discouraged; use static_cast
• Line 71: use nullptr
• Line 75: do not manage memory manually; use RAII
• Line 76: use nullptr
• Line 81: do not use C-style cast to convert between unrelated types
• Line 81: C-style casts are discouraged; use reinterpret_cast
• Line 82: use nullptr
• Line 86: throw expression should throw anonymous temporary values instead
• Line 97: do not use pointer arithmetic
• Line 102: do not use reinterpret_cast
• Line 113: do not use reinterpret_cast
• Line 115: do not use pointer arithmetic
• Line 115: Array access (via field 'mBar') results in a null pointer dereference
• Line 115: do not use pointer arithmetic
• Line 134: do not use pointer arithmetic
• Line 134: narrowing conversion from 'unsigned char' to signed type 'char' is implementation-defined
• Line 139: do not use reinterpret_cast
• Line 142: do not use pointer arithmetic
• Line 142: do not use pointer arithmetic
• Line 155: throw expression should throw anonymous temporary values instead
• Line 343: use emplace_back instead of push_back
• Line 343: do not use reinterpret_cast
• Line 344: use emplace_back instead of push_back
• Line 370: do not use C-style cast to convert between unrelated types
• Line 370: C-style casts are discouraged; use reinterpret_cast
• Line 381: throw expression should throw anonymous temporary values instead
• Line 406: do not use C-style cast to convert between unrelated types
• Line 406: C-style casts are discouraged; use reinterpret_cast
• Line 421: C-style casts are discouraged; use static_cast
• Line 428: use nullptr
• Line 437: throw expression should throw anonymous temporary values instead
• Line 440: use nullptr

src/ProtocolFlx.cpp

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 62: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 63: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 64: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 65: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 66: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 67: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 68: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 70: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 72: Found other header after system header. Should be: ProtocolFlx.hh, other non-system headers, system headers.
• Line 84: Use of NULL #define found; prefer using the nullptr keyword
• Line 178: An unsigned integer appears to be used here.
• Line 183: Using C-style cast. Use one of the C++ cast operators instead
• Line 186: An unsigned integer appears to be used here.
• Line 187: An unsigned integer appears to be used here.
• Line 189: An unsigned integer appears to be used here.
• Line 193: An unsigned integer appears to be used here.
• Line 206: An unsigned integer appears to be used here.
• Line 212: Using C-style cast. Use one of the C++ cast operators instead
• Line 220: Use of NULL #define found; prefer using the nullptr keyword
• Line 221: Using C-style cast. Use one of the C++ cast operators instead
• Line 222: Use of NULL #define found; prefer using the nullptr keyword
• Line 237: An unsigned integer appears to be used here.
• Line 243: Using C-style cast. Use one of the C++ cast operators instead
• Line 243: An unsigned integer appears to be used here.
• Line 246: If an else has a brace on one side, it should have it on both
• Line 247: Using C-style cast. Use one of the C++ cast operators instead
• Line 247: An unsigned integer appears to be used here.
• Line 274: If an else has a brace on one side, it should have it on both
• Line 299: Use of NULL #define found; prefer using the nullptr keyword
• Line 314: If an else has a brace on one side, it should have it on both
• Line 369: An unsigned integer appears to be used here.
• Line 378: An unsigned integer appears to be used here.
• Line 398: Use operator && instead of and
• Line 398: Use operator ! instead of not
• Line 402: An unsigned integer appears to be used here.
• Line 404: An unsigned integer appears to be used here.
• Line 441: Use operator ! instead of not
• Line 448: Use operator || instead of or
• Line 455: An unsigned integer appears to be used here.
• Line 456: An unsigned integer appears to be used here.
• Line 457: Use of reinterpret_cast can be dangerous. If there's no way to avoid its use add " // NOLINT" to the end of this line so this script will ignore it next time
• Line 457: An unsigned integer appears to be used here.
• Line 476: An unsigned integer appears to be used here.
• Line 478: An unsigned integer appears to be used here.
• Line 485: An unsigned integer appears to be used here.
• Line 513: An unsigned integer appears to be used here.
• Line 516: An unsigned integer appears to be used here.
• Line 520: An unsigned integer appears to be used here.
• Line 523: An unsigned integer appears to be used here.
• Line 529: An unsigned integer appears to be used here.
• Line 535: An unsigned integer appears to be used here.
• Line 562: Namespace should be terminated with '// namespace uhallibs '
• Line 289: Add #include for string
• Line 516: Add #include for vector<>
• Line 529: Add #include for pair<>
• Line 510: Add #include for shared_ptr<>
• Line 529: Add #include for deque<>

Warnings

• Line 43: declaration uses identifier '_DUNEDAQ_UHALLIBS_PROTOCOLFLX_V0_HPP_', which is a reserved identifier
• Line 10: nested namespaces can be concatenated
• Line 7: nested namespaces can be concatenated
• Line 33: use nullptr
• Line 46: inclusion of deprecated C++ header 'assert.h'; consider using 'cassert' instead
• Line 54: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
• Line 55: inclusion of deprecated C++ header 'string.h'; consider using 'cstring' instead
• Line 83: variable 'reg' is not initialized
• Line 84: use nullptr
• Line 84: do not use pointer arithmetic
• Line 105: throw expression should throw anonymous temporary values instead
• Line 116: throw expression should throw anonymous temporary values instead
• Line 136: throw expression should throw anonymous temporary values instead
• Line 183: use auto when initializing with a cast to avoid duplicating the type name
• Line 183: do not use C-style cast to convert between unrelated types
• Line 183: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
• Line 212: use auto when initializing with a cast to avoid duplicating the type name
• Line 212: do not use C-style cast to convert between unrelated types
• Line 212: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
• Line 220: use nullptr
• Line 221: do not use C-style cast to convert between unrelated types
• Line 221: C-style casts are discouraged; use reinterpret_cast
• Line 222: use nullptr
• Line 225: throw expression should throw anonymous temporary values instead
• Line 232: do not use pointer arithmetic
• Line 241: do not use pointer arithmetic
• Line 243: do not use C-style cast to convert between unrelated types
• Line 243: C-style casts are discouraged; use reinterpret_cast
• Line 247: C-style casts are discouraged; use static_cast
• Line 247: do not use C-style cast to convert between unrelated types
• Line 247: C-style casts are discouraged; use reinterpret_cast
• Line 254: do not manage memory manually; use RAII
• Line 265: throw expression should throw anonymous temporary values instead
• Line 299: use nullptr
• Line 303: use auto when declaring iterators
• Line 404: do not use C-style cast to convert between unrelated types
• Line 404: C-style casts are discouraged; use reinterpret_cast
• Line 417: throw expression should throw anonymous temporary values instead
• Line 423: throw expression should throw anonymous temporary values instead
• Line 457: use emplace_back instead of push_back
• Line 457: do not use reinterpret_cast
• Line 458: use emplace_back instead of push_back
• Line 485: do not use C-style cast to convert between unrelated types
• Line 485: C-style casts are discouraged; use reinterpret_cast
• Line 496: throw expression should throw anonymous temporary values instead
• Line 520: do not use C-style cast to convert between unrelated types
• Line 520: C-style casts are discouraged; use reinterpret_cast
• Line 529: use auto when declaring iterators
• Line 535: C-style casts are discouraged; use static_cast
• Line 551: suspicious exception object created but not thrown; did you mean 'throw ValidationError'?

src/RobustMutex.cpp

Errors

• Line 0: The standard copyright message wasn't found.
• Line 4: Found other header after system header. Should be: RobustMutex.hh, other non-system headers, system headers.
• Line 68: An unsigned integer appears to be used here.
• Line 80: Namespace should be terminated with '// namespace uhallibs '

Warnings

• Line 10: nested namespaces can be concatenated
• Line 7: nested namespaces can be concatenated
• Line 18: throw expression should throw anonymous temporary values instead
• Line 26: throw expression should throw anonymous temporary values instead
• Line 34: throw expression should throw anonymous temporary values instead
• Line 42: throw expression should throw anonymous temporary values instead
• Line 57: throw expression should throw anonymous temporary values instead

src/formatters.cpp

Errors

• Line 0: The standard copyright message wasn't found.
• Line 8: An unsigned integer appears to be used here.
• Line 9: An unsigned integer appears to be used here.
• Line 13: An unsigned integer appears to be used here.
• Line 28: An unsigned integer appears to be used here.
• Line 31: An unsigned integer appears to be used here.
• Line 39: An unsigned integer appears to be used here.
• Line 40: An unsigned integer appears to be used here.
• Line 52: An unsigned integer appears to be used here.
• Line 55: An unsigned integer appears to be used here.
• Line 65: Namespace should be terminated with '// namespace uhallibs '
• Line 40: Add #include for pair<>
• Line 13: Add #include for vector<>

Warnings

• Line 28: do not use pointer arithmetic
• Line 31: C-style casts are discouraged; use static_cast
• Line 52: do not use pointer arithmetic
• Line 55: C-style casts are discouraged; use static_cast

test/apps/test-axi4l-if.cxx

Errors

• Line 0: The standard copyright message wasn't found.
• Line 30: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 21: Found other header after system header. Should be: test-axi4l-if.hh, other non-system headers, system headers.
• Line 22: Found other header after system header. Should be: test-axi4l-if.hh, other non-system headers, system headers.
• Line 23: Found other header after system header. Should be: test-axi4l-if.hh, other non-system headers, system headers.
• Line 24: Found other header after system header. Should be: test-axi4l-if.hh, other non-system headers, system headers.
• Line 27: Found other header after system header. Should be: test-axi4l-if.hh, other non-system headers, system headers.
• Line 34: "cout" should not be used for output in DUNE DAQ software.
• Line 36: An unsigned integer appears to be used here.
• Line 37: An unsigned integer appears to be used here.
• Line 38: An unsigned integer appears to be used here.
• Line 39: An unsigned integer appears to be used here.
• Line 43: An unsigned integer appears to be used here.
• Line 49: "cout" should not be used for output in DUNE DAQ software.
• Line 53: An unsigned integer appears to be used here.
• Line 54: "cout" should not be used for output in DUNE DAQ software.
• Line 64: "cout" should not be used for output in DUNE DAQ software.
• Line 65: "cout" should not be used for output in DUNE DAQ software.
• Line 66: "cout" should not be used for output in DUNE DAQ software.
• Line 67: "cout" should not be used for output in DUNE DAQ software.
• Line 68: "cout" should not be used for output in DUNE DAQ software.
• Line 73: An unsigned integer appears to be used here.
• Line 78: An unsigned integer appears to be used here.
• Line 79: "cout" should not be used for output in DUNE DAQ software.
• Line 82: An unsigned integer appears to be used here.
• Line 88: An unsigned integer appears to be used here.
• Line 89: "cout" should not be used for output in DUNE DAQ software.
• Line 97: "cout" should not be used for output in DUNE DAQ software.
• Line 101: An unsigned integer appears to be used here.
• Line 102: "cout" should not be used for output in DUNE DAQ software.
• Line 112: "cout" should not be used for output in DUNE DAQ software.
• Line 113: "cout" should not be used for output in DUNE DAQ software.
• Line 114: "cout" should not be used for output in DUNE DAQ software.
• Line 115: "cout" should not be used for output in DUNE DAQ software.
• Line 116: "cout" should not be used for output in DUNE DAQ software.
• Line 120: "cout" should not be used for output in DUNE DAQ software.
• Line 123: An unsigned integer appears to be used here.
• Line 126: An unsigned integer appears to be used here.
• Line 127: "cout" should not be used for output in DUNE DAQ software.

Warnings

• Line 43: declaration uses identifier '_DUNEDAQ_UHALLIBS_PROTOCOLAXI4LITE_HPP_', which is a reserved identifier
• Line 10: nested namespaces can be concatenated
• Line 7: nested namespaces can be concatenated
• Line 33: use nullptr
• Line 36: variable 'n_pages' is not initialized
• Line 37: variable 'page_size' is not initialized
• Line 38: variable 'next_write_index' is not initialized
• Line 39: variable 'read_counts' is not initialized

test/apps/test-pcie-axil.cxx

Errors

• Line 0: The standard copyright message wasn't found.
• Line 14: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 15: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
• Line 20: An unsigned integer appears to be used here.
• Line 27: Use of NULL #define found; prefer using the nullptr keyword
• Line 29: Use of NULL #define found; prefer using the nullptr keyword
• Line 32: Using C-style cast. Use one of the C++ cast operators instead
• Line 32: Use of NULL #define found; prefer using the nullptr keyword
• Line 32: An unsigned integer appears to be used here.
• Line 37: An unsigned integer appears to be used here.
• Line 39: An unsigned integer appears to be used here.
• Line 47: An unsigned integer appears to be used here.
• Line 53: An unsigned integer appears to be used here.
• Line 55: An unsigned integer appears to be used here.
• Line 58: "cout" should not be used for output in DUNE DAQ software.
• Line 70: "cout" should not be used for output in DUNE DAQ software.
• Line 86: An unsigned integer appears to be used here.
• Line 87: An unsigned integer appears to be used here.
• Line 88: An unsigned integer appears to be used here.
• Line 89: An unsigned integer appears to be used here.
• Line 91: An unsigned integer appears to be used here.
• Line 93: An unsigned integer appears to be used here.
• Line 94: An unsigned integer appears to be used here.
• Line 96: "cout" should not be used for output in DUNE DAQ software.
• Line 108: "cout" should not be used for output in DUNE DAQ software.
• Line 109: "cout" should not be used for output in DUNE DAQ software.
• Line 110: "cout" should not be used for output in DUNE DAQ software.
• Line 111: "cout" should not be used for output in DUNE DAQ software.
• Line 113: An unsigned integer appears to be used here.
• Line 114: "cout" should not be used for output in DUNE DAQ software.
• Line 122: An unsigned integer appears to be used here.
• Line 123: An unsigned integer appears to be used here.
• Line 125: "cout" should not be used for output in DUNE DAQ software.
• Line 134: "cout" should not be used for output in DUNE DAQ software.
• Line 135: "cout" should not be used for output in DUNE DAQ software.
• Line 136: "cout" should not be used for output in DUNE DAQ software.
• Line 137: "cout" should not be used for output in DUNE DAQ software.
• Line 141: "cout" should not be used for output in DUNE DAQ software.
• Line 143: An unsigned integer appears to be used here.
• Line 144: An unsigned integer appears to be used here.
• Line 146: "cout" should not be used for output in DUNE DAQ software.
• Line 150: An unsigned integer appears to be used here.
• Line 151: An unsigned integer appears to be used here.
• Line 152: "cout" should not be used for output in DUNE DAQ software.

Warnings

• Line 4: inclusion of deprecated C++ header 'errno.h'; consider using 'cerrno' instead
• Line 8: inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
• Line 9: inclusion of deprecated C++ header 'string.h'; consider using 'cstring' instead
• Line 15: function-like macro 'get_syspath_bar_mmap' used; consider a 'constexpr' template function
• Line 22: variable 'fd' is not initialized
• Line 23: variable 'bar' is not initialized
• Line 27: use nullptr
• Line 29: use nullptr
• Line 32: use nullptr
• Line 32: do not use C-style cast to convert between unrelated types
• Line 32: C-style casts are discouraged; use static_cast
• Line 38: do not declare C-style arrays, use std::array<> instead
• Line 39: variable 'bar' is not initialized
• Line 47: do not use pointer arithmetic
• Line 55: variable 'bar' is not initialized
• Line 56: do not declare C-style arrays, use std::array<> instead
• Line 64: do not use pointer arithmetic
• Line 86: variable 'n_pages' is not initialized
• Line 87: variable 'page_size' is not initialized
• Line 88: variable 'next_write_index' is not initialized
• Line 89: variable 'read_counts' is not initialized

test/apps/test-uhal-axi4lite.cxx

Errors

• Line 0: The standard copyright message wasn't found.
• Line 34: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 29: "cout" should not be used for output in DUNE DAQ software.
• Line 33: "cout" should not be used for output in DUNE DAQ software.

test/apps/test-wuip.cxx

Errors

• Line 0: The standard copyright message wasn't found.
• Line 26: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 21: "cout" should not be used for output in DUNE DAQ software.
• Line 25: "cout" should not be used for output in DUNE DAQ software.

test/apps/toy-tohost-dma-test.cxx

Errors

• Line 0: The standard copyright message wasn't found.
• Line 2: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 3: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 4: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 5: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 6: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
• Line 14: Found other header after system header. Should be: toy-tohost-dma-test.hh, other non-system headers, system headers.
• Line 15: Found other header after system header. Should be: toy-tohost-dma-test.hh, other non-system headers, system headers.
• Line 16: Found other header after system header. Should be: toy-tohost-dma-test.hh, other non-system headers, system headers.
• Line 19: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
• Line 20: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
• Line 21: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
• Line 24: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
• Line 27: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
• Line 39: "printf" should not be used for output in DUNE DAQ software.
• Line 40: "printf" should not be used for output in DUNE DAQ software.
• Line 41: "printf" should not be used for output in DUNE DAQ software.
• Line 42: "printf" should not be used for output in DUNE DAQ software.
• Line 43: "printf" should not be used for output in DUNE DAQ software.
• Line 51: Using C-style cast. Use one of the C++ cast operators instead
• Line 57: "printf" should not be used for output in DUNE DAQ software.
• Line 58: "printf" should not be used for output in DUNE DAQ software.
• Line 58: Increment/decrement operator should appear alone on its line unless in a while/for loop head
• Line 60: "printf" should not be used for output in DUNE DAQ software.
• Line 114: "printf" should not be used for output in DUNE DAQ software.
• Line 119: Using C-style cast. Use const_cast(...) instead
• Line 133: "printf" should not be used for output in DUNE DAQ software.
• Line 135: "printf" should not be used for output in DUNE DAQ software.
• Line 142: Using C-style cast. Use one of the C++ cast operators instead
• Line 144: "printf" should not be used for output in DUNE DAQ software.
• Line 145: "printf" should not be used for output in DUNE DAQ software.
• Line 146: "printf" should not be used for output in DUNE DAQ software.
• Line 147: "printf" should not be used for output in DUNE DAQ software.
• Line 148: "printf" should not be used for output in DUNE DAQ software.
• Line 149: "printf" should not be used for output in DUNE DAQ software.
• Line 150: "printf" should not be used for output in DUNE DAQ software.
• Line 151: "printf" should not be used for output in DUNE DAQ software.
• Line 152: "printf" should not be used for output in DUNE DAQ software.
• Line 153: "printf" should not be used for output in DUNE DAQ software.
• Line 154: "printf" should not be used for output in DUNE DAQ software.
• Line 156: "printf" should not be used for output in DUNE DAQ software.
• Line 158: "printf" should not be used for output in DUNE DAQ software.
• Line 163: "printf" should not be used for output in DUNE DAQ software.
• Line 167: "printf" should not be used for output in DUNE DAQ software.
• Line 168: "printf" should not be used for output in DUNE DAQ software.
• Line 169: "printf" should not be used for output in DUNE DAQ software.
• Line 170: "printf" should not be used for output in DUNE DAQ software.
• Line 171: "printf" should not be used for output in DUNE DAQ software.
• Line 191: "cout" should not be used for output in DUNE DAQ software.
• Line 196: Small and focused functions are preferred: main() has 97 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)

Warnings

• Line 9: inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
• Line 11: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
• Line 19: macro 'APPLICATION_NAME' used to declare a constant; consider using a 'constexpr' constant
• Line 32: initialization of 'flxCard' with static storage duration may throw an exception that cannot be caught
• Line 51: use auto when initializing with a cast to avoid duplicating the type name
• Line 51: do not use C-style cast to convert between unrelated types
• Line 51: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
• Line 52: variable 'i' is not initialized
• Line 58: do not use pointer arithmetic
• Line 68: multiple declarations in a single statement reduces readability
• Line 68: variable 'i' is not initialized
• Line 68: variable 'loop' is not initialized
• Line 68: variable 'ret' is not initialized
• Line 68: variable 'opt' is not initialized
• Line 68: variable 'handle' is not initialized
• Line 69: multiple declarations in a single statement reduces readability
• Line 69: variable 'baraddr0' is not initialized
• Line 69: variable 'vaddr' is not initialized
• Line 69: variable 'paddr' is not initialized
• Line 69: variable 'board_id' is not initialized
• Line 69: variable 'bsize' is not initialized
• Line 69: variable 'opt_emu_ena_to_host' is not initialized
• Line 69: variable 'opt_emu_ena_to_frontend' is not initialized
• Line 70: variable 'bar0' is not initialized
• Line 77: 'atoi' used to convert a string to an integer value, but function will not report conversion errors; consider using 'strtol' instead
• Line 96: Value stored to 'opt_emu_ena_to_host' is never read
• Line 97: Value stored to 'opt_emu_ena_to_frontend' is never read
• Line 110: variable 'loop' is not initialized
• Line 111: do not declare C-style arrays, use std::array<> instead
• Line 113: do not use array subscript when the index is not an integer constant expression
• Line 113: narrowing conversion from 'u_long' (aka 'unsigned long') to signed type 'char' is implementation-defined
• Line 116: narrowing conversion from 'CMEM_Error_code_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
• Line 119: narrowing conversion from 'CMEM_Error_code_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
• Line 119: do not use C-style cast to cast away constness
• Line 119: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
• Line 122: narrowing conversion from 'CMEM_Error_code_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
• Line 125: narrowing conversion from 'CMEM_Error_code_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
• Line 142: do not use C-style cast to convert between unrelated types
• Line 142: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
• Line 149: format specifies type 'unsigned long' but the argument has type 'int'
• Line 150: format specifies type 'unsigned long' but the argument has type 'int'
• Line 151: format specifies type 'unsigned long' but the argument has type 'int'
• Line 152: format specifies type 'unsigned long' but the argument has type 'int'
• Line 153: format specifies type 'unsigned long' but the argument has type 'int'
• Line 154: format specifies type 'unsigned long' but the argument has type 'int'
• Line 169: format specifies type 'unsigned long' but the argument has type 'int'
• Line 170: format specifies type 'unsigned long' but the argument has type 'int'
• Line 171: format specifies type 'unsigned long' but the argument has type 'int'
• Line 181: narrowing conversion from 'CMEM_Error_code_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
• Line 183: narrowing conversion from 'CMEM_Error_code_t' (aka 'unsigned int') to signed type 'int' is implementation-defined