ers
Unit Tests
No unit tests written.
Clang Formatting
| File | Status |
|---|---|
| apps/ers_config.cxx | ❌ Needs formatting |
| include/ers/AnyIssue.hpp | ❌ Needs formatting |
| include/ers/Assertion.hpp | ❌ Needs formatting |
| include/ers/Configuration.hpp | ❌ Needs formatting |
| include/ers/Context.hpp | ❌ Needs formatting |
| include/ers/InputStream.hpp | ❌ Needs formatting |
| include/ers/Issue.hpp | ❌ Needs formatting |
| include/ers/IssueCatcherHandler.hpp | ❌ Needs formatting |
| include/ers/IssueFactory.hpp | ❌ Needs formatting |
| include/ers/IssueReceiver.hpp | ❌ Needs formatting |
| include/ers/LocalContext.hpp | ❌ Needs formatting |
| include/ers/LocalStream.hpp | ❌ Needs formatting |
| include/ers/OutputStream.hpp | ❌ Needs formatting |
| include/ers/RemoteContext.hpp | ❌ Needs formatting |
| include/ers/SampleIssues.hpp | ❌ Needs formatting |
| include/ers/Schema.hpp | ❌ Needs formatting |
| include/ers/Severity.hpp | ❌ Needs formatting |
| include/ers/StandardStreamOutput.hpp | ❌ Needs formatting |
| include/ers/StreamFactory.hpp | ❌ Needs formatting |
| include/ers/StreamManager.hpp | ❌ Needs formatting |
| include/ers/ers.hpp | ❌ Needs formatting |
| include/ers/internal/AbortStream.hpp | ❌ Needs formatting |
| include/ers/internal/ExitStream.hpp | ❌ Needs formatting |
| include/ers/internal/FilterStream.hpp | ❌ Needs formatting |
| include/ers/internal/FormattedStandardStream.hpp | ❌ Needs formatting |
| include/ers/internal/FormattedStandardStream.hxx | ❌ Needs formatting |
| include/ers/internal/GlobalLockStream.hpp | ❌ Needs formatting |
| include/ers/internal/IssueDeclarationMacro.hpp | ❌ Needs formatting |
| include/ers/internal/LockStream.hpp | ❌ Needs formatting |
| include/ers/internal/NullStream.hpp | ❌ Needs formatting |
| include/ers/internal/PluginManager.hpp | ❌ Needs formatting |
| include/ers/internal/RFilterStream.hpp | ❌ Needs formatting |
| include/ers/internal/SingletonCreator.hpp | ❌ Needs formatting |
| include/ers/internal/StandardStream.hpp | ❌ Needs formatting |
| include/ers/internal/ThrottleStream.hpp | ❌ Needs formatting |
| include/ers/internal/ThrowStream.hpp | ❌ Needs formatting |
| include/ers/internal/Util.hpp | ❌ Needs formatting |
| include/ers/internal/macro.hpp | ❌ Needs formatting |
| plugins/AbortStream.cpp | ❌ Needs formatting |
| plugins/ExitStream.cpp | ❌ Needs formatting |
| plugins/FilterStream.cpp | ❌ Needs formatting |
| plugins/GlobalLockStream.cpp | ❌ Needs formatting |
| plugins/LockStream.cpp | ❌ Needs formatting |
| plugins/NullStream.cpp | ❌ Needs formatting |
| plugins/RFilterStream.cpp | ❌ Needs formatting |
| plugins/StandardStream.cpp | ❌ Needs formatting |
| plugins/ThrottleStream.cpp | ❌ Needs formatting |
| plugins/ThrowStream.cpp | ❌ Needs formatting |
| pybindsrc/module.cpp | ❌ Needs formatting |
| src/Configuration.cpp | ❌ Needs formatting |
| src/Context.cpp | ❌ Needs formatting |
| src/ErrorHandler.cpp | ❌ Needs formatting |
| src/InputStream.cpp | ❌ Needs formatting |
| src/Issue.cpp | ❌ Needs formatting |
| src/IssueCatcherHandler.cpp | ❌ Needs formatting |
| src/IssueFactory.cpp | ❌ Needs formatting |
| src/LocalContext.cpp | ❌ Needs formatting |
| src/LocalStream.cpp | ❌ Needs formatting |
| src/OutputStream.cpp | ❌ Needs formatting |
| src/PluginManager.cpp | ❌ Needs formatting |
| src/SampleIssues.cpp | ❌ Needs formatting |
| src/Schema.cpp | ❌ Needs formatting |
| src/Severity.cpp | ❌ Needs formatting |
| src/StandardStreamOutput.cpp | ❌ Needs formatting |
| src/StreamFactory.cpp | ❌ Needs formatting |
| src/StreamManager.cpp | ❌ Needs formatting |
| src/Util.cpp | ❌ Needs formatting |
| test/apps/ers_inheritance.cxx | ✅ Already formatted |
| test/apps/ers_receiver.cxx | ✅ Already formatted |
| test/apps/ers_schema_test.cxx | ✅ Already formatted |
| test/apps/ers_test.cxx | ✅ Already formatted |
apps/ers_config.cxx
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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 30: "cout" should not be used for output in DUNE DAQ software.
- Line 31: "cout" should not be used for output in DUNE DAQ software.
- Line 36: "cout" should not be used for output in DUNE DAQ software.
- Line 37: "cout" should not be used for output in DUNE DAQ software.
- Line 38: "cout" should not be used for output in DUNE DAQ software.
- Line 57: "cout" should not be used for output in DUNE DAQ software.
Warnings
- Line 21: inclusion of deprecated C++ header 'string.h'; consider using 'cstring' instead
- Line 45: do not use pointer arithmetic
- Line 46: do not use pointer arithmetic
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
include/ers/AnyIssue.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 21: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_ANYISSUE_HPP_
- Line 80: #endif line should be "#endif // ERS_INCLUDE_ERS_ANYISSUE_HPP_"
- Line 67: "virtual" is redundant since function is already declared as "final"
- Line 78: Namespace should be terminated with '// namespace ers '
- Line 75: Add #include
for string - Line 51: Add #include
for vector<> - Line 52: Add #include
include/ers/Assertion.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_ASSERTION_HPP_
- Line 100: #endif line should be "#endif // ERS_INCLUDE_ERS_ASSERTION_HPP_"
- Line 50: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 52: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 57: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 58: If/else bodies with multiple statements require braces
- Line 73: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 80: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 87: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 96: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 62: Add #include
for string
include/ers/Configuration.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 15: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 12: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_CONFIGURATION_HPP_
- Line 79: #endif line should be "#endif // ERS_INCLUDE_ERS_CONFIGURATION_HPP_"
- Line 70: Single-parameter constructors should be marked explicit.
- Line 77: Namespace should be terminated with '// namespace ers '
include/ers/Context.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 22: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_CONTEXT_HPP_
- Line 68: #endif line should be "#endif // ERS_INCLUDE_ERS_CONTEXT_HPP_"
- Line 66: Namespace should be terminated with '// namespace ers '
include/ers/InputStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 21: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INPUTSTREAM_HPP_
- Line 78: #endif line should be "#endif // ERS_INCLUDE_ERS_INPUTSTREAM_HPP_"
- Line 64: Namespace should be terminated with '// namespace ers '
- Line 68: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 76: Anonymous namespace should be terminated with '// namespace "" '
include/ers/Issue.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 22: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_ISSUE_HPP_
- Line 247: #endif line should be "#endif // ERS_INCLUDE_ERS_ISSUE_HPP_"
- Line 195: Namespace should be terminated with '// namespace ers '
- Line 241: Using C-style cast. Use static_cast
(...) instead - Line 187: Add #include
for vector<>
include/ers/IssueCatcherHandler.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 15: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 12: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_ISSUECATCHERHANDLER_HPP_
- Line 55: #endif line should be "#endif // ERS_INCLUDE_ERS_ISSUECATCHERHANDLER_HPP_"
- Line 53: Namespace should be terminated with '// namespace ers '
include/ers/IssueFactory.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 22: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_ISSUEFACTORY_HPP_
- Line 90: #endif line should be "#endif // ERS_INCLUDE_ERS_ISSUEFACTORY_HPP_"
- Line 88: Namespace should be terminated with '// namespace ers '
- Line 69: Add #include
- for list<>
include/ers/IssueReceiver.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 21: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_ISSUERECEIVER_HPP_
- Line 58: #endif line should be "#endif // ERS_INCLUDE_ERS_ISSUERECEIVER_HPP_"
- Line 56: Namespace should be terminated with '// namespace ers '
include/ers/LocalContext.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 21: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_LOCALCONTEXT_HPP_
- Line 146: #endif line should be "#endif // ERS_INCLUDE_ERS_LOCALCONTEXT_HPP_"
- Line 35: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 37: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 134: Namespace should be terminated with '// namespace ers '
- Line 138: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 141: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 143: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
include/ers/LocalStream.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 15: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 12: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_LOCALSTREAM_HPP_
- Line 99: #endif line should be "#endif // ERS_INCLUDE_ERS_LOCALSTREAM_HPP_"
- Line 79: Single-parameter constructors should be marked explicit.
- Line 88: Access specifier "private" has already appeared in class LocalStream
- Line 97: Namespace should be terminated with '// namespace ers '
- Line 90: Add #include
for unique_ptr<>
include/ers/OutputStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 23: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_OUTPUTSTREAM_HPP_
- Line 79: #endif line should be "#endif // ERS_INCLUDE_ERS_OUTPUTSTREAM_HPP_"
- Line 61: Single-parameter constructors should be marked explicit.
- Line 75: Namespace should be terminated with '// namespace ers '
include/ers/RemoteContext.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 21: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_REMOTECONTEXT_HPP_
- Line 136: #endif line should be "#endif // ERS_INCLUDE_ERS_REMOTECONTEXT_HPP_"
- Line 134: Namespace should be terminated with '// namespace ers '
- Line 131: Add #include
for string
include/ers/SampleIssues.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 15: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 12: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_SAMPLEISSUES_HPP_
- Line 76: #endif line should be "#endif // ERS_INCLUDE_ERS_SAMPLEISSUES_HPP_"
- Line 73: Using C-style cast. Use static_cast
(...) instead
include/ers/Schema.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 7: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_SCHEMA_HPP_
- Line 24: #endif line should be "#endif // ERS_INCLUDE_ERS_SCHEMA_HPP_"
- Line 22: Namespace should be terminated with '// namespace ers '
include/ers/Severity.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_SEVERITY_HPP_
- Line 87: #endif line should be "#endif // ERS_INCLUDE_ERS_SEVERITY_HPP_"
- Line 41: Constructors callable with one argument should be marked explicit.
- Line 85: Namespace should be terminated with '// namespace ers '
include/ers/StandardStreamOutput.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_STANDARDSTREAMOUTPUT_HPP_
- Line 47: #endif line should be "#endif // ERS_INCLUDE_ERS_STANDARDSTREAMOUTPUT_HPP_"
- Line 45: Namespace should be terminated with '// namespace ers '
include/ers/StreamFactory.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 22: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_STREAMFACTORY_HPP_
- Line 97: #endif line should be "#endif // ERS_INCLUDE_ERS_STREAMFACTORY_HPP_"
- Line 84: Single-parameter constructors should be marked explicit.
- Line 95: Namespace should be terminated with '// namespace ers '
- Line 88: Add #include
for string
include/ers/StreamManager.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 21: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_STREAMMANAGER_HPP_
- Line 124: #endif line should be "#endif // ERS_INCLUDE_ERS_STREAMMANAGER_HPP_"
- Line 109: Single-parameter constructors should be marked explicit.
- Line 122: Namespace should be terminated with '// namespace ers '
- Line 112: Add #include
for string - Line 112: Add #include
for vector<>
include/ers/ers.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 45: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 53: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 58: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 74: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 80: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 88: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 95: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 102: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 109: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 116: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 122: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 28: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_ERS_HPP_
- Line 145: #endif line should be "#endif // ERS_INCLUDE_ERS_ERS_HPP_"
- Line 133: Namespace should be terminated with '// namespace ers '
- Line 137: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
include/ers/internal/AbortStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_ABORTSTREAM_HPP_
- Line 52: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_ABORTSTREAM_HPP_"
- Line 50: Namespace should be terminated with '// namespace ers '
include/ers/internal/ExitStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_EXITSTREAM_HPP_
- Line 63: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_EXITSTREAM_HPP_"
- Line 61: Namespace should be terminated with '// namespace ers '
- Line 54: Add #include
for string
include/ers/internal/FilterStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 27: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_FILTERSTREAM_HPP_
- Line 64: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_FILTERSTREAM_HPP_"
- Line 62: Namespace should be terminated with '// namespace ers '
- Line 60: Add #include
for string - Line 60: Add #include
for vector<>
include/ers/internal/FormattedStandardStream.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 15: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_FORMATTEDSTANDARDSTREAM_HPP_
- Line 70: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_FORMATTEDSTANDARDSTREAM_HPP_"
- Line 33: Namespace should be terminated with '// namespace format '
- Line 40: Single-parameter constructors should be marked explicit.
- Line 66: Namespace should be terminated with '// namespace ers '
- Line 61: Add #include
for string - Line 64: Add #include
for vector<>
include/ers/internal/GlobalLockStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_GLOBALLOCKSTREAM_HPP_
- Line 52: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_GLOBALLOCKSTREAM_HPP_"
- Line 50: Namespace should be terminated with '// namespace ers '
include/ers/internal/IssueDeclarationMacro.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 12: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_ISSUEDECLARATIONMACRO_HPP_
- Line 163: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_ISSUEDECLARATIONMACRO_HPP_"
- Line 24: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 25: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 26: #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 28: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 29: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 30: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 32: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 33: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 35: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 38: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 42: #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 54: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 59: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 62: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 68: Single-parameter constructors should be marked explicit.
- Line 74: Access specifier "public:" appears after one (or both) of "private:" and/or "protected:", not before
- Line 94: "virtual" is redundant since function is already declared as "override"
- Line 98: Namespace should be terminated with '// namespace namespace_name '
- Line 100: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 138: Namespace should be terminated with '// namespace namespace_name '
- Line 141: Anonymous namespace should be terminated with '// namespace "" '
- Line 143: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 146: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 149: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 152: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 155: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 159: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 121: Add #include
for string - Line 94: Add #include
- for list<>
include/ers/internal/LockStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_LOCKSTREAM_HPP_
- Line 56: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_LOCKSTREAM_HPP_"
- Line 54: Namespace should be terminated with '// namespace ers '
include/ers/internal/NullStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_NULLSTREAM_HPP_
- Line 56: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_NULLSTREAM_HPP_"
- Line 26: Use of NULL #define found; prefer using the nullptr keyword
- Line 27: Use of NULL #define found; prefer using the nullptr keyword
- Line 54: Namespace should be terminated with '// namespace ers '
include/ers/internal/PluginManager.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_PLUGINMANAGER_HPP_
- Line 83: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_PLUGINMANAGER_HPP_"
- Line 37: Single-parameter constructors should be marked explicit.
- Line 59: Single-parameter constructors should be marked explicit.
- Line 81: Namespace should be terminated with '// namespace ers '
include/ers/internal/RFilterStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_RFILTERSTREAM_HPP_
- Line 65: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_RFILTERSTREAM_HPP_"
- Line 53: Single-parameter constructors should be marked explicit.
- Line 63: Namespace should be terminated with '// namespace ers '
- Line 53: Add #include
for string - Line 61: Add #include
for vector<>
include/ers/internal/SingletonCreator.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 22: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 12: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_SINGLETONCREATOR_HPP_
- Line 63: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_SINGLETONCREATOR_HPP_"
- Line 61: Namespace should be terminated with '// namespace ers '
include/ers/internal/StandardStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_STANDARDSTREAM_HPP_
- Line 63: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_STANDARDSTREAM_HPP_"
- Line 51: Single-parameter constructors should be marked explicit.
- Line 61: Namespace should be terminated with '// namespace ers '
- Line 51: Add #include
for string
include/ers/internal/ThrottleStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 21: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_THROTTLESTREAM_HPP_
- Line 79: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_THROTTLESTREAM_HPP_"
- Line 65: Access specifier "private" has already appeared in class ThrottleStream
- Line 77: Namespace should be terminated with '// namespace ers '
- Line 70: Add #include
for string
include/ers/internal/ThrowStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_THROWSTREAM_HPP_
- Line 52: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_THROWSTREAM_HPP_"
- Line 50: Namespace should be terminated with '// namespace ers '
include/ers/internal/Util.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_UTIL_HPP_
- Line 43: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_UTIL_HPP_"
- Line 41: Namespace should be terminated with '// namespace ers '
include/ers/internal/macro.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 12: #ifndef header guard has wrong style, please use: ERS_INCLUDE_ERS_INTERNAL_MACRO_HPP_
- Line 80: #endif line should be "#endif // ERS_INCLUDE_ERS_INTERNAL_MACRO_HPP_"
- Line 27: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 30: static storage declaration outside of class or function not allowed (if this isn't a header, please contact John Freeman)
- Line 35: Anonymous namespace should be terminated with '// namespace "" '
- Line 37: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 38: If/else bodies with multiple statements require braces
- Line 47: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 55: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 63: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 71: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 30: Add #include
for string
plugins/AbortStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 22: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
plugins/ExitStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 26: Add #include
for string
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 22: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
plugins/FilterStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 33: Anonymous namespace should be terminated with '// namespace "" '
- Line 45: Else clause should be indented at the same level as if. Ambiguous nested if/else chains require braces.
- Line 75: Add #include
for string - Line 75: Add #include
for vector<>
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 66: use auto when declaring iterators
- Line 75: use auto when declaring iterators
plugins/GlobalLockStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
plugins/LockStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
plugins/NullStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 71: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
plugins/RFilterStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 31: Anonymous namespace should be terminated with '// namespace "" '
- Line 93: Add #include
for string - Line 93: Add #include
for vector<>
Warnings
- Line 98: The value '131584' provided to the cast expression is not in the valid range of values for '_match_flags'
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 48: use emplace_back instead of push_back
- Line 50: use emplace_back instead of push_back
plugins/StandardStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 48: Access specifier "private" has already appeared in class OutDevice
- Line 66: Access specifier "private" has already appeared in class ObjectLock
- Line 104: Access specifier "private" has already appeared in class LockedDevice
- Line 108: Single-parameter constructors should be marked explicit.
- Line 122: Constructors callable with one argument should be marked explicit.
- Line 123: "cout" should not be used for output in DUNE DAQ software.
- Line 130: Constructors callable with one argument should be marked explicit.
- Line 131: "cerr" should not be used for output in DUNE DAQ software.
- Line 138: Single-parameter constructors should be marked explicit.
- Line 151: Anonymous namespace should be terminated with '// namespace "" '
- Line 138: Add #include
for string
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 25: macro 'FIELD_SEPARATOR' used to declare a constant; consider using a 'constexpr' constant
- Line 26: macro 'DELIMITER' used to declare a constant; consider using a 'constexpr' constant
- Line 29: initialization of 's_supported_fields' with static storage duration may throw an exception that cannot be caught
- Line 56: use auto when declaring iterators
- Line 68: use auto when declaring iterators
- Line 86: use auto when declaring iterators
- Line 123: use auto when declaring iterators
- Line 132: use auto when declaring iterators
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 48: redundant access specifier has the same accessibility as the previous access specifier
- Line 66: redundant access specifier has the same accessibility as the previous access specifier
- Line 78: use auto when initializing with new to avoid duplicating the type name
- Line 104: redundant access specifier has the same accessibility as the previous access specifier
plugins/ThrottleStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 57: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 83: If an else has a brace on one side, it should have it on both
- Line 87: If an else has a brace on one side, it should have it on both
- Line 90: If an else has a brace on one side, it should have it on both
- Line 129: Add #include
for string - Line 103: Add #include
for vector<>
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 65: redundant access specifier has the same accessibility as the previous access specifier
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
plugins/ThrowStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
pybindsrc/module.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 79: Add #include
for string - Line 42: Add #include
- Line 42: Add #include
for vector<>
Warnings
- Line 53: use nullptr
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 113: use nullptr
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 26: object destroyed immediately after creation; did you mean to name the object?
- Line 28: object destroyed immediately after creation; did you mean to name the object?
- Line 30: object destroyed immediately after creation; did you mean to name the object?
src/Configuration.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
src/Context.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 48: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 92: Anonymous namespace should be terminated with '// namespace "" '
- Line 99: Using C-style cast. Use one of the C++ cast operators instead
- Line 115: Add #include
for string - Line 97: Add #include
for vector<>
Warnings
- Line 20: inclusion of deprecated C++ header 'string.h'; consider using 'cstring' instead
- Line 45: variable 'status' is not initialized
- Line 48: do not use pointer arithmetic
- Line 52: use nullptr
- Line 52: use nullptr
- Line 60: do not manage memory manually; use RAII
- Line 81: do not use pointer arithmetic
- Line 84: do not use pointer arithmetic
- Line 99: do not use C-style cast to cast away constness
- Line 99: C-style casts are discouraged; use const_cast
- Line 103: do not use pointer arithmetic
- Line 105: do not manage memory manually; use RAII
- Line 123: do not use pointer arithmetic
- Line 124: do not use pointer arithmetic
- Line 125: do not use pointer arithmetic
- Line 127: do not use pointer arithmetic
src/ErrorHandler.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 104: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 42: Using C-style cast. Use static_cast
(...) instead - Line 95: "cerr" should not be used for output in DUNE DAQ software.
- Line 105: Using C-style cast. Use one of the C++ cast operators instead
- Line 106: Using C-style cast. Use one of the C++ cast operators instead
- Line 131: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 139: "cerr" should not be used for output in DUNE DAQ software.
- Line 153: A catch-all-exceptions construct was found in a file which doesn't contain int main(); this can only be used to wrap main()
- Line 163: Namespace should be terminated with '// namespace ers '
- Line 170: Anonymous namespace should be terminated with '// namespace "" '
- Line 52: Add #include
for string
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 74: use nullptr
- Line 105: use auto when initializing with a cast to avoid duplicating the type name
- Line 105: do not use C-style cast to convert between unrelated types
- Line 105: C-style casts are discouraged; use static_cast
- Line 106: do not use pointer arithmetic
- Line 106: do not use C-style cast to convert between unrelated types
- Line 106: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
- Line 169: declaration uses identifier '__handler__', which is a reserved identifier
- Line 169: initialization of '__handler__' with static storage duration may throw an exception that cannot be caught
src/InputStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 31: Anonymous namespace should be terminated with '// namespace "" '
Warnings
- Line 71: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
src/Issue.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 62: Anonymous namespace should be terminated with '// namespace "" '
- Line 101: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 120: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 227: Namespace should be terminated with '// namespace ers '
- Line 212: Add #include
for string - Line 130: Add #include
for vector<> - Line 131: Add #include
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 27: inclusion of deprecated C++ header 'time.h'; consider using 'ctime' instead
- Line 57: use auto when declaring iterators
- Line 66: redundant get() call on smart pointer
- Line 66: use nullptr
- Line 101: use auto when initializing with a cast to avoid duplicating the type name
- Line 120: use auto when initializing with a cast to avoid duplicating the type name
- Line 154: use auto when declaring iterators
- Line 168: use auto when declaring iterators
src/IssueCatcherHandler.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
src/IssueFactory.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 94: Use of Run Time Type Information not allowed unless this code is meant to test other code
- Line 82: Add #include
for string - Line 77: Add #include
- for list<>
- Line 82: Add #include
for vector<>
Warnings
- Line 53: use nullptr
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 47: use auto when declaring iterators
- Line 63: use auto when declaring iterators
src/LocalContext.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 84: Anonymous namespace should be terminated with '// namespace "" '
- Line 75: Add #include
for string
Warnings
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 23: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
- Line 40: narrowing conversion from 'long' to signed type 'pid_t' (aka 'int') is implementation-defined
- Line 54: do not declare C-style arrays, use std::array<> instead
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 114: initialization of 'c_process' with static storage duration may throw an exception that cannot be caught
- Line 116: narrowing conversion from '__uid_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
src/LocalStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 84: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 53: Add #include
for unique_ptr<>
Warnings
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 56: redundant get() call on smart pointer
- Line 96: redundant get() call on smart pointer
- Line 101: prefer a lambda to std::bind
src/OutputStream.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 71: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 31: redundant get() call on smart pointer
src/PluginManager.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 27: Anonymous namespace should be terminated with '// namespace "" '
- Line 63: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 64: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 93: Namespace should be terminated with '// namespace ers '
- Line 73: Add #include
for string
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 42: throwing an exception whose type 'PluginException' is not derived from 'std::exception'
- Line 60: use auto when declaring iterators
- Line 79: use auto when declaring iterators
src/SampleIssues.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 53: Using C-style cast. Use static_cast
(...) instead
Warnings
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
src/Schema.cpp
Errors
- Line 0: The standard copyright message wasn't found.
Warnings
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_ANY_H__', which is a reserved identifier
- Line 43: nested namespaces can be concatenated
- Line 51: do not declare C-style arrays, use std::array<> instead
- Line 65: nested namespaces can be concatenated
- Line 110: do not use reinterpret_cast
- Line 482: use nullptr
- Line 76: declaration uses identifier 'GOOGLE_PROTOBUF_ARENA_ALIGN_H__', which is a reserved identifier
- Line 87: nested namespaces can be concatenated
- Line 98: do not use reinterpret_cast
- Line 115: do not use reinterpret_cast
- Line 160: do not use reinterpret_cast
- Line 37: nested namespaces can be concatenated
- Line 69: do not use reinterpret_cast
- Line 48: nested namespaces can be concatenated
- Line 56: do not use reinterpret_cast
- Line 139: variable 'elem' is not initialized
- Line 147: do not use reinterpret_cast
- Line 173: do not use reinterpret_cast
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_ARENASTRING_H__', which is a reserved identifier
- Line 53: nested namespaces can be concatenated
- Line 80: do not declare C-style arrays, use std::array<> instead
- Line 44: nested namespaces can be concatenated
- Line 55: declaration uses identifier 'GOOGLE_PROTOBUF_DESCRIPTOR_H__', which is a reserved identifier
- Line 87: nested namespaces can be concatenated
- Line 378: declaration uses identifier '__WELLKNOWNTYPE__DO_NOT_USE__ADD_DEFAULT_INSTEAD__', which is a reserved identifier
- Line 1147: do not declare C-style arrays, use std::array<> instead
- Line 43: nested namespaces can be concatenated
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_EXPLICITLY_CONSTRUCTED_H__', which is a reserved identifier
- Line 43: nested namespaces can be concatenated
- Line 72: do not use reinterpret_cast
- Line 68: nested namespaces can be concatenated
- Line 95: nested namespaces can be concatenated
- Line 418: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 426: variable 'tag' is not initialized
- Line 462: declaration uses identifier '_InternalSerialize', which is a reserved identifier
- Line 549: declaration uses identifier '_InternalSerializeImpl', which is a reserved identifier
- Line 591: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 852: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 55: nested namespaces can be concatenated
- Line 61: nested namespaces can be concatenated
- Line 81: variable 'tmp' is not initialized
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_GENERATED_ENUM_UTIL_H__', which is a reserved identifier
- Line 46: nested namespaces can be concatenated
- Line 39: declaration uses identifier 'GOOGLE_PROTOBUF_GENERATED_MESSAGE_REFLECTION_H__', which is a reserved identifier
- Line 59: nested namespaces can be concatenated
- Line 69: nested namespaces can be concatenated
- Line 143: do not use pointer arithmetic
- Line 50: nested namespaces can be concatenated
- Line 369: do not use reinterpret_cast
- Line 69: nested namespaces can be concatenated
- Line 132: do not use reinterpret_cast
- Line 143: do not use pointer arithmetic
- Line 147: do not use reinterpret_cast
- Line 44: nested namespaces can be concatenated
- Line 58: do not use array subscript when the index is not an integer constant expression
- Line 51: nested namespaces can be concatenated
- Line 88: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_INLINED_STRING_FIELD_H__', which is a reserved identifier
- Line 50: nested namespaces can be concatenated
- Line 365: do not declare C-style arrays, use std::array<> instead
- Line 33: nested namespaces can be concatenated
- Line 110: declaration uses identifier 'GOOGLE_PROTOBUF_IO_CODED_STREAM_H__', which is a reserved identifier
- Line 142: nested namespaces can be concatenated
- Line 658: do not use pointer arithmetic
- Line 1645: variable 'size' is not initialized
- Line 1681: do not use pointer arithmetic
- Line 118: nested namespaces can be concatenated
- Line 45: declaration uses identifier 'GOOGLE_PROTOBUF_IO_ZERO_COPY_STREAM_IMPL_LITE_H__', which is a reserved identifier
- Line 64: nested namespaces can be concatenated
- Line 257: do not declare C-style arrays, use std::array<> instead
- Line 74: nested namespaces can be concatenated
- Line 144: do not use reinterpret_cast
- Line 698: do not use pointer arithmetic
- Line 714: variable 'lo' is not initialized
- Line 800: do not use pointer arithmetic
- Line 49: nested namespaces can be concatenated
- Line 61: nested namespaces can be concatenated
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_MAP_ENTRY_LITE_H__', which is a reserved identifier
- Line 58: nested namespaces can be concatenated
- Line 73: nested namespaces can be concatenated
- Line 200: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 202: variable 'tag' is not initialized
- Line 236: declaration uses identifier '_InternalSerialize', which is a reserved identifier
- Line 302: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 304: do not use pointer arithmetic
- Line 59: nested namespaces can be concatenated
- Line 71: function-like macro 'TYPE_CHECK' used; consider a 'constexpr' template function
- Line 74: macro argument should be enclosed in parentheses
- Line 307: nested namespaces can be concatenated
- Line 473: do not use reinterpret_cast
- Line 705: do not use reinterpret_cast
- Line 853: do not use reinterpret_cast
- Line 52: nested namespaces can be concatenated
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_MAP_FIELD_LITE_H__', which is a reserved identifier
- Line 51: nested namespaces can be concatenated
- Line 107: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 135: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_MAP_TYPE_HANDLER_H__', which is a reserved identifier
- Line 44: nested namespaces can be concatenated
- Line 398: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 143: nested namespaces can be concatenated
- Line 222: do not use reinterpret_cast
- Line 67: nested namespaces can be concatenated
- Line 456: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 501: declaration uses identifier '_InternalSerialize', which is a reserved identifier
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_METADATA_LITE_H__', which is a reserved identifier
- Line 47: nested namespaces can be concatenated
- Line 71: do not use reinterpret_cast
- Line 198: do not use reinterpret_cast
- Line 64: nested namespaces can be concatenated
- Line 127: do not use pointer arithmetic
- Line 130: do not use pointer arithmetic
- Line 633: variable 'tmp' is not initialized
- Line 642: variable 'tmp' is not initialized
- Line 658: do not use reinterpret_cast
- Line 1113: variable 'tmp' is not initialized
- Line 1119: variable 'tmp' is not initialized
- Line 1125: variable 'tmp' is not initialized
- Line 1159: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 1247: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 1319: use auto when initializing with a template cast to avoid duplicating the type name
- Line 1320: do not use pointer arithmetic
- Line 1340: do not use pointer arithmetic
- Line 37: declaration uses identifier 'GOOGLE_PROTOBUF_PORT_H__', which is a reserved identifier
- Line 56: nested namespaces can be concatenated
- Line 148: function-like macro 'PROTOBUF_BUILTIN_BSWAP16' used; consider a 'constexpr' template function
- Line 214: function-like macro 'PROTOBUF_ABSL_MIN' used; consider a 'constexpr' template function
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_REFLECTION_H__', which is a reserved identifier
- Line 49: nested namespaces can be concatenated
- Line 361: do not use reinterpret_cast
- Line 39: declaration uses identifier 'GOOGLE_PROTOBUF_REFLECTION_OPS_H__', which is a reserved identifier
- Line 51: nested namespaces can be concatenated
- Line 45: declaration uses identifier 'GOOGLE_PROTOBUF_REPEATED_FIELD_H__', which is a reserved identifier
- Line 79: nested namespaces can be concatenated
- Line 116: variable 'buf' is not initialized
- Line 121: do not use pointer arithmetic
- Line 1019: do not use reinterpret_cast
- Line 74: nested namespaces can be concatenated
- Line 108: consider replacing 'long' with 'int64'
- Line 527: do not use reinterpret_cast
- Line 644: redundant access specifier has the same accessibility as the previous access specifier
- Line 739: do not use reinterpret_cast
- Line 751: do not use reinterpret_cast
- Line 753: do not use reinterpret_cast
- Line 1542: use auto when initializing with a cast to avoid duplicating the type name
- Line 1684: do not use reinterpret_cast
- Line 58: nested namespaces can be concatenated
- Line 77: do not use reinterpret_cast
- Line 217: do not use pointer arithmetic
- Line 438: variable 'p' is not initialized
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_STRING_BLOCK_H__', which is a reserved identifier
- Line 49: nested namespaces can be concatenated
- Line 184: do not use reinterpret_cast
- Line 36: declaration uses identifier 'GOOGLE_PROTOBUF_COMMON_H__', which is a reserved identifier
- Line 60: nested namespaces can be concatenated
- Line 69: macro 'GOOGLE_PROTOBUF_VERSION' used to declare a constant; consider using a 'constexpr' constant
- Line 150: variable 't' is not initialized
- Line 156: variable 't' is not initialized
- Line 162: variable 't' is not initialized
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_THREAD_SAFE_ARENA_H__', which is a reserved identifier
- Line 53: nested namespaces can be concatenated
- Line 92: variable 'arena' is not initialized
- Line 101: variable 'arena' is not initialized
- Line 113: variable 'arena' is not initialized
- Line 39: declaration uses identifier 'GOOGLE_PROTOBUF_UNKNOWN_FIELD_SET_H__', which is a reserved identifier
- Line 62: nested namespaces can be concatenated
- Line 220: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 74: nested namespaces can be concatenated
- Line 210: function-like macro 'GOOGLE_PROTOBUF_WIRE_FORMAT_MAKE_TAG' used; consider a 'constexpr' template function
- Line 950: variable 'temp' is not initialized
- Line 970: variable 'temp' is not initialized
- Line 978: variable 'temp' is not initialized
- Line 999: variable 'temp' is not initialized
- Line 1008: variable 'temp' is not initialized
- Line 1016: variable 'temp' is not initialized
- Line 1024: variable 'temp' is not initialized
- Line 1032: variable 'temp' is not initialized
- Line 1040: variable 'temp' is not initialized
- Line 1062: variable 'temp' is not initialized
- Line 1071: variable 'temp' is not initialized
- Line 1080: variable 'temp' is not initialized
- Line 1089: variable 'temp' is not initialized
- Line 1129: variable 'void_pointer' is not initialized
- Line 1130: variable 'size' is not initialized
- Line 1133: use auto when initializing with a cast to avoid duplicating the type name
- Line 1133: do not use reinterpret_cast
- Line 1204: variable 'length' is not initialized
- Line 1235: do not use reinterpret_cast
- Line 1316: variable 'length' is not initialized
- Line 1492: do not use pointer arithmetic
- Line 1902: do not use reinterpret_cast
- Line 1927: use auto when initializing with a cast to avoid duplicating the type name
- Line 1930: do not use reinterpret_cast
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 64: loop variable is copied but only used as const reference; consider making it a const reference
src/Severity.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 27: Anonymous namespace should be terminated with '// namespace "" '
- Line 73: Use int16/int64/etc, rather than the C type short
- Line 101: Add #include
for string
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 12: inclusion of deprecated C++ header 'assert.h'; consider using 'cassert' instead
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 44: do not use array subscript when the index is not an integer constant expression
- Line 60: do not use array subscript when the index is not an integer constant expression
- Line 63: do not use array subscript when the index is not an integer constant expression
- Line 73: consider replacing 'short' with 'int16'
- Line 75: do not use array subscript when the index is not an integer constant expression
- Line 77: C-style casts are discouraged; use static_cast
- Line 91: variable 'type' is not initialized
src/StandardStreamOutput.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 29: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 67: Anonymous namespace should be terminated with '// namespace "" '
- Line 127: Add #include
for string - Line 127: Add #include
for vector<>
Warnings
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 29: macro 'FIELD_SEPARATOR' used to declare a constant; consider using a 'constexpr' constant
- Line 66: initialization of 'formatted_time' with static storage duration may throw an exception that cannot be caught
- Line 100: use auto when declaring iterators
- Line 106: use auto when declaring iterators
src/StreamFactory.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 164: Add #include
for string
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 71: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 71: use auto when declaring iterators
- Line 89: use nullptr
- Line 120: use auto when declaring iterators
- Line 172: use auto when declaring iterators
- Line 178: use auto when declaring iterators
src/StreamManager.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 12: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 77: Use int16/int64/etc, rather than the C type short
- Line 109: Anonymous namespace should be terminated with '// namespace "" '
- Line 118: Single-parameter constructors should be marked explicit.
- Line 131: If an else has a brace on one side, it should have it on both
- Line 135: "cout" should not be used for output in DUNE DAQ software.
- Line 137: "cerr" should not be used for output in DUNE DAQ software.
- Line 155: Namespace should be terminated with '// namespace ers '
- Line 176: Use int16/int64/etc, rather than the C type short
- Line 239: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 386: Use int16/int64/etc, rather than the C type short
- Line 280: Add #include
for string - Line 280: Add #include
for vector<> - Line 235: Add #include
for shared_ptr<> - Line 235: Add #include
- for list<>
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 71: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 22: inclusion of deprecated C++ header 'assert.h'; consider using 'cassert' instead
- Line 51: do not declare C-style arrays, use std::array<> instead
- Line 69: do not use array subscript when the index is not an integer constant expression
- Line 76: multiple declarations in a single statement reduces readability
- Line 77: consider replacing 'short' with 'int16'
- Line 141: do not use array subscript when the index is not an integer constant expression
- Line 142: do not use array subscript when the index is not an integer constant expression
- Line 176: consider replacing 'short' with 'int16'
- Line 178: do not use array subscript when the index is not an integer constant expression
- Line 179: do not use array subscript when the index is not an integer constant expression
- Line 179: do not use array subscript when the index is not an integer constant expression
- Line 191: do not use array subscript when the index is not an integer constant expression
- Line 203: do not use array subscript when the index is not an integer constant expression
- Line 235: use auto when declaring iterators
- Line 267: do not use array subscript when the index is not an integer constant expression
- Line 283: use nullptr
- Line 293: use nullptr
- Line 319: do not use array subscript when the index is not an integer constant expression
- Line 386: consider replacing 'short' with 'int16'
- Line 388: C-style casts are discouraged; use static_cast
- Line 389: C-style casts are discouraged; use static_cast
src/Util.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 34: Add #include
for string - Line 20: Add #include
for vector<>
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 12: inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
- Line 22: multiple declarations in a single statement reduces readability
- Line 22: variable 'start_p' is not initialized
- Line 22: variable 'end_p' is not initialized
- Line 44: 'sscanf' used to convert a string to an integer value, but function will not report conversion errors; consider using 'strtol' instead
test/apps/ers_inheritance.cxx
Errors
- Line 0: The standard copyright message wasn't found.
- Line 44: "cout" should not be used for output in DUNE DAQ software.
- Line 46: "cout" should not be used for output in DUNE DAQ software.
- Line 39: Add #include
for make_unique<> - Line 35: Add #include
for vector<>
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
test/apps/ers_receiver.cxx
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 16: "cout" should not be used for output in DUNE DAQ software.
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 71: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
test/apps/ers_schema_test.cxx
Errors
- Line 0: The standard copyright message wasn't found.
- Line 18: Using C-style cast. Use static_cast
(...) instead - Line 42: "cerr" should not be used for output in DUNE DAQ software.
- Line 45: "cout" should not be used for output in DUNE DAQ software.
- Line 59: "cout" should not be used for output in DUNE DAQ software.
- Line 61: "cout" should not be used for output in DUNE DAQ software.
Warnings
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_ANY_H__', which is a reserved identifier
- Line 43: nested namespaces can be concatenated
- Line 51: do not declare C-style arrays, use std::array<> instead
- Line 52: do not declare C-style arrays, use std::array<> instead
- Line 60: declaration uses identifier '_Any_default_instance_', which is a reserved identifier
- Line 64: nested namespaces can be concatenated
- Line 124: do not use reinterpret_cast
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_ARENA_H__', which is a reserved identifier
- Line 65: nested namespaces can be concatenated
- Line 110: do not use reinterpret_cast
- Line 482: use nullptr
- Line 76: declaration uses identifier 'GOOGLE_PROTOBUF_ARENA_ALIGN_H__', which is a reserved identifier
- Line 87: nested namespaces can be concatenated
- Line 98: do not use reinterpret_cast
- Line 115: do not use reinterpret_cast
- Line 160: do not use reinterpret_cast
- Line 37: nested namespaces can be concatenated
- Line 69: do not use reinterpret_cast
- Line 48: nested namespaces can be concatenated
- Line 56: do not use reinterpret_cast
- Line 139: variable 'elem' is not initialized
- Line 147: do not use reinterpret_cast
- Line 173: do not use reinterpret_cast
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_ARENASTRING_H__', which is a reserved identifier
- Line 53: nested namespaces can be concatenated
- Line 80: do not declare C-style arrays, use std::array<> instead
- Line 44: nested namespaces can be concatenated
- Line 55: declaration uses identifier 'GOOGLE_PROTOBUF_DESCRIPTOR_H__', which is a reserved identifier
- Line 87: nested namespaces can be concatenated
- Line 378: declaration uses identifier '__WELLKNOWNTYPE__DO_NOT_USE__ADD_DEFAULT_INSTEAD__', which is a reserved identifier
- Line 1147: do not declare C-style arrays, use std::array<> instead
- Line 43: nested namespaces can be concatenated
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_EXPLICITLY_CONSTRUCTED_H__', which is a reserved identifier
- Line 43: nested namespaces can be concatenated
- Line 72: do not use reinterpret_cast
- Line 68: nested namespaces can be concatenated
- Line 95: nested namespaces can be concatenated
- Line 418: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 426: variable 'tag' is not initialized
- Line 462: declaration uses identifier '_InternalSerialize', which is a reserved identifier
- Line 549: declaration uses identifier '_InternalSerializeImpl', which is a reserved identifier
- Line 591: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 852: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 55: nested namespaces can be concatenated
- Line 61: nested namespaces can be concatenated
- Line 81: variable 'tmp' is not initialized
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_GENERATED_ENUM_UTIL_H__', which is a reserved identifier
- Line 46: nested namespaces can be concatenated
- Line 39: declaration uses identifier 'GOOGLE_PROTOBUF_GENERATED_MESSAGE_REFLECTION_H__', which is a reserved identifier
- Line 59: nested namespaces can be concatenated
- Line 69: nested namespaces can be concatenated
- Line 143: do not use pointer arithmetic
- Line 50: nested namespaces can be concatenated
- Line 369: do not use reinterpret_cast
- Line 69: nested namespaces can be concatenated
- Line 132: do not use reinterpret_cast
- Line 143: do not use pointer arithmetic
- Line 147: do not use reinterpret_cast
- Line 44: nested namespaces can be concatenated
- Line 58: do not use array subscript when the index is not an integer constant expression
- Line 51: nested namespaces can be concatenated
- Line 88: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_INLINED_STRING_FIELD_H__', which is a reserved identifier
- Line 50: nested namespaces can be concatenated
- Line 365: do not declare C-style arrays, use std::array<> instead
- Line 33: nested namespaces can be concatenated
- Line 110: declaration uses identifier 'GOOGLE_PROTOBUF_IO_CODED_STREAM_H__', which is a reserved identifier
- Line 142: nested namespaces can be concatenated
- Line 658: do not use pointer arithmetic
- Line 1645: variable 'size' is not initialized
- Line 1681: do not use pointer arithmetic
- Line 118: nested namespaces can be concatenated
- Line 45: declaration uses identifier 'GOOGLE_PROTOBUF_IO_ZERO_COPY_STREAM_IMPL_LITE_H__', which is a reserved identifier
- Line 64: nested namespaces can be concatenated
- Line 257: do not declare C-style arrays, use std::array<> instead
- Line 46: nested namespaces can be concatenated
- Line 38: declaration uses identifier 'GOOGLE_PROTOBUF_MAP_H__', which is a reserved identifier
- Line 74: nested namespaces can be concatenated
- Line 144: do not use reinterpret_cast
- Line 698: do not use pointer arithmetic
- Line 714: variable 'lo' is not initialized
- Line 800: do not use pointer arithmetic
- Line 49: nested namespaces can be concatenated
- Line 61: nested namespaces can be concatenated
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_MAP_ENTRY_LITE_H__', which is a reserved identifier
- Line 58: nested namespaces can be concatenated
- Line 73: nested namespaces can be concatenated
- Line 200: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 202: variable 'tag' is not initialized
- Line 236: declaration uses identifier '_InternalSerialize', which is a reserved identifier
- Line 302: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 304: do not use pointer arithmetic
- Line 59: nested namespaces can be concatenated
- Line 71: function-like macro 'TYPE_CHECK' used; consider a 'constexpr' template function
- Line 74: macro argument should be enclosed in parentheses
- Line 307: nested namespaces can be concatenated
- Line 473: do not use reinterpret_cast
- Line 705: do not use reinterpret_cast
- Line 853: do not use reinterpret_cast
- Line 52: nested namespaces can be concatenated
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_MAP_FIELD_LITE_H__', which is a reserved identifier
- Line 51: nested namespaces can be concatenated
- Line 107: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 135: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_MAP_TYPE_HANDLER_H__', which is a reserved identifier
- Line 44: nested namespaces can be concatenated
- Line 398: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 143: nested namespaces can be concatenated
- Line 222: do not use reinterpret_cast
- Line 67: nested namespaces can be concatenated
- Line 456: declaration uses identifier '_InternalParse', which is a reserved identifier
- Line 501: declaration uses identifier '_InternalSerialize', which is a reserved identifier
- Line 32: declaration uses identifier 'GOOGLE_PROTOBUF_METADATA_LITE_H__', which is a reserved identifier
- Line 47: nested namespaces can be concatenated
- Line 71: do not use reinterpret_cast
- Line 198: do not use reinterpret_cast
- Line 64: nested namespaces can be concatenated
- Line 127: do not use pointer arithmetic
- Line 130: do not use pointer arithmetic
- Line 633: variable 'tmp' is not initialized
- Line 642: variable 'tmp' is not initialized
- Line 658: do not use reinterpret_cast
- Line 1113: variable 'tmp' is not initialized
- Line 1119: variable 'tmp' is not initialized
- Line 1125: variable 'tmp' is not initialized
- Line 1159: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 1247: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 1319: use auto when initializing with a template cast to avoid duplicating the type name
- Line 1320: do not use pointer arithmetic
- Line 1340: do not use pointer arithmetic
- Line 37: declaration uses identifier 'GOOGLE_PROTOBUF_PORT_H__', which is a reserved identifier
- Line 56: nested namespaces can be concatenated
- Line 148: function-like macro 'PROTOBUF_BUILTIN_BSWAP16' used; consider a 'constexpr' template function
- Line 214: function-like macro 'PROTOBUF_ABSL_MIN' used; consider a 'constexpr' template function
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_REFLECTION_H__', which is a reserved identifier
- Line 49: nested namespaces can be concatenated
- Line 361: do not use reinterpret_cast
- Line 39: declaration uses identifier 'GOOGLE_PROTOBUF_REFLECTION_OPS_H__', which is a reserved identifier
- Line 51: nested namespaces can be concatenated
- Line 45: declaration uses identifier 'GOOGLE_PROTOBUF_REPEATED_FIELD_H__', which is a reserved identifier
- Line 79: nested namespaces can be concatenated
- Line 116: variable 'buf' is not initialized
- Line 121: do not use pointer arithmetic
- Line 1019: do not use reinterpret_cast
- Line 74: nested namespaces can be concatenated
- Line 108: consider replacing 'long' with 'int64'
- Line 527: do not use reinterpret_cast
- Line 644: redundant access specifier has the same accessibility as the previous access specifier
- Line 739: do not use reinterpret_cast
- Line 751: do not use reinterpret_cast
- Line 753: do not use reinterpret_cast
- Line 1542: use auto when initializing with a cast to avoid duplicating the type name
- Line 1684: do not use reinterpret_cast
- Line 58: nested namespaces can be concatenated
- Line 77: do not use reinterpret_cast
- Line 217: do not use pointer arithmetic
- Line 438: variable 'p' is not initialized
- Line 42: nested namespaces can be concatenated
- Line 52: do not declare C-style arrays, use std::array<> instead
- Line 60: declaration uses identifier '_SourceContext_default_instance_', which is a reserved identifier
- Line 64: nested namespaces can be concatenated
- Line 124: do not use reinterpret_cast
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_STRING_BLOCK_H__', which is a reserved identifier
- Line 49: nested namespaces can be concatenated
- Line 184: do not use reinterpret_cast
- Line 36: declaration uses identifier 'GOOGLE_PROTOBUF_COMMON_H__', which is a reserved identifier
- Line 60: nested namespaces can be concatenated
- Line 69: macro 'GOOGLE_PROTOBUF_VERSION' used to declare a constant; consider using a 'constexpr' constant
- Line 150: variable 't' is not initialized
- Line 156: variable 't' is not initialized
- Line 162: variable 't' is not initialized
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_THREAD_SAFE_ARENA_H__', which is a reserved identifier
- Line 53: nested namespaces can be concatenated
- Line 92: variable 'arena' is not initialized
- Line 101: variable 'arena' is not initialized
- Line 113: variable 'arena' is not initialized
- Line 45: nested namespaces can be concatenated
- Line 55: do not declare C-style arrays, use std::array<> instead
- Line 63: declaration uses identifier '_Enum_default_instance_', which is a reserved identifier
- Line 66: declaration uses identifier '_EnumValue_default_instance_', which is a reserved identifier
- Line 69: declaration uses identifier '_Field_default_instance_', which is a reserved identifier
- Line 72: declaration uses identifier '_Option_default_instance_', which is a reserved identifier
- Line 75: declaration uses identifier '_Type_default_instance_', which is a reserved identifier
- Line 79: nested namespaces can be concatenated
- Line 255: do not use reinterpret_cast
- Line 537: do not use reinterpret_cast
- Line 894: do not use reinterpret_cast
- Line 1146: do not use reinterpret_cast
- Line 1342: do not use reinterpret_cast
- Line 1625: do not use reinterpret_cast
- Line 39: declaration uses identifier 'GOOGLE_PROTOBUF_UNKNOWN_FIELD_SET_H__', which is a reserved identifier
- Line 62: nested namespaces can be concatenated
- Line 220: narrowing conversion from 'uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 39: nested namespaces can be concatenated
- Line 34: declaration uses identifier 'GOOGLE_PROTOBUF_UTIL_TYPE_RESOLVER_H__', which is a reserved identifier
- Line 46: nested namespaces can be concatenated
- Line 41: declaration uses identifier 'GOOGLE_PROTOBUF_WIRE_FORMAT_LITE_H__', which is a reserved identifier
- Line 74: nested namespaces can be concatenated
- Line 210: function-like macro 'GOOGLE_PROTOBUF_WIRE_FORMAT_MAKE_TAG' used; consider a 'constexpr' template function
- Line 950: variable 'temp' is not initialized
- Line 970: variable 'temp' is not initialized
- Line 978: variable 'temp' is not initialized
- Line 999: variable 'temp' is not initialized
- Line 1008: variable 'temp' is not initialized
- Line 1016: variable 'temp' is not initialized
- Line 1024: variable 'temp' is not initialized
- Line 1032: variable 'temp' is not initialized
- Line 1040: variable 'temp' is not initialized
- Line 1062: variable 'temp' is not initialized
- Line 1071: variable 'temp' is not initialized
- Line 1080: variable 'temp' is not initialized
- Line 1089: variable 'temp' is not initialized
- Line 1129: variable 'void_pointer' is not initialized
- Line 1130: variable 'size' is not initialized
- Line 1133: use auto when initializing with a cast to avoid duplicating the type name
- Line 1133: do not use reinterpret_cast
- Line 1204: variable 'length' is not initialized
- Line 1235: do not use reinterpret_cast
- Line 1316: variable 'length' is not initialized
- Line 1492: do not use pointer arithmetic
- Line 1902: do not use reinterpret_cast
- Line 1927: use auto when initializing with a cast to avoid duplicating the type name
- Line 1930: do not use reinterpret_cast
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
test/apps/ers_test.cxx
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 64: Using C-style cast. Use static_cast
(...) instead - Line 78: "cout" should not be used for output in DUNE DAQ software.
- Line 110: Add #include
for unique_ptr<>
Warnings
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 89: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 98: macro argument should be enclosed in parentheses
- Line 47: using declarations in the global namespace in headers are prohibited
- Line 112: redundant get() call on smart pointer
- Line 158: use nullptr
- Line 177: use emplace_back instead of push_back
- Line 209: use auto when declaring iterators
- Line 236: do not declare C-style arrays, use std::array<> instead
- Line 241: C-style casts are discouraged; use static_cast
- Line 242: do not use pointer arithmetic
- Line 38: using declarations in the global namespace in headers are prohibited
- Line 76: use nullptr
- Line 131: do not declare C-style arrays, use std::array<> instead
- Line 88: redundant access specifier has the same accessibility as the previous access specifier
- Line 117: do not declare C-style arrays, use std::array<> instead
- Line 118: do not declare C-style arrays, use std::array<> instead
- Line 140: macro argument should be enclosed in parentheses
- Line 56: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 64: macro argument should be enclosed in parentheses
- Line 95: macro argument should be enclosed in parentheses
- Line 30: macro argument should be enclosed in parentheses
- Line 38: macro argument should be enclosed in parentheses
- Line 41: macro argument should be enclosed in parentheses
- Line 49: macro argument should be enclosed in parentheses
- Line 57: macro argument should be enclosed in parentheses
- Line 65: macro argument should be enclosed in parentheses
- Line 73: macro argument should be enclosed in parentheses
- Line 30: throw expression should throw anonymous temporary values instead
- Line 34: throw expression should throw anonymous temporary values instead
- Line 39: throw expression should throw anonymous temporary values instead
- Line 57: throwing an exception whose type 'UnhandledException' is not derived from 'std::exception'
- Line 89: prefer a lambda to std::bind
- Line 112: prefer a lambda to std::bind
- Line 120: prefer a lambda to std::bind
- Line 121: prefer a lambda to std::bind
- Line 122: prefer a lambda to std::bind
- Line 123: prefer a lambda to std::bind
- Line 132: do not use pointer arithmetic