wibmod
Unit Tests
No unit tests written.
Errors in Documentation Links
docs/README.md
- [ERROR] file:///home/runner/work/wibmod/wibmod/docs/schema/wibmod/wibconfigurator.jsonnet File not found. Check if file exists and path is correct
Clang Formatting
| File | Status |
|---|---|
| include/wibmod/Issues.hpp | ❌ Needs formatting |
| plugins/WIBModule.cpp | ❌ Needs formatting |
| plugins/WIBModule.hpp | ❌ Needs formatting |
| src/WIB1/ADC_ASIC_reg_mapping.cpp | ❌ Needs formatting |
| src/WIB1/ASIC_reg_mapping.cpp | ❌ Needs formatting |
| src/WIB1/AddressTable.cpp | ❌ Needs formatting |
| src/WIB1/AddressTable_IO.cpp | ❌ Needs formatting |
| src/WIB1/AddressTable_fileIO.cpp | ❌ Needs formatting |
| src/WIB1/AddressTable_item.cpp | ❌ Needs formatting |
| src/WIB1/AddressTable_search.cpp | ❌ Needs formatting |
| src/WIB1/BNL_UDP.cpp | ❌ Needs formatting |
| src/WIB1/BUException/ExceptionBase.cpp | ❌ Needs formatting |
| src/WIB1/BUException/ExceptionBase_stacktrace.cpp | ❌ Needs formatting |
| src/WIB1/FE_ASIC_reg_mapping.cpp | ❌ Needs formatting |
| src/WIB1/WIB.cpp | ❌ Needs formatting |
| src/WIB1/WIBBase.cpp | ❌ Needs formatting |
| src/WIB1/WIB_CDS.cpp | ❌ Needs formatting |
| src/WIB1/WIB_DTS.cpp | ❌ Needs formatting |
| src/WIB1/WIB_FAKE_CD.cpp | ❌ Needs formatting |
| src/WIB1/WIB_FEMB.cpp | ❌ Needs formatting |
| src/WIB1/WIB_Flash.cpp | ❌ Needs formatting |
| src/WIB1/WIB_History.cpp | ❌ Needs formatting |
| src/WIB1/WIB_QSFP.cpp | ❌ Needs formatting |
| src/WIB1/WIB_SI5342.cpp | ❌ Needs formatting |
| src/WIB1/WIB_SI5344.cpp | ❌ Needs formatting |
| src/WIB1/WIB_localFlash.cpp | ❌ Needs formatting |
| src/WIB1/WIB_spybuffer.cpp | ❌ Needs formatting |
| src/WIB1/conversions.cpp | ❌ Needs formatting |
| src/WIBCommon.cpp | ❌ Needs formatting |
| src/wibmod/WIBCommon.hpp | ❌ Needs formatting |
include/wibmod/Issues.hpp
Errors
- Line 49: Using C-style cast. Use static_cast
(...) instead - Line 76: An unsigned integer appears to be used here.
plugins/WIBModule.cpp
Errors
- Line 219: Use operator && instead of and
- Line 253: Use of NULL #define found; prefer using the nullptr keyword
- Line 133: Add #include
for unique_ptr<>
Warnings
- Line 20: macro 'TLVL_ENTER_EXIT_METHODS' used to declare a constant; consider using a 'constexpr' constant
- Line 22: macro 'TLVL_GENERATION' used to declare a constant; consider using a 'constexpr' constant
- Line 24: macro 'TLVL_CANDIDATE' used to declare a constant; consider using a 'constexpr' constant
- Line 33: nested namespaces can be concatenated
- Line 111: comparison of integers of different signs: 'int' and 'size_type' (aka 'unsigned long')
- Line 119: comparison of integers of different signs: 'int' and 'size_type' (aka 'unsigned long')
- Line 153: narrowing conversion from '::uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 166: narrowing conversion from '::uint32_t' (aka 'unsigned int') to signed type 'int' is implementation-defined
- Line 190: use auto when initializing with new to avoid duplicating the type name
- Line 202: use auto when initializing with new to avoid duplicating the type name
- Line 253: use nullptr
- Line 30: nested namespaces can be concatenated
- Line 44: nested namespaces can be concatenated
- Line 54: do not declare C-style arrays, use std::array<> instead
- Line 61: declaration uses identifier '_CDFastCmd_default_instance_', which is a reserved identifier
- Line 64: declaration uses identifier '_CDPeek_default_instance_', which is a reserved identifier
- Line 67: declaration uses identifier '_CDPoke_default_instance_', which is a reserved identifier
- Line 70: declaration uses identifier '_CDRegValue_default_instance_', which is a reserved identifier
- Line 73: declaration uses identifier '_Calibrate_default_instance_', which is a reserved identifier
- Line 76: declaration uses identifier '_Command_default_instance_', which is a reserved identifier
- Line 79: declaration uses identifier '_ConfigurePower_default_instance_', which is a reserved identifier
- Line 82: declaration uses identifier '_ConfigureWIB_default_instance_', which is a reserved identifier
- Line 85: declaration uses identifier '_ConfigureWIB_ConfigureCOLDADC_default_instance_', which is a reserved identifier
- Line 88: declaration uses identifier '_ConfigureWIB_ConfigureFEMB_default_instance_', which is a reserved identifier
- Line 91: declaration uses identifier '_ConfigureWIB_ConfigureWIBPulser_default_instance_', which is a reserved identifier
- Line 94: declaration uses identifier '_Empty_default_instance_', which is a reserved identifier
- Line 97: declaration uses identifier '_GetSWVersion_default_instance_', which is a reserved identifier
- Line 100: declaration uses identifier '_GetSWVersion_Version_default_instance_', which is a reserved identifier
- Line 103: declaration uses identifier '_GetSensors_default_instance_', which is a reserved identifier
- Line 106: declaration uses identifier '_GetSensors_Sensors_default_instance_', which is a reserved identifier
- Line 109: declaration uses identifier '_GetTimestamp_default_instance_', which is a reserved identifier
- Line 112: declaration uses identifier '_GetTimestamp_Timestamp_default_instance_', which is a reserved identifier
- Line 115: declaration uses identifier '_GetTimingStatus_default_instance_', which is a reserved identifier
- Line 118: declaration uses identifier '_GetTimingStatus_TimingStatus_default_instance_', which is a reserved identifier
- Line 121: declaration uses identifier '_LogControl_default_instance_', which is a reserved identifier
- Line 124: declaration uses identifier '_LogControl_Log_default_instance_', which is a reserved identifier
- Line 127: declaration uses identifier '_Peek_default_instance_', which is a reserved identifier
- Line 130: declaration uses identifier '_Poke_default_instance_', which is a reserved identifier
- Line 133: declaration uses identifier '_PowerWIB_default_instance_', which is a reserved identifier
- Line 136: declaration uses identifier '_ReadDaqSpy_default_instance_', which is a reserved identifier
- Line 139: declaration uses identifier '_ReadDaqSpy_DaqSpy_default_instance_', which is a reserved identifier
- Line 142: declaration uses identifier '_ReadDaqSpy_DeframedDaqSpy_default_instance_', which is a reserved identifier
- Line 145: declaration uses identifier '_Reboot_default_instance_', which is a reserved identifier
- Line 148: declaration uses identifier '_Recompile_default_instance_', which is a reserved identifier
- Line 151: declaration uses identifier '_RegValue_default_instance_', which is a reserved identifier
- Line 154: declaration uses identifier '_ResetTiming_default_instance_', which is a reserved identifier
- Line 157: declaration uses identifier '_Script_default_instance_', which is a reserved identifier
- Line 160: declaration uses identifier '_SetFakeTime_default_instance_', which is a reserved identifier
- Line 163: declaration uses identifier '_StartFakeTime_default_instance_', which is a reserved identifier
- Line 166: declaration uses identifier '_Status_default_instance_', which is a reserved identifier
- Line 169: declaration uses identifier '_Update_default_instance_', which is a reserved identifier
- Line 171: nested namespaces can be concatenated
- Line 235: do not use reinterpret_cast
- Line 265: default arguments on virtual or override methods are prohibited
- Line 292: redundant access specifier has the same accessibility as the previous access specifier
- Line 330: declaration uses identifier '_Internal', which is a reserved identifier
- Line 399: do not use reinterpret_cast
- Line 429: default arguments on virtual or override methods are prohibited
- Line 456: redundant access specifier has the same accessibility as the previous access specifier
- Line 506: declaration uses identifier '_Internal', which is a reserved identifier
- Line 575: do not use reinterpret_cast
- Line 605: default arguments on virtual or override methods are prohibited
- Line 632: redundant access specifier has the same accessibility as the previous access specifier
- Line 699: declaration uses identifier '_Internal', which is a reserved identifier
- Line 769: do not use reinterpret_cast
- Line 799: default arguments on virtual or override methods are prohibited
- Line 826: redundant access specifier has the same accessibility as the previous access specifier
- Line 926: declaration uses identifier '_Internal', which is a reserved identifier
- Line 999: do not use reinterpret_cast
- Line 1029: default arguments on virtual or override methods are prohibited
- Line 1056: redundant access specifier has the same accessibility as the previous access specifier
- Line 1158: declaration uses identifier '_Internal', which is a reserved identifier
- Line 1232: do not use reinterpret_cast
- Line 1262: default arguments on virtual or override methods are prohibited
- Line 1289: redundant access specifier has the same accessibility as the previous access specifier
- Line 1377: declaration uses identifier '_Internal', which is a reserved identifier
- Line 1450: do not use reinterpret_cast
- Line 1480: default arguments on virtual or override methods are prohibited
- Line 1507: redundant access specifier has the same accessibility as the previous access specifier
- Line 1595: declaration uses identifier '_Internal', which is a reserved identifier
- Line 1668: do not use reinterpret_cast
- Line 1698: default arguments on virtual or override methods are prohibited
- Line 1725: redundant access specifier has the same accessibility as the previous access specifier
- Line 1939: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2023: do not use reinterpret_cast
- Line 2053: default arguments on virtual or override methods are prohibited
- Line 2080: redundant access specifier has the same accessibility as the previous access specifier
- Line 2190: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2265: do not use reinterpret_cast
- Line 2295: default arguments on virtual or override methods are prohibited
- Line 2322: redundant access specifier has the same accessibility as the previous access specifier
- Line 2407: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2479: do not use reinterpret_cast
- Line 2509: default arguments on virtual or override methods are prohibited
- Line 2536: redundant access specifier has the same accessibility as the previous access specifier
- Line 2668: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2743: do not use reinterpret_cast
- Line 2773: default arguments on virtual or override methods are prohibited
- Line 2784: redundant access specifier has the same accessibility as the previous access specifier
- Line 2806: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2868: do not use reinterpret_cast
- Line 2898: default arguments on virtual or override methods are prohibited
- Line 2909: redundant access specifier has the same accessibility as the previous access specifier
- Line 2931: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2994: do not use reinterpret_cast
- Line 3024: default arguments on virtual or override methods are prohibited
- Line 3051: redundant access specifier has the same accessibility as the previous access specifier
- Line 3107: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3175: do not use reinterpret_cast
- Line 3205: default arguments on virtual or override methods are prohibited
- Line 3216: redundant access specifier has the same accessibility as the previous access specifier
- Line 3238: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3301: do not use reinterpret_cast
- Line 3331: default arguments on virtual or override methods are prohibited
- Line 3358: redundant access specifier has the same accessibility as the previous access specifier
- Line 3391: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3459: do not use reinterpret_cast
- Line 3489: default arguments on virtual or override methods are prohibited
- Line 3516: redundant access specifier has the same accessibility as the previous access specifier
- Line 3560: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3629: do not use reinterpret_cast
- Line 3659: default arguments on virtual or override methods are prohibited
- Line 3686: redundant access specifier has the same accessibility as the previous access specifier
- Line 3774: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3847: do not use reinterpret_cast
- Line 3877: default arguments on virtual or override methods are prohibited
- Line 3904: redundant access specifier has the same accessibility as the previous access specifier
- Line 3981: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4053: do not use reinterpret_cast
- Line 4083: default arguments on virtual or override methods are prohibited
- Line 4110: redundant access specifier has the same accessibility as the previous access specifier
- Line 4143: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4211: do not use reinterpret_cast
- Line 4241: default arguments on virtual or override methods are prohibited
- Line 4268: redundant access specifier has the same accessibility as the previous access specifier
- Line 4532: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4612: do not use reinterpret_cast
- Line 4642: default arguments on virtual or override methods are prohibited
- Line 4653: redundant access specifier has the same accessibility as the previous access specifier
- Line 4677: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4740: do not use reinterpret_cast
- Line 4770: default arguments on virtual or override methods are prohibited
- Line 4797: redundant access specifier has the same accessibility as the previous access specifier
- Line 4896: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4969: do not use reinterpret_cast
- Line 4999: default arguments on virtual or override methods are prohibited
- Line 5010: redundant access specifier has the same accessibility as the previous access specifier
- Line 5034: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5097: do not use reinterpret_cast
- Line 5127: default arguments on virtual or override methods are prohibited
- Line 5154: redundant access specifier has the same accessibility as the previous access specifier
- Line 5193: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5260: do not use reinterpret_cast
- Line 5290: default arguments on virtual or override methods are prohibited
- Line 5301: redundant access specifier has the same accessibility as the previous access specifier
- Line 5325: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5387: do not use reinterpret_cast
- Line 5417: default arguments on virtual or override methods are prohibited
- Line 5428: redundant access specifier has the same accessibility as the previous access specifier
- Line 5450: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5513: do not use reinterpret_cast
- Line 5543: default arguments on virtual or override methods are prohibited
- Line 5570: redundant access specifier has the same accessibility as the previous access specifier
- Line 5647: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5718: do not use reinterpret_cast
- Line 5748: default arguments on virtual or override methods are prohibited
- Line 5759: redundant access specifier has the same accessibility as the previous access specifier
- Line 5783: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5846: do not use reinterpret_cast
- Line 5876: default arguments on virtual or override methods are prohibited
- Line 5903: redundant access specifier has the same accessibility as the previous access specifier
- Line 5936: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6003: do not use reinterpret_cast
- Line 6033: default arguments on virtual or override methods are prohibited
- Line 6044: redundant access specifier has the same accessibility as the previous access specifier
- Line 6066: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6129: do not use reinterpret_cast
- Line 6159: default arguments on virtual or override methods are prohibited
- Line 6186: redundant access specifier has the same accessibility as the previous access specifier
- Line 6225: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6293: do not use reinterpret_cast
- Line 6323: default arguments on virtual or override methods are prohibited
- Line 6350: redundant access specifier has the same accessibility as the previous access specifier
- Line 6407: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6476: do not use reinterpret_cast
- Line 6506: default arguments on virtual or override methods are prohibited
- Line 6517: redundant access specifier has the same accessibility as the previous access specifier
- Line 6539: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6602: do not use reinterpret_cast
- Line 6632: default arguments on virtual or override methods are prohibited
- Line 6659: redundant access specifier has the same accessibility as the previous access specifier
- Line 6709: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6778: do not use reinterpret_cast
- Line 6808: default arguments on virtual or override methods are prohibited
- Line 6835: redundant access specifier has the same accessibility as the previous access specifier
- Line 6879: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6948: do not use reinterpret_cast
- Line 6978: default arguments on virtual or override methods are prohibited
- Line 7005: redundant access specifier has the same accessibility as the previous access specifier
- Line 7093: declaration uses identifier '_Internal', which is a reserved identifier
- Line 7139: do not use reinterpret_cast
- Line 7148: do not use reinterpret_cast
- Line 7150: do not use reinterpret_cast
- Line 7191: do not use reinterpret_cast
- Line 7204: do not use reinterpret_cast
- Line 7209: do not use reinterpret_cast
- Line 7218: do not use reinterpret_cast
- Line 8890: do not use reinterpret_cast
- Line 8899: do not use reinterpret_cast
- Line 8901: do not use reinterpret_cast
- Line 8942: do not use reinterpret_cast
- Line 8955: do not use reinterpret_cast
- Line 8960: do not use reinterpret_cast
- Line 8969: do not use reinterpret_cast
- Line 9030: do not use reinterpret_cast
- Line 9039: do not use reinterpret_cast
- Line 9041: do not use reinterpret_cast
- Line 9082: do not use reinterpret_cast
- Line 9095: do not use reinterpret_cast
- Line 9100: do not use reinterpret_cast
- Line 9109: do not use reinterpret_cast
- Line 21: nested namespaces can be concatenated
plugins/WIBModule.hpp
Errors
- Line 14: #ifndef header guard has wrong style, please use: WIBMOD_PLUGINS_WIBMODULE_HPP_
- Line 79: #endif line should be "#endif // WIBMOD_PLUGINS_WIBMODULE_HPP_"
- Line 19: Include the directory when naming .hh files
src/WIB1/ADC_ASIC_reg_mapping.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 8: An unsigned integer appears to be used here.
- Line 9: An unsigned integer appears to be used here.
- Line 11: An unsigned integer appears to be used here.
- Line 12: An unsigned integer appears to be used here.
- Line 13: An unsigned integer appears to be used here.
- Line 14: An unsigned integer appears to be used here.
- Line 15: An unsigned integer appears to be used here.
- Line 18: Use int16/int64/etc, rather than the C type long
- Line 27: An unsigned integer appears to be used here.
- Line 28: An unsigned integer appears to be used here.
- Line 29: An unsigned integer appears to be used here.
- Line 30: An unsigned integer appears to be used here.
- Line 31: An unsigned integer appears to be used here.
- Line 57: An unsigned integer appears to be used here.
- Line 58: An unsigned integer appears to be used here.
- Line 59: An unsigned integer appears to be used here.
- Line 60: An unsigned integer appears to be used here.
- Line 61: An unsigned integer appears to be used here.
- Line 62: An unsigned integer appears to be used here.
- Line 63: An unsigned integer appears to be used here.
- Line 64: An unsigned integer appears to be used here.
- Line 76: An unsigned integer appears to be used here.
- Line 77: An unsigned integer appears to be used here.
- Line 78: An unsigned integer appears to be used here.
- Line 79: An unsigned integer appears to be used here.
- Line 80: An unsigned integer appears to be used here.
- Line 81: An unsigned integer appears to be used here.
- Line 82: An unsigned integer appears to be used here.
- Line 101: "cout" should not be used for output in DUNE DAQ software.
- Line 109: "cout" should not be used for output in DUNE DAQ software.
- Line 111: "cout" should not be used for output in DUNE DAQ software.
- Line 113: "cout" should not be used for output in DUNE DAQ software.
- Line 102: Add #include
for string
Warnings
- Line 18: consider replacing 'unsigned long' with 'uint64'
- Line 2: declaration uses identifier '__ADC_ASIC_REG_MAPPING_HH__', which is a reserved identifier
src/WIB1/ASIC_reg_mapping.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 30: An unsigned integer appears to be used here.
- Line 36: An unsigned integer appears to be used here.
- Line 43: "cout" should not be used for output in DUNE DAQ software.
- Line 45: An unsigned integer appears to be used here.
- Line 47: An unsigned integer appears to be used here.
- Line 48: "cout" should not be used for output in DUNE DAQ software.
- Line 36: Add #include
for vector<>
Warnings
- Line 2: declaration uses identifier '__ADC_ASIC_REG_MAPPING_HH__', which is a reserved identifier
- Line 2: declaration uses identifier '__ASIC_REG_MAPPING_HH__', which is a reserved identifier
- Line 2: declaration uses identifier '__FE_ASIC_REG_MAPPING_HH__', which is a reserved identifier
- Line 31: do not declare C-style arrays, use std::array<> instead
src/WIB1/AddressTable.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 3: Found other header after system header. Should be: AddressTable.hh, other non-system headers, system headers.
- Line 10: An unsigned integer appears to be used here.
- Line 10: Add #include
for string
Warnings
- Line 5: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__ADDRESSTABLEEXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__ADDRESSTABLEEXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/AddressTable_IO.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 3: Found other header after system header. Should be: AddressTable_IO.hh, other non-system headers, system headers.
- Line 10: An unsigned integer appears to be used here.
- Line 13: An unsigned integer appears to be used here.
- Line 18: An unsigned integer appears to be used here.
- Line 21: An unsigned integer appears to be used here.
- Line 25: An unsigned integer appears to be used here.
- Line 28: An unsigned integer appears to be used here.
- Line 33: An unsigned integer appears to be used here.
- Line 43: An unsigned integer appears to be used here.
- Line 50: An unsigned integer appears to be used here.
- Line 60: An unsigned integer appears to be used here.
- Line 67: An unsigned integer appears to be used here.
- Line 78: An unsigned integer appears to be used here.
- Line 88: An unsigned integer appears to be used here.
- Line 99: An unsigned integer appears to be used here.
- Line 110: An unsigned integer appears to be used here.
- Line 113: An unsigned integer appears to be used here.
- Line 110: Add #include
for vector<> - Line 114: Add #include
for string - Line 114: Add #include
Warnings
- Line 5: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
- Line 34: use auto when declaring iterators
- Line 40: throw expression should throw anonymous temporary values instead
- Line 51: use auto when declaring iterators
- Line 57: throw expression should throw anonymous temporary values instead
- Line 68: use auto when declaring iterators
- Line 74: throw expression should throw anonymous temporary values instead
- Line 89: use auto when declaring iterators
- Line 95: throw expression should throw anonymous temporary values instead
- Line 114: use auto when declaring iterators
- Line 120: throw expression should throw anonymous temporary values instead
- Line 127: throw expression should throw anonymous temporary values instead
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__ADDRESSTABLEEXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__ADDRESSTABLEEXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/AddressTable_fileIO.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 3: Found other header after system header. Should be: AddressTable_fileIO.hh, other non-system headers, system headers.
- Line 10: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 13: An unsigned integer appears to be used here.
- Line 19: Use of NULL #define found; prefer using the nullptr keyword
- Line 33: Do not use variable-length arrays. Use an appropriately named ('k' followed by CamelCase) compile-time constant for the size.
- Line 35: An unsigned integer appears to be used here.
- Line 45: An unsigned integer appears to be used here.
- Line 94: Use of NULL #define found; prefer using the nullptr keyword
- Line 123: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 129: Use of NULL #define found; prefer using the nullptr keyword
- Line 136: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 262: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 264: Small and focused functions are preferred: AddressTable::ProcessLine() has 161 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 247: Add #include
for string
Warnings
- Line 5: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
- Line 10: macro 'MAX_FILE_LEVEL' used to declare a constant; consider using a 'constexpr' constant
- Line 19: use nullptr
- Line 29: throw expression should throw anonymous temporary values instead
- Line 33: do not declare C-style arrays, use std::array<> instead
- Line 68: string concatenation results in allocation of unnecessary temporary strings; consider using 'operator+=' or 'string::append()' instead
- Line 68: string concatenation results in allocation of unnecessary temporary strings; consider using 'operator+=' or 'string::append()' instead
- Line 81: throw expression should throw anonymous temporary values instead
- Line 94: use nullptr
- Line 100: do not use pointer arithmetic
- Line 119: throw expression should throw anonymous temporary values instead
- Line 129: use nullptr
- Line 135: C-style casts are discouraged; use static_cast
- Line 193: do not declare C-style arrays, use std::array<> instead
- Line 199: throw expression should throw anonymous temporary values instead
- Line 210: do not declare C-style arrays, use std::array<> instead
- Line 213: throw expression should throw anonymous temporary values instead
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__ADDRESSTABLEEXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__ADDRESSTABLEEXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/AddressTable_item.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 3: Found other header after system header. Should be: AddressTable_item.hh, other non-system headers, system headers.
- 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 44: An unsigned integer appears to be used here.
- Line 100: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 107: Using C-style cast. Use one of the C++ cast operators instead
- Line 110: Add #include
for string - Line 78: Add #include
- Line 86: Add #include
for vector<>
Warnings
- Line 5: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
- Line 12: use auto when declaring iterators
- Line 18: throw expression should throw anonymous temporary values instead
- Line 26: use nullptr
- Line 29: throw expression should throw anonymous temporary values instead
- Line 36: throw expression should throw anonymous temporary values instead
- Line 44: use auto when declaring iterators
- Line 78: use auto when declaring iterators
- Line 102: throw expression should throw anonymous temporary values instead
- Line 107: C-style casts are discouraged; use static_cast/const_cast/reinterpret_cast
- Line 107: use nullptr
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__ADDRESSTABLEEXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__ADDRESSTABLEEXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/AddressTable_search.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 3: Found other header after system header. Should be: AddressTable_search.hh, other non-system headers, system headers.
- Line 34: "printf" should not be used for output in DUNE DAQ software.
- Line 66: An unsigned integer appears to be used here.
- Line 69: An unsigned integer appears to be used here.
- Line 93: "printf" should not be used for output in DUNE DAQ software.
- Line 137: Add #include
for string - Line 136: Add #include
for vector<> - Line 137: Add #include
- Line 32: Add #include
for transform - Line 113: Add #include
for set<>
Warnings
- Line 98: The value '163840' provided to the cast expression is not in the valid range of values for '_match_flags'
- Line 5: inclusion of deprecated C++ header 'stdlib.h'; consider using 'cstdlib' instead
- Line 12: use auto when declaring iterators
- Line 52: throw expression should throw anonymous temporary values instead
- Line 55: use auto when declaring iterators
- Line 69: use auto when declaring iterators
- Line 111: throw expression should throw anonymous temporary values instead
- Line 115: use auto when declaring iterators
- Line 137: use auto when declaring iterators
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__ADDRESSTABLEEXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__ADDRESSTABLEEXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/BNL_UDP.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 14: "errno.h" already included at sourcecode/wibmod/src/WIB1/BNL_UDP.cpp:5
- Line 17: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 18: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 19: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 21: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 23: #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 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 30: An unsigned integer appears to be used here.
- Line 31: An unsigned integer appears to be used here.
- Line 32: An unsigned integer appears to be used here.
- Line 33: An unsigned integer appears to be used here.
- Line 34: An unsigned integer appears to be used here.
- Line 37: An unsigned integer appears to be used here.
- Line 74: Use of NULL #define found; prefer using the nullptr keyword
- Line 75: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 96: An unsigned integer appears to be used here.
- Line 117: Use of NULL #define found; prefer using the nullptr keyword
- Line 123: Use of NULL #define found; prefer using the nullptr keyword
- Line 123: An unsigned integer appears to be used here.
- Line 124: Use of NULL #define found; prefer using the nullptr keyword
- Line 124: An unsigned integer appears to be used here.
- Line 147: Use of NULL #define found; prefer using the nullptr keyword
- Line 197: An unsigned integer appears to be used here.
- Line 217: An unsigned integer appears to be used here.
- Line 227: An unsigned integer appears to be used here.
- Line 228: An unsigned integer appears to be used here.
- Line 257: Using C-style cast. Use one of the C++ cast operators instead
- Line 257: An unsigned integer appears to be used here.
- Line 268: An unsigned integer appears to be used here.
- Line 272: An unsigned integer appears to be used here.
- Line 280: An unsigned integer appears to be used here.
- Line 283: An unsigned integer appears to be used here.
- Line 370: An unsigned integer appears to be used here.
- Line 371: An unsigned integer appears to be used here.
- Line 391: An unsigned integer appears to be used here.
- Line 427: Using C-style cast. Use one of the C++ cast operators instead
- Line 427: An unsigned integer appears to be used here.
- Line 438: An unsigned integer appears to be used here.
- Line 442: An unsigned integer appears to be used here.
- Line 451: An unsigned integer appears to be used here.
- Line 452: An unsigned integer appears to be used here.
- Line 453: An unsigned integer appears to be used here.
- Line 454: An unsigned integer appears to be used here.
- Line 469: Use of NULL #define found; prefer using the nullptr keyword
- Line 470: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 472: An unsigned integer appears to be used here.
- Line 280: Add #include
for vector<>
Warnings
- Line 4: inclusion of deprecated C++ header 'string.h'; consider using 'cstring' instead
- Line 5: inclusion of deprecated C++ header 'errno.h'; consider using 'cerrno' instead
- Line 14: inclusion of deprecated C++ header 'errno.h'; consider using 'cerrno' instead
- Line 17: macro 'WIB_WR_BASE_PORT' used to declare a constant; consider using a 'constexpr' constant
- Line 18: macro 'WIB_RD_BASE_PORT' used to declare a constant; consider using a 'constexpr' constant
- Line 19: macro 'WIB_RPLY_BASE_PORT' used to declare a constant; consider using a 'constexpr' constant
- Line 21: macro 'WIB_PACKET_KEY' used to declare a constant; consider using a 'constexpr' constant
- Line 23: macro 'WIB_REQUEST_PACKET_TRAILER' used to declare a constant; consider using a 'constexpr' constant
- Line 25: macro 'WIB_RPLY_PACKET_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 27: macro 'TIMEOUT_SECONDS' used to declare a constant; consider using a 'constexpr' constant
- Line 28: macro 'TIMEOUT_MICROSECONDS' used to declare a constant; consider using a 'constexpr' constant
- Line 42: C-style casts are discouraged; use static_cast
- Line 42: do not use pointer arithmetic
- Line 45: C-style casts are discouraged; use static_cast
- Line 45: do not use pointer arithmetic
- Line 56: variable 'ret' is not initialized
- Line 58: narrowing conversion from 'ssize_t' (aka 'long') to signed type 'int' is implementation-defined
- Line 74: use nullptr
- Line 106: throw expression should throw anonymous temporary values instead
- Line 116: variable 'res' is not initialized
- Line 117: use nullptr
- Line 117: use nullptr
- Line 123: use nullptr
- Line 124: use nullptr
- Line 134: narrowing conversion from 'int' to signed type 'char' is implementation-defined
- Line 134: an integer is interpreted as a character code when assigning it to a string; if this is intended, cast the integer to the appropriate character type; if you want a string representation, use the appropriate conversion facility
- Line 141: narrowing conversion from 'int' to signed type 'char' is implementation-defined
- Line 141: an integer is interpreted as a character code when assigning it to a string; if this is intended, cast the integer to the appropriate character type; if you want a string representation, use the appropriate conversion facility
- Line 147: use nullptr
- Line 147: use nullptr
- Line 152: throw expression should throw anonymous temporary values instead
- Line 162: throw expression should throw anonymous temporary values instead
- Line 167: throw expression should throw anonymous temporary values instead
- Line 171: do not use C-style cast to convert between unrelated types
- Line 171: C-style casts are discouraged; use reinterpret_cast
- Line 172: do not use C-style cast to convert between unrelated types
- Line 172: C-style casts are discouraged; use reinterpret_cast
- Line 175: do not use C-style cast to convert between unrelated types
- Line 175: C-style casts are discouraged; use reinterpret_cast
- Line 177: do not use C-style cast to convert between unrelated types
- Line 177: C-style casts are discouraged; use reinterpret_cast
- Line 181: throw expression should throw anonymous temporary values instead
- Line 184: do not use C-style cast to convert between unrelated types
- Line 184: C-style casts are discouraged; use reinterpret_cast
- Line 186: do not use C-style cast to convert between unrelated types
- Line 186: C-style casts are discouraged; use reinterpret_cast
- Line 190: throw expression should throw anonymous temporary values instead
- Line 243: throw expression should throw anonymous temporary values instead
- Line 257: do not use C-style cast to convert between unrelated types
- Line 257: C-style casts are discouraged; use reinterpret_cast
- Line 258: throw expression should throw anonymous temporary values instead
- Line 266: throw expression should throw anonymous temporary values instead
- Line 268: use auto when initializing with a cast to avoid duplicating the type name
- Line 268: C-style casts are discouraged; use static_cast
- Line 268: do not use pointer arithmetic
- Line 268: do not use pointer arithmetic
- Line 272: C-style casts are discouraged; use static_cast
- Line 272: C-style casts are discouraged; use static_cast
- Line 276: throw expression should throw anonymous temporary values instead
- Line 285: do not use pointer arithmetic
- Line 371: variable 'val' is not initialized
- Line 414: throw expression should throw anonymous temporary values instead
- Line 421: C-style casts are discouraged; use static_cast
- Line 427: do not use C-style cast to convert between unrelated types
- Line 427: C-style casts are discouraged; use reinterpret_cast
- Line 428: throw expression should throw anonymous temporary values instead
- Line 436: throw expression should throw anonymous temporary values instead
- Line 438: use auto when initializing with a cast to avoid duplicating the type name
- Line 438: C-style casts are discouraged; use static_cast
- Line 438: do not use pointer arithmetic
- Line 438: do not use pointer arithmetic
- Line 442: C-style casts are discouraged; use static_cast
- Line 442: C-style casts are discouraged; use static_cast
- Line 446: throw expression should throw anonymous temporary values instead
- Line 451: C-style casts are discouraged; use static_cast
- Line 451: do not use pointer arithmetic
- Line 452: C-style casts are discouraged; use static_cast
- Line 452: do not use pointer arithmetic
- Line 453: C-style casts are discouraged; use static_cast
- Line 453: do not use pointer arithmetic
- Line 454: C-style casts are discouraged; use static_cast
- Line 454: do not use pointer arithmetic
- Line 469: use nullptr
- Line 470: Attempt to free released memory
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__BNL_UDP_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__BNL_UDP_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/BUException/ExceptionBase.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 11: Using C-style cast. Use one of the C++ cast operators instead
- Line 12: Using C-style cast. Use one of the C++ cast operators instead
- Line 19: Use of NULL #define found; prefer using the nullptr keyword
- Line 28: Use of NULL #define found; prefer using the nullptr keyword
- Line 43: Use of NULL #define found; prefer using the nullptr keyword
- Line 45: Use of NULL #define found; prefer using the nullptr keyword
- Line 55: Use of NULL #define found; prefer using the nullptr keyword
- Line 58: Use of NULL #define found; prefer using the nullptr keyword
- Line 63: Using C-style cast. Use one of the C++ cast operators instead
- Line 64: Use of NULL #define found; prefer using the nullptr keyword
- Line 75: Use of NULL #define found; prefer using the nullptr keyword
- Line 78: Use of NULL #define found; prefer using the nullptr keyword
- Line 82: Using C-style cast. Use one of the C++ cast operators instead
- Line 83: Use of NULL #define found; prefer using the nullptr keyword
- Line 116: Use of NULL #define found; prefer using the nullptr keyword
- Line 122: Use of NULL #define found; prefer using the nullptr keyword
Warnings
- Line 4: do not declare C-style arrays, use std::array<> instead
- Line 11: do not use C-style cast to convert between unrelated types
- Line 11: C-style casts are discouraged; use static_cast
- Line 11: do not manage memory manually; consider a container or a smart pointer
- Line 12: do not use C-style cast to convert between unrelated types
- Line 12: C-style casts are discouraged; use static_cast
- Line 12: do not manage memory manually; consider a container or a smart pointer
- Line 19: use nullptr
- Line 21: do not use pointer arithmetic
- Line 22: do not use pointer arithmetic
- Line 28: use nullptr
- Line 30: do not use pointer arithmetic
- Line 31: do not use pointer arithmetic
- Line 43: use nullptr
- Line 44: do not manage memory manually; use RAII
- Line 45: use nullptr
- Line 46: do not manage memory manually; use RAII
- Line 55: use nullptr
- Line 58: use nullptr
- Line 63: do not use C-style cast to convert between unrelated types
- Line 63: C-style casts are discouraged; use static_cast
- Line 63: do not manage memory manually; consider a container or a smart pointer
- Line 64: use nullptr
- Line 75: use nullptr
- Line 78: use nullptr
- Line 82: do not use C-style cast to convert between unrelated types
- Line 82: C-style casts are discouraged; use static_cast
- Line 82: do not manage memory manually; consider a container or a smart pointer
- Line 83: use nullptr
- Line 106: do not use pointer arithmetic
- Line 116: use nullptr
- Line 117: do not use pointer arithmetic
- Line 122: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/BUException/ExceptionBase_stacktrace.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 8: Use of NULL #define found; prefer using the nullptr keyword
- Line 16: Use of NULL #define found; prefer using the nullptr keyword
- Line 34: Use of NULL #define found; prefer using the nullptr keyword
- Line 56: Use of NULL #define found; prefer using the nullptr keyword
- Line 62: Use of NULL #define found; prefer using the nullptr keyword
- Line 68: Use of NULL #define found; prefer using the nullptr keyword
- Line 74: Use of NULL #define found; prefer using the nullptr keyword
- Line 75: Use of NULL #define found; prefer using the nullptr keyword
- Line 76: Use of NULL #define found; prefer using the nullptr keyword
- Line 89: Use of NULL #define found; prefer using the nullptr keyword
- Line 97: Use of NULL #define found; prefer using the nullptr keyword
- Line 98: Use of NULL #define found; prefer using the nullptr keyword
- Line 99: Use of NULL #define found; prefer using the nullptr keyword
- Line 121: Use of NULL #define found; prefer using the nullptr keyword
- Line 138: Use of NULL #define found; prefer using the nullptr keyword
Warnings
- Line 4: do not declare C-style arrays, use std::array<> instead
- Line 8: use nullptr
- Line 16: use nullptr
- Line 24: do not use pointer arithmetic
- Line 30: do not use pointer arithmetic
- Line 34: use nullptr
- Line 36: do not use pointer arithmetic
- Line 43: narrowing conversion from 'long' to signed type 'pid_t' (aka 'int') is implementation-defined
- Line 47: do not declare C-style arrays, use std::array<> instead
- Line 48: variable 'stackSize' is not initialized
- Line 56: use nullptr
- Line 62: use nullptr
- Line 68: do not use pointer arithmetic
- Line 68: use nullptr
- Line 74: use nullptr
- Line 75: use nullptr
- Line 76: use nullptr
- Line 79: do not use pointer arithmetic
- Line 79: do not use pointer arithmetic
- Line 79: do not use pointer arithmetic
- Line 80: do not use pointer arithmetic
- Line 80: do not use pointer arithmetic
- Line 89: use nullptr
- Line 97: use nullptr
- Line 98: use nullptr
- Line 99: use nullptr
- Line 104: do not use pointer arithmetic
- Line 109: do not use pointer arithmetic
- Line 121: use nullptr
- Line 134: do not use pointer arithmetic
- Line 138: use nullptr
- Line 139: do not manage memory manually; use RAII
- Line 140: do not manage memory manually; use RAII
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
src/WIB1/FE_ASIC_reg_mapping.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 8: An unsigned integer appears to be used here.
- Line 9: An unsigned integer appears to be used here.
- Line 11: Use int16/int64/etc, rather than the C type long
- Line 20: An unsigned integer appears to be used here.
- Line 21: An unsigned integer appears to be used here.
- Line 22: An unsigned integer appears to be used here.
- Line 24: Use int16/int64/etc, rather than the C type long
- Line 27: Use int16/int64/etc, rather than the C type long
- Line 49: An unsigned integer appears to be used here.
- Line 50: An unsigned integer appears to be used here.
- Line 51: An unsigned integer appears to be used here.
- Line 52: An unsigned integer appears to be used here.
- Line 53: An unsigned integer appears to be used here.
- Line 62: An unsigned integer appears to be used here.
- Line 63: An unsigned integer appears to be used here.
- Line 64: An unsigned integer appears to be used here.
- Line 65: An unsigned integer appears to be used here.
- Line 79: An unsigned integer appears to be used here.
- Line 97: "cout" should not be used for output in DUNE DAQ software.
- Line 105: "cout" should not be used for output in DUNE DAQ software.
- Line 107: "cout" should not be used for output in DUNE DAQ software.
- Line 109: "cout" should not be used for output in DUNE DAQ software.
- Line 116: An unsigned integer appears to be used here.
- Line 98: Add #include
for string
Warnings
- Line 11: consider replacing 'unsigned long' with 'uint64'
- Line 24: consider replacing 'unsigned long' with 'uint64'
- Line 27: consider replacing 'unsigned long' with 'uint64'
- Line 85: do not use array subscript when the index is not an integer constant expression
- Line 116: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__FE_ASIC_REG_MAPPING_HH__', which is a reserved identifier
- Line 31: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 101: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 632: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 25: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 25: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 25: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 26: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 26: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 26: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 45: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 45: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 45: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 46: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 46: Missing author name in TODO comment; it should appear on the same line as the TODO, as Firstname Lastname
- Line 46: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 51: An unsigned integer appears to be used here.
- Line 64: "printf" should not be used for output in DUNE DAQ software.
- Line 67: An unsigned integer appears to be used here.
- Line 77: An unsigned integer appears to be used here.
- Line 91: An unsigned integer appears to be used here.
- Line 169: "printf" should not be used for output in DUNE DAQ software.
- Line 197: An unsigned integer appears to be used here.
- Line 219: An unsigned integer appears to be used here.
- Line 236: "printf" should not be used for output in DUNE DAQ software.
- Line 248: If an else has a brace on one side, it should have it on both
- Line 250: "printf" should not be used for output in DUNE DAQ software.
- Line 252: "printf" should not be used for output in DUNE DAQ software.
- Line 272: An unsigned integer appears to be used here.
- Line 297: "printf" should not be used for output in DUNE DAQ software.
- Line 305: "printf" should not be used for output in DUNE DAQ software.
- Line 307: If an else has a brace on one side, it should have it on both
- Line 308: "printf" should not be used for output in DUNE DAQ software.
- Line 311: If an else has a brace on one side, it should have it on both
- Line 312: "printf" should not be used for output in DUNE DAQ software.
- Line 324: "printf" should not be used for output in DUNE DAQ software.
- Line 326: If an else has a brace on one side, it should have it on both
- Line 327: "printf" should not be used for output in DUNE DAQ software.
- Line 340: "printf" should not be used for output in DUNE DAQ software.
- Line 343: If an else has a brace on one side, it should have it on both
- Line 344: "printf" should not be used for output in DUNE DAQ software.
- Line 350: An unsigned integer appears to be used here.
- Line 366: "printf" should not be used for output in DUNE DAQ software.
- Line 378: If an else has a brace on one side, it should have it on both
- Line 380: "printf" should not be used for output in DUNE DAQ software.
- Line 382: "printf" should not be used for output in DUNE DAQ software.
- Line 396: "cout" should not be used for output in DUNE DAQ software.
- Line 400: "cout" should not be used for output in DUNE DAQ software.
- Line 409: Small and focused functions are preferred: WIB::CheckedResetWIBAndCfgDTS() has 113 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 421: An unsigned integer appears to be used here.
- Line 435: If an else has a brace on one side, it should have it on both
- Line 456: An unsigned integer appears to be used here.
- Line 509: An unsigned integer appears to be used here.
- Line 518: An unsigned integer appears to be used here.
- Line 547: An unsigned integer appears to be used here.
- Line 556: An unsigned integer appears to be used here.
- Line 585: An unsigned integer appears to be used here.
- Line 594: An unsigned integer appears to be used here.
- Line 603: An unsigned integer appears to be used here.
- Line 626: An unsigned integer appears to be used here.
- Line 628: "printf" should not be used for output in DUNE DAQ software.
- Line 639: Add #include
for string
Warnings
- Line 4: inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
- Line 57: throw expression should throw anonymous temporary values instead
- Line 83: throw expression should throw anonymous temporary values instead
- Line 173: converting integer literal to bool, use bool literal instead
- Line 200: throw expression should throw anonymous temporary values instead
- Line 205: throw expression should throw anonymous temporary values instead
- Line 210: throw expression should throw anonymous temporary values instead
- Line 215: throw expression should throw anonymous temporary values instead
- Line 226: converting integer literal to bool, use bool literal instead
- Line 240: converting integer literal to bool, use bool literal instead
- Line 275: throw expression should throw anonymous temporary values instead
- Line 280: throw expression should throw anonymous temporary values instead
- Line 285: throw expression should throw anonymous temporary values instead
- Line 290: throw expression should throw anonymous temporary values instead
- Line 356: converting integer literal to bool, use bool literal instead
- Line 370: converting integer literal to bool, use bool literal instead
- Line 414: throw expression should throw anonymous temporary values instead
- Line 513: throw expression should throw anonymous temporary values instead
- Line 539: do not declare C-style arrays, use std::array<> instead
- Line 542: throw expression should throw anonymous temporary values instead
- Line 551: throw expression should throw anonymous temporary values instead
- Line 577: do not declare C-style arrays, use std::array<> instead
- Line 580: throw expression should throw anonymous temporary values instead
- Line 589: throw expression should throw anonymous temporary values instead
- Line 598: throw expression should throw anonymous temporary values instead
- Line 618: do not declare C-style arrays, use std::array<> instead
- Line 621: throw expression should throw anonymous temporary values instead
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__BNL_UDP_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__BNL_UDP_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIBBase.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 6: Use of NULL #define found; prefer using the nullptr keyword
- Line 10: Use of NULL #define found; prefer using the nullptr keyword
- Line 21: Use of NULL #define found; prefer using the nullptr keyword
- Line 22: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 23: Use of NULL #define found; prefer using the nullptr keyword
- Line 26: Use of NULL #define found; prefer using the nullptr keyword
- Line 27: The delete operator appears to be used; owning memory should be done with a smart pointer rather than a raw pointer
- Line 28: Use of NULL #define found; prefer using the nullptr keyword
- Line 39: An unsigned integer appears to be used here.
- Line 56: "printf" should not be used for output in DUNE DAQ software.
- Line 69: An unsigned integer appears to be used here.
- Line 116: An unsigned integer appears to be used here.
- Line 119: An unsigned integer appears to be used here.
- Line 122: An unsigned integer appears to be used here.
- Line 125: An unsigned integer appears to be used here.
- Line 129: An unsigned integer appears to be used here.
- Line 132: An unsigned integer appears to be used here.
- Line 135: An unsigned integer appears to be used here.
- Line 138: An unsigned integer appears to be used here.
- Line 141: An unsigned integer appears to be used here.
- Line 144: An unsigned integer appears to be used here.
- Line 147: An unsigned integer appears to be used here.
- Line 150: An unsigned integer appears to be used here.
- Line 155: An unsigned integer appears to be used here.
- Line 164: An unsigned integer appears to be used here.
- Line 174: An unsigned integer appears to be used here.
- Line 183: An unsigned integer appears to be used here.
- Line 193: An unsigned integer appears to be used here.
- Line 200: An unsigned integer appears to be used here.
- Line 201: An unsigned integer appears to be used here.
- Line 202: An unsigned integer appears to be used here.
- Line 203: An unsigned integer appears to be used here.
- Line 209: An unsigned integer appears to be used here.
- Line 215: Using deprecated casting style. Use static_cast
(...) instead - Line 183: Add #include
for string - Line 144: Add #include
for vector<>
Warnings
- Line 6: use nullptr
- Line 10: do not use array subscript when the index is not an integer constant expression
- Line 10: use nullptr
- Line 16: do not use array subscript when the index is not an integer constant expression
- Line 21: use nullptr
- Line 23: use nullptr
- Line 26: do not use array subscript when the index is not an integer constant expression
- Line 26: use nullptr
- Line 27: do not use array subscript when the index is not an integer constant expression
- Line 28: do not use array subscript when the index is not an integer constant expression
- Line 28: use nullptr
- Line 59: do not declare C-style arrays, use std::array<> instead
- Line 65: throw expression should throw anonymous temporary values instead
- Line 92: do not declare C-style arrays, use std::array<> instead
- Line 98: throw expression should throw anonymous temporary values instead
- Line 111: throw expression should throw anonymous temporary values instead
- Line 113: do not use array subscript when the index is not an integer constant expression
- Line 159: throw expression should throw anonymous temporary values instead
- Line 161: do not use array subscript when the index is not an integer constant expression
- Line 162: narrowing conversion from 'double' to '__useconds_t' (aka 'unsigned int')
- Line 162: C-style casts are discouraged; use static_cast
- Line 168: throw expression should throw anonymous temporary values instead
- Line 170: do not use array subscript when the index is not an integer constant expression
- Line 171: narrowing conversion from 'double' to '__useconds_t' (aka 'unsigned int')
- Line 171: C-style casts are discouraged; use static_cast
- Line 178: throw expression should throw anonymous temporary values instead
- Line 180: do not use array subscript when the index is not an integer constant expression
- Line 181: narrowing conversion from 'double' to '__useconds_t' (aka 'unsigned int')
- Line 181: C-style casts are discouraged; use static_cast
- Line 187: throw expression should throw anonymous temporary values instead
- Line 189: do not use array subscript when the index is not an integer constant expression
- Line 190: narrowing conversion from 'double' to '__useconds_t' (aka 'unsigned int')
- Line 190: C-style casts are discouraged; use static_cast
- Line 197: throw expression should throw anonymous temporary values instead
- Line 202: do not use array subscript when the index is not an integer constant expression
- Line 205: do not use array subscript when the index is not an integer constant expression
- Line 206: narrowing conversion from 'double' to '__useconds_t' (aka 'unsigned int')
- Line 206: C-style casts are discouraged; use static_cast
- Line 213: throw expression should throw anonymous temporary values instead
- Line 215: C-style casts are discouraged; use static_cast
- Line 215: do not use array subscript when the index is not an integer constant expression
- Line 216: do not use array subscript when the index is not an integer constant expression
- Line 217: narrowing conversion from 'double' to '__useconds_t' (aka 'unsigned int')
- Line 217: C-style casts are discouraged; use static_cast
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__BNL_UDP_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__BNL_UDP_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_CDS.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 4: Found other header after system header. Should be: WIB_CDS.hh, other non-system headers, system headers.
- Line 6: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 17: An unsigned integer appears to be used here.
- Line 20: An unsigned integer appears to be used here.
- Line 24: An unsigned integer appears to be used here.
- Line 49: An unsigned integer appears to be used here.
- Line 51: An unsigned integer appears to be used here.
Warnings
- Line 6: macro 'COURSE_DATARATE_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 7: do not declare C-style arrays, use std::array<> instead
- Line 28: throw expression should throw anonymous temporary values instead
- Line 49: C-style casts are discouraged; use static_cast
- Line 51: C-style casts are discouraged; use static_cast
- Line 55: do not use array subscript when the index is not an integer constant expression
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_DTS.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 12: An unsigned integer appears to be used here.
- Line 27: "printf" should not be used for output in DUNE DAQ software.
- Line 37: An unsigned integer appears to be used here.
- Line 38: An unsigned integer appears to be used here.
- Line 39: "printf" should not be used for output in DUNE DAQ software.
- Line 40: "printf" should not be used for output in DUNE DAQ software.
- Line 50: "printf" should not be used for output in DUNE DAQ software.
- Line 55: "printf" should not be used for output in DUNE DAQ software.
- Line 112: "printf" should not be used for output in DUNE DAQ software.
- Line 127: An unsigned integer appears to be used here.
- Line 128: "printf" should not be used for output in DUNE DAQ software.
- Line 137: If an else has a brace on one side, it should have it on both
- Line 140: If an else has a brace on one side, it should have it on both
- Line 143: If an else has a brace on one side, it should have it on both
- Line 161: Small and focused functions are preferred: WIB::InitializeDTS() has 105 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
Warnings
- Line 20: macro 'TLVL_ENTER_EXIT_METHODS' used to declare a constant; consider using a 'constexpr' constant
- Line 22: macro 'TLVL_GENERATION' used to declare a constant; consider using a 'constexpr' constant
- Line 24: macro 'TLVL_CANDIDATE' used to declare a constant; consider using a 'constexpr' constant
- Line 8: inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
- Line 47: throw expression should throw anonymous temporary values instead
- Line 86: throw expression should throw anonymous temporary values instead
- Line 93: do not declare C-style arrays, use std::array<> instead
- Line 128: do not use array subscript when the index is not an integer constant expression
- Line 159: throw expression should throw anonymous temporary values instead
- Line 173: throw expression should throw anonymous temporary values instead
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_FAKE_CD.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 30: An unsigned integer appears to be used here.
- Line 31: An unsigned integer appears to be used here.
- Line 32: An unsigned integer appears to be used here.
- Line 33: An unsigned integer appears to be used here.
- Line 44: An unsigned integer appears to be used here.
- Line 49: An unsigned integer appears to be used here.
- Line 52: An unsigned integer appears to be used here.
- Line 57: An unsigned integer appears to be used here.
- Line 68: An unsigned integer appears to be used here.
- Line 74: An unsigned integer appears to be used here.
- Line 77: An unsigned integer appears to be used here.
- Line 78: Add #include
for string
Warnings
- Line 12: throw expression should throw anonymous temporary values instead
- Line 74: C-style casts are discouraged; use static_cast
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_FEMB.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 417: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 12: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 37: An unsigned integer appears to be used here.
- Line 38: An unsigned integer appears to be used here.
- Line 83: If an else has a brace on one side, it should have it on both
- Line 84: "cout" should not be used for output in DUNE DAQ software.
- Line 95: "cout" should not be used for output in DUNE DAQ software.
- Line 98: An unsigned integer appears to be used here.
- Line 103: "cout" should not be used for output in DUNE DAQ software.
- Line 154: An unsigned integer appears to be used here.
- Line 155: An unsigned integer appears to be used here.
- Line 161: An unsigned integer appears to be used here.
- Line 192: "cout" should not be used for output in DUNE DAQ software.
- Line 201: "cout" should not be used for output in DUNE DAQ software.
- Line 204: "cout" should not be used for output in DUNE DAQ software.
- Line 206: If an else has a brace on one side, it should have it on both
- Line 207: An unsigned integer appears to be used here.
- Line 223: An unsigned integer appears to be used here.
- Line 239: An unsigned integer appears to be used here.
- Line 240: "cout" should not be used for output in DUNE DAQ software.
- Line 241: "cout" should not be used for output in DUNE DAQ software.
- Line 264: Small and focused functions are preferred: WIB::ConfigFEMB() has 174 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 275: An unsigned integer appears to be used here.
- Line 276: An unsigned integer appears to be used here.
- Line 324: An unsigned integer appears to be used here.
- Line 375: An unsigned integer appears to be used here.
- Line 378: An unsigned integer appears to be used here.
- Line 379: An unsigned integer appears to be used here.
- Line 380: An unsigned integer appears to be used here.
- Line 381: An unsigned integer appears to be used here.
- Line 382: An unsigned integer appears to be used here.
- Line 383: An unsigned integer appears to be used here.
- Line 384: An unsigned integer appears to be used here.
- Line 385: An unsigned integer appears to be used here.
- Line 386: An unsigned integer appears to be used here.
- Line 387: An unsigned integer appears to be used here.
- Line 388: An unsigned integer appears to be used here.
- Line 389: An unsigned integer appears to be used here.
- Line 390: An unsigned integer appears to be used here.
- Line 391: An unsigned integer appears to be used here.
- Line 392: An unsigned integer appears to be used here.
- Line 393: An unsigned integer appears to be used here.
- Line 394: An unsigned integer appears to be used here.
- Line 395: An unsigned integer appears to be used here.
- Line 396: An unsigned integer appears to be used here.
- Line 398: An unsigned integer appears to be used here.
- Line 399: An unsigned integer appears to be used here.
- Line 400: An unsigned integer appears to be used here.
- Line 401: An unsigned integer appears to be used here.
- Line 402: An unsigned integer appears to be used here.
- Line 403: An unsigned integer appears to be used here.
- Line 404: An unsigned integer appears to be used here.
- Line 405: An unsigned integer appears to be used here.
- Line 406: An unsigned integer appears to be used here.
- Line 407: An unsigned integer appears to be used here.
- Line 408: An unsigned integer appears to be used here.
- Line 409: An unsigned integer appears to be used here.
- Line 410: An unsigned integer appears to be used here.
- Line 411: An unsigned integer appears to be used here.
- Line 412: An unsigned integer appears to be used here.
- Line 413: An unsigned integer appears to be used here.
- Line 414: An unsigned integer appears to be used here.
- Line 441: An unsigned integer appears to be used here.
- Line 442: An unsigned integer appears to be used here.
- Line 443: An unsigned integer appears to be used here.
- Line 444: An unsigned integer appears to be used here.
- Line 445: An unsigned integer appears to be used here.
- Line 446: An unsigned integer appears to be used here.
- Line 447: An unsigned integer appears to be used here.
- Line 448: An unsigned integer appears to be used here.
- Line 449: An unsigned integer appears to be used here.
- Line 450: An unsigned integer appears to be used here.
- Line 451: An unsigned integer appears to be used here.
- Line 453: An unsigned integer appears to be used here.
- Line 454: An unsigned integer appears to be used here.
- Line 455: An unsigned integer appears to be used here.
- Line 456: An unsigned integer appears to be used here.
- Line 457: An unsigned integer appears to be used here.
- Line 458: An unsigned integer appears to be used here.
- Line 459: An unsigned integer appears to be used here.
- Line 460: An unsigned integer appears to be used here.
- Line 461: An unsigned integer appears to be used here.
- Line 462: An unsigned integer appears to be used here.
- Line 463: An unsigned integer appears to be used here.
- Line 468: An unsigned integer appears to be used here.
- Line 469: An unsigned integer appears to be used here.
- Line 470: An unsigned integer appears to be used here.
- Line 472: An unsigned integer appears to be used here.
- Line 523: An unsigned integer appears to be used here.
- Line 534: An unsigned integer appears to be used here.
- Line 539: Small and focused functions are preferred: WIB::SetupFEMBExtClock() has 108 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 550: An unsigned integer appears to be used here.
- Line 593: An unsigned integer appears to be used here.
- Line 621: An unsigned integer appears to be used here.
- Line 623: An unsigned integer appears to be used here.
- Line 655: Using C-style cast. Use static_cast
(...) instead - Line 688: An unsigned integer appears to be used here.
- Line 715: An unsigned integer appears to be used here.
- Line 718: An unsigned integer appears to be used here.
- Line 725: "cout" should not be used for output in DUNE DAQ software.
- Line 744: "cout" should not be used for output in DUNE DAQ software.
- Line 745: An unsigned integer appears to be used here.
- Line 748: An unsigned integer appears to be used here.
- Line 754: "cout" should not be used for output in DUNE DAQ software.
- Line 760: "cout" should not be used for output in DUNE DAQ software.
- Line 772: "cout" should not be used for output in DUNE DAQ software.
- Line 781: "cout" should not be used for output in DUNE DAQ software.
- Line 800: An unsigned integer appears to be used here.
- Line 807: Small and focused functions are preferred: WIB::SetupFEMBASICs() has 158 non-comment lines (error triggered by exceeding 80 lines) (disable this by putting "// NOLINT" after the function's closing brace.)
- Line 809: An unsigned integer appears to be used here.
- Line 811: An unsigned integer appears to be used here.
- Line 812: An unsigned integer appears to be used here.
- Line 813: An unsigned integer appears to be used here.
- Line 824: "cout" should not be used for output in DUNE DAQ software.
- Line 826: An unsigned integer appears to be used here.
- Line 827: An unsigned integer appears to be used here.
- Line 828: An unsigned integer appears to be used here.
- Line 829: An unsigned integer appears to be used here.
- Line 832: An unsigned integer appears to be used here.
- Line 833: "cout" should not be used for output in DUNE DAQ software.
- Line 837: An unsigned integer appears to be used here.
- Line 838: An unsigned integer appears to be used here.
- Line 839: An unsigned integer appears to be used here.
- Line 840: An unsigned integer appears to be used here.
- Line 860: "cout" should not be used for output in DUNE DAQ software.
- Line 861: An unsigned integer appears to be used here.
- Line 864: An unsigned integer appears to be used here.
- Line 869: "cout" should not be used for output in DUNE DAQ software.
- Line 873: "cout" should not be used for output in DUNE DAQ software.
- Line 884: "cout" should not be used for output in DUNE DAQ software.
- Line 903: An unsigned integer appears to be used here.
- Line 912: An unsigned integer appears to be used here.
- Line 913: "cout" should not be used for output in DUNE DAQ software.
- Line 928: An unsigned integer appears to be used here.
- Line 929: "cout" should not be used for output in DUNE DAQ software.
- Line 943: An unsigned integer appears to be used here.
- Line 945: An unsigned integer appears to be used here.
- Line 946: An unsigned integer appears to be used here.
- Line 948: An unsigned integer appears to be used here.
- Line 949: An unsigned integer appears to be used here.
- Line 962: If an else has a brace on one side, it should have it on both
- Line 967: "cout" should not be used for output in DUNE DAQ software.
- Line 981: If an else has a brace on one side, it should have it on both
- Line 986: "cout" should not be used for output in DUNE DAQ software.
- Line 1001: If an else has a brace on one side, it should have it on both
- Line 1006: "cout" should not be used for output in DUNE DAQ software.
- Line 1021: If an else has a brace on one side, it should have it on both
- Line 1026: "cout" should not be used for output in DUNE DAQ software.
- Line 1031: An unsigned integer appears to be used here.
- Line 1034: "cout" should not be used for output in DUNE DAQ software.
- Line 1036: An unsigned integer appears to be used here.
- Line 1037: "cout" should not be used for output in DUNE DAQ software.
- Line 1038: "cout" should not be used for output in DUNE DAQ software.
- Line 1043: An unsigned integer appears to be used here.
- Line 1048: "cout" should not be used for output in DUNE DAQ software.
- Line 1051: "cout" should not be used for output in DUNE DAQ software.
- Line 1052: "cout" should not be used for output in DUNE DAQ software.
- Line 1052: An unsigned integer appears to be used here.
- Line 1061: An unsigned integer appears to be used here.
- Line 1062: An unsigned integer appears to be used here.
- Line 1063: An unsigned integer appears to be used here.
- Line 1064: An unsigned integer appears to be used here.
- Line 1066: An unsigned integer appears to be used here.
- Line 1070: An unsigned integer appears to be used here.
- Line 1074: An unsigned integer appears to be used here.
- Line 1085: "cout" should not be used for output in DUNE DAQ software.
- Line 1088: "cout" should not be used for output in DUNE DAQ software.
- Line 1091: "cout" should not be used for output in DUNE DAQ software.
- Line 1092: "cout" should not be used for output in DUNE DAQ software.
- Line 1095: If an else has a brace on one side, it should have it on both
- Line 1096: "cout" should not be used for output in DUNE DAQ software.
- Line 1098: An unsigned integer appears to be used here.
- Line 1099: An unsigned integer appears to be used here.
- Line 1106: If an else has a brace on one side, it should have it on both
- Line 1109: If an else has a brace on one side, it should have it on both
- Line 1113: If an else has a brace on one side, it should have it on both
- Line 1117: An unsigned integer appears to be used here.
- Line 1126: An unsigned integer appears to be used here.
- Line 1127: An unsigned integer appears to be used here.
- Line 1131: If an else has a brace on one side, it should have it on both
- Line 1136: An unsigned integer appears to be used here.
- Line 1031: Add #include
for vector<>
Warnings
- Line 12: function-like macro 'sleep' used; consider a 'constexpr' template function
- Line 12: macro argument should be enclosed in parentheses
- Line 45: C-style casts are discouraged; use static_cast
- Line 47: throw expression should throw anonymous temporary values instead
- Line 54: C-style casts are discouraged; use static_cast
- Line 56: throw expression should throw anonymous temporary values instead
- Line 64: throw expression should throw anonymous temporary values instead
- Line 81: throw expression should throw anonymous temporary values instead
- Line 95: C-style casts are discouraged; use static_cast
- Line 95: C-style casts are discouraged; use static_cast
- Line 103: C-style casts are discouraged; use static_cast
- Line 108: C-style casts are discouraged; use static_cast
- Line 110: throw expression should throw anonymous temporary values instead
- Line 169: C-style casts are discouraged; use static_cast
- Line 171: throw expression should throw anonymous temporary values instead
- Line 183: C-style casts are discouraged; use static_cast
- Line 185: throw expression should throw anonymous temporary values instead
- Line 192: C-style casts are discouraged; use static_cast
- Line 201: C-style casts are discouraged; use static_cast
- Line 204: C-style casts are discouraged; use static_cast
- Line 211: C-style casts are discouraged; use static_cast
- Line 217: throw expression should throw anonymous temporary values instead
- Line 227: C-style casts are discouraged; use static_cast
- Line 236: throw expression should throw anonymous temporary values instead
- Line 240: C-style casts are discouraged; use static_cast
- Line 241: C-style casts are discouraged; use static_cast
- Line 283: C-style casts are discouraged; use static_cast
- Line 285: throw expression should throw anonymous temporary values instead
- Line 293: throw expression should throw anonymous temporary values instead
- Line 302: throw expression should throw anonymous temporary values instead
- Line 311: throw expression should throw anonymous temporary values instead
- Line 318: C-style casts are discouraged; use static_cast
- Line 320: throw expression should throw anonymous temporary values instead
- Line 472: variable 'addr_data' is not initialized
- Line 564: throw expression should throw anonymous temporary values instead
- Line 593: C-style casts are discouraged; use static_cast
- Line 632: C-style casts are discouraged; use static_cast
- Line 634: throw expression should throw anonymous temporary values instead
- Line 641: C-style casts are discouraged; use static_cast
- Line 643: throw expression should throw anonymous temporary values instead
- Line 655: C-style casts are discouraged; use static_cast
- Line 689: converting integer literal to bool, use bool literal instead
- Line 690: converting integer literal to bool, use bool literal instead
- Line 691: converting integer literal to bool, use bool literal instead
- Line 692: converting integer literal to bool, use bool literal instead
- Line 693: converting integer literal to bool, use bool literal instead
- Line 694: converting integer literal to bool, use bool literal instead
- Line 695: converting integer literal to bool, use bool literal instead
- Line 696: converting integer literal to bool, use bool literal instead
- Line 697: converting integer literal to bool, use bool literal instead
- Line 698: converting integer literal to bool, use bool literal instead
- Line 699: converting integer literal to bool, use bool literal instead
- Line 700: converting integer literal to bool, use bool literal instead
- Line 701: converting integer literal to bool, use bool literal instead
- Line 702: converting integer literal to bool, use bool literal instead
- Line 703: converting integer literal to bool, use bool literal instead
- Line 704: converting integer literal to bool, use bool literal instead
- Line 781: C-style casts are discouraged; use static_cast
- Line 787: C-style casts are discouraged; use static_cast
- Line 789: throw expression should throw anonymous temporary values instead
- Line 800: C-style casts are discouraged; use static_cast
- Line 893: throw expression should throw anonymous temporary values instead
- Line 903: C-style casts are discouraged; use static_cast
- Line 913: C-style casts are discouraged; use static_cast
- Line 913: C-style casts are discouraged; use static_cast
- Line 929: C-style casts are discouraged; use static_cast
- Line 948: variable 'data' is not initialized
- Line 949: variable 'read_back' is not initialized
- Line 1043: Value stored to 'adc_fifo_sync' during its initialization is never read
- Line 1048: C-style casts are discouraged; use static_cast
- Line 1051: C-style casts are discouraged; use static_cast
- Line 1052: C-style casts are discouraged; use static_cast
- Line 1066: do not declare C-style arrays, use std::array<> instead
- Line 1070: do not declare C-style arrays, use std::array<> instead
- Line 1074: do not declare C-style arrays, use std::array<> instead
- Line 1080: Value stored to 'adc_fifo_sync' is never read
- Line 1085: C-style casts are discouraged; use static_cast
- Line 1088: C-style casts are discouraged; use static_cast
- Line 1098: do not use array subscript when the index is not an integer constant expression
- Line 1101: do not use array subscript when the index is not an integer constant expression
- Line 1102: do not use array subscript when the index is not an integer constant expression
- Line 1103: do not use array subscript when the index is not an integer constant expression
- Line 1103: do not use array subscript when the index is not an integer constant expression
- Line 1104: do not use array subscript when the index is not an integer constant expression
- Line 1104: do not use array subscript when the index is not an integer constant expression
- Line 1106: do not use array subscript when the index is not an integer constant expression
- Line 1106: do not use array subscript when the index is not an integer constant expression
- Line 1107: do not use array subscript when the index is not an integer constant expression
- Line 1107: do not use array subscript when the index is not an integer constant expression
- Line 1109: do not use array subscript when the index is not an integer constant expression
- Line 1127: variable 'tp_sel' is not initialized
- Line 2: declaration uses identifier '__ADC_ASIC_REG_MAPPING_HH__', which is a reserved identifier
- Line 2: declaration uses identifier '__ASIC_REG_MAPPING_HH__', which is a reserved identifier
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__FE_ASIC_REG_MAPPING_HH__', which is a reserved identifier
- Line 31: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_Flash.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 7: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 9: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 10: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 12: An unsigned integer appears to be used here.
- Line 38: An unsigned integer appears to be used here.
- Line 40: An unsigned integer appears to be used here.
- Line 41: An unsigned integer appears to be used here.
- Line 67: An unsigned integer appears to be used here.
- Line 68: An unsigned integer appears to be used here.
- Line 70: An unsigned integer appears to be used here.
- Line 77: An unsigned integer appears to be used here.
- Line 80: An unsigned integer appears to be used here.
- Line 93: An unsigned integer appears to be used here.
- Line 95: An unsigned integer appears to be used here.
- Line 113: An unsigned integer appears to be used here.
- Line 116: An unsigned integer appears to be used here.
- Line 126: An unsigned integer appears to be used here.
- Line 131: Using deprecated casting style. Use static_cast
(...) instead - Line 135: Use of NULL #define found; prefer using the nullptr keyword
- Line 151: An unsigned integer appears to be used here.
- Line 153: An unsigned integer appears to be used here.
- Line 170: An unsigned integer appears to be used here.
- Line 184: "printf" should not be used for output in DUNE DAQ software.
- Line 185: "printf" should not be used for output in DUNE DAQ software.
- Line 209: An unsigned integer appears to be used here.
- Line 222: An unsigned integer appears to be used here.
- Line 236: An unsigned integer appears to be used here.
- Line 242: Using deprecated casting style. Use static_cast
(...) instead - Line 255: An unsigned integer appears to be used here.
- Line 256: An unsigned integer appears to be used here.
- Line 266: An unsigned integer appears to be used here.
- Line 278: An unsigned integer appears to be used here.
- Line 294: An unsigned integer appears to be used here.
- Line 299: Using deprecated casting style. Use static_cast
(...) instead - Line 311: An unsigned integer appears to be used here.
- Line 312: An unsigned integer appears to be used here.
- Line 322: An unsigned integer appears to be used here.
- Line 332: An unsigned integer appears to be used here.
- Line 348: An unsigned integer appears to be used here.
- Line 356: "printf" should not be used for output in DUNE DAQ software.
- Line 357: "printf" should not be used for output in DUNE DAQ software.
- Line 209: Add #include
for string - Line 294: Add #include
for vector<>
Warnings
- Line 5: inclusion of deprecated C++ header 'assert.h'; consider using 'cassert' instead
- Line 7: macro 'FLASH_TIMEOUT' used to declare a constant; consider using a 'constexpr' constant
- Line 9: macro 'MIN_LINE_LENGTH' used to declare a constant; consider using a 'constexpr' constant
- Line 10: macro 'MAX_LINE_LENGTH' used to declare a constant; consider using a 'constexpr' constant
- Line 16: throw expression should throw anonymous temporary values instead
- Line 20: throw expression should throw anonymous temporary values instead
- Line 27: throw expression should throw anonymous temporary values instead
- Line 31: throw expression should throw anonymous temporary values instead
- Line 35: throw expression should throw anonymous temporary values instead
- Line 38: multiple declarations in a single statement reduces readability
- Line 42: 'sscanf' used to convert a string to an unsigned integer value, but function will not report conversion errors; consider using 'strtoul' instead
- Line 42: do not use pointer arithmetic
- Line 49: Value stored to 'line_sum' is never read
- Line 52: throw expression should throw anonymous temporary values instead
- Line 54: Value stored to 'line_sum' is never read
- Line 57: 'sscanf' used to convert a string to an unsigned integer value, but function will not report conversion errors; consider using 'strtoul' instead
- Line 60: 'sscanf' used to convert a string to an unsigned integer value, but function will not report conversion errors; consider using 'strtoul' instead
- Line 60: do not use pointer arithmetic
- Line 69: 'sscanf' used to convert a string to an unsigned integer value, but function will not report conversion errors; consider using 'strtoul' instead
- Line 69: do not use pointer arithmetic
- Line 69: do not use pointer arithmetic
- Line 70: C-style casts are discouraged; use static_cast
- Line 89: throw expression should throw anonymous temporary values instead
- Line 105: throw expression should throw anonymous temporary values instead
- Line 109: throw expression should throw anonymous temporary values instead
- Line 131: narrowing conversion from 'float' to 'size_t' (aka 'unsigned long')
- Line 131: narrowing conversion from 'int' to 'float'
- Line 131: C-style casts are discouraged; use static_cast
- Line 131: result of integer division used in a floating point context; possible loss of precision
- Line 135: use nullptr
- Line 139: throw expression should throw anonymous temporary values instead
- Line 170: C-style casts are discouraged; use static_cast
- Line 204: throw expression should throw anonymous temporary values instead
- Line 242: narrowing conversion from 'float' to 'size_t' (aka 'unsigned long')
- Line 242: narrowing conversion from 'int' to 'float'
- Line 242: C-style casts are discouraged; use static_cast
- Line 262: C-style casts are discouraged; use static_cast
- Line 299: narrowing conversion from 'float' to 'size_t' (aka 'unsigned long')
- Line 299: narrowing conversion from 'int' to 'float'
- Line 299: C-style casts are discouraged; use static_cast
- Line 299: result of integer division used in a floating point context; possible loss of precision
- Line 318: C-style casts are discouraged; use static_cast
- Line 332: variable 'dataRead' is not initialized
- Line 336: do not declare C-style arrays, use std::array<> instead
- Line 344: throw expression should throw anonymous temporary values instead
- Line 372: throw expression should throw anonymous temporary values instead
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__BNL_UDP_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__BNL_UDP_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 9: operator=() does not handle self-assignment properly
- Line 9: do not declare C-style arrays, use std::array<> instead
- Line 10: operator=() does not handle self-assignment properly
- Line 10: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_History.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 3: An unsigned integer appears to be used here.
- Line 4: An unsigned integer appears to be used here.
- Line 5: An unsigned integer appears to be used here.
- Line 12: An unsigned integer appears to be used here.
- Line 13: An unsigned integer appears to be used here.
- Line 15: An unsigned integer appears to be used here.
- Line 17: An unsigned integer appears to be used here.
- Line 20: An unsigned integer appears to be used here.
- Line 12: Add #include
for string - Line 13: Add #include
for vector<>
Warnings
- Line 5: variable 'val' is not initialized
- Line 20: C-style casts are discouraged; use static_cast
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_QSFP.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 7: An unsigned integer appears to be used here.
- Line 10: An unsigned integer appears to be used here.
Warnings
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_SI5342.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 6: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 8: An unsigned integer appears to be used here.
- Line 11: An unsigned integer appears to be used here.
- Line 22: An unsigned integer appears to be used here.
- Line 26: An unsigned integer appears to be used here.
- Line 27: An unsigned integer appears to be used here.
- Line 30: An unsigned integer appears to be used here.
- Line 31: An unsigned integer appears to be used here.
- Line 44: Use of NULL #define found; prefer using the nullptr keyword
- Line 70: An unsigned integer appears to be used here.
- Line 82: "printf" should not be used for output in DUNE DAQ software.
- Line 85: Use of NULL #define found; prefer using the nullptr keyword
- Line 85: An unsigned integer appears to be used here.
- Line 86: Use of NULL #define found; prefer using the nullptr keyword
- Line 86: An unsigned integer appears to be used here.
- Line 87: An unsigned integer appears to be used here.
- Line 94: An unsigned integer appears to be used here.
- Line 98: "printf" should not be used for output in DUNE DAQ software.
- Line 114: An unsigned integer appears to be used here.
- Line 115: An unsigned integer appears to be used here.
- Line 116: An unsigned integer appears to be used here.
- Line 135: "printf" should not be used for output in DUNE DAQ software.
- Line 139: An unsigned integer appears to be used here.
- Line 141: An unsigned integer appears to be used here.
- Line 81: Add #include
for string - Line 87: Add #include
for pair<> - Line 70: Add #include
for vector<>
Warnings
- Line 6: macro 'SI5342_CONFIG_FILENAME' used to declare a constant; consider using a 'constexpr' constant
- Line 27: C-style casts are discouraged; use static_cast
- Line 31: C-style casts are discouraged; use static_cast
- Line 44: use nullptr
- Line 45: The parameter must not be null
- Line 58: throw expression should throw anonymous temporary values instead
- Line 85: use nullptr
- Line 86: use nullptr
- Line 87: use emplace_back instead of push_back
- Line 141: C-style casts are discouraged; use static_cast
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_SI5344.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 6: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 8: An unsigned integer appears to be used here.
- Line 11: An unsigned integer appears to be used here.
- Line 22: An unsigned integer appears to be used here.
- Line 26: An unsigned integer appears to be used here.
- Line 27: An unsigned integer appears to be used here.
- Line 30: An unsigned integer appears to be used here.
- Line 31: An unsigned integer appears to be used here.
- Line 44: Use of NULL #define found; prefer using the nullptr keyword
- Line 70: An unsigned integer appears to be used here.
- Line 82: "printf" should not be used for output in DUNE DAQ software.
- Line 85: Use of NULL #define found; prefer using the nullptr keyword
- Line 85: An unsigned integer appears to be used here.
- Line 86: Use of NULL #define found; prefer using the nullptr keyword
- Line 86: An unsigned integer appears to be used here.
- Line 87: An unsigned integer appears to be used here.
- Line 94: An unsigned integer appears to be used here.
- Line 98: "printf" should not be used for output in DUNE DAQ software.
- Line 114: An unsigned integer appears to be used here.
- Line 115: An unsigned integer appears to be used here.
- Line 116: An unsigned integer appears to be used here.
- Line 135: "printf" should not be used for output in DUNE DAQ software.
- Line 138: An unsigned integer appears to be used here.
- Line 140: An unsigned integer appears to be used here.
- Line 81: Add #include
for string - Line 87: Add #include
for pair<> - Line 70: Add #include
for vector<>
Warnings
- Line 6: macro 'SI5344_CONFIG_FILENAME' used to declare a constant; consider using a 'constexpr' constant
- Line 27: C-style casts are discouraged; use static_cast
- Line 31: C-style casts are discouraged; use static_cast
- Line 44: use nullptr
- Line 45: The parameter must not be null
- Line 58: throw expression should throw anonymous temporary values instead
- Line 85: use nullptr
- Line 86: use nullptr
- Line 87: use emplace_back instead of push_back
- Line 140: C-style casts are discouraged; use static_cast
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_localFlash.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 4: An unsigned integer appears to be used here.
- Line 13: "printf" should not be used for output in DUNE DAQ software.
- Line 21: An unsigned integer appears to be used here.
- Line 22: An unsigned integer appears to be used here.
- Line 32: An unsigned integer appears to be used here.
- Line 49: An unsigned integer appears to be used here.
- Line 49: Add #include
for vector<>
Warnings
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/WIB_spybuffer.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 13: An unsigned integer appears to be used here.
- Line 14: An unsigned integer appears to be used here.
- Line 20: An unsigned integer appears to be used here.
- Line 21: Missing date in TODO comment; it should appear on same line as the TODO, preferably in a form like "Apr-14-2020"
- Line 21: Missing email address in TODO comment; it should appear on the same line as the TODO
- Line 54: An unsigned integer appears to be used here.
- Line 56: An unsigned integer appears to be used here.
- Line 50: Add #include
for vector<> - Line 22: Add #include
for string
Warnings
- Line 8: throw expression should throw anonymous temporary values instead
- Line 14: use emplace_back instead of push_back
- Line 14: C-style casts are discouraged; use static_cast
- Line 31: throw expression should throw anonymous temporary values instead
- Line 61: use emplace_back instead of push_back
- Line 2: declaration uses identifier '__ADDRESSTABLE_HPP__', which is a reserved identifier
- Line 2: declaration uses identifier '__BNL_UDP_HH__', which is a reserved identifier
- Line 12: macro 'WIB_RESPONSE_PACKET_BUFFER_SIZE' used to declare a constant; consider using a 'constexpr' constant
- Line 16: use nullptr
- Line 2: declaration uses identifier '__EXCEPTIONBASE_HPP__', which is a reserved identifier
- Line 2: macro '__EXCEPTIONBASE_HPP__' used to declare a constant; consider using a 'constexpr' constant
- Line 28: function-like macro 'ExceptionClassGenerator' used; consider a 'constexpr' template function
- Line 29: macro argument should be enclosed in parentheses
- Line 34: macro argument should be enclosed in parentheses
- Line 2: declaration uses identifier '__WIB_HH__', which is a reserved identifier
- Line 199: converting integer literal to bool, use bool literal instead
- Line 2: declaration uses identifier '__WIBBASE_HH__', which is a reserved identifier
- Line 9: macro 'FEMB_COUNT' used to declare a constant; consider using a 'constexpr' constant
- Line 78: do not declare C-style arrays, use std::array<> instead
- Line 2: declaration uses identifier '__WIB_EXCEPTION_HH__', which is a reserved identifier
- Line 2: macro '__WIB_EXCEPTION_HH__' used to declare a constant; consider using a 'constexpr' constant
- Line 8: operator=() does not handle self-assignment properly
- Line 8: do not declare C-style arrays, use std::array<> instead
- Line 11: operator=() does not handle self-assignment properly
- Line 11: do not declare C-style arrays, use std::array<> instead
- Line 12: operator=() does not handle self-assignment properly
- Line 12: do not declare C-style arrays, use std::array<> instead
- Line 13: operator=() does not handle self-assignment properly
- Line 13: do not declare C-style arrays, use std::array<> instead
- Line 14: operator=() does not handle self-assignment properly
- Line 14: do not declare C-style arrays, use std::array<> instead
- Line 15: operator=() does not handle self-assignment properly
- Line 15: do not declare C-style arrays, use std::array<> instead
- Line 16: operator=() does not handle self-assignment properly
- Line 16: do not declare C-style arrays, use std::array<> instead
- Line 17: operator=() does not handle self-assignment properly
- Line 17: do not declare C-style arrays, use std::array<> instead
- Line 18: operator=() does not handle self-assignment properly
- Line 18: do not declare C-style arrays, use std::array<> instead
- Line 19: operator=() does not handle self-assignment properly
- Line 19: do not declare C-style arrays, use std::array<> instead
- Line 20: operator=() does not handle self-assignment properly
- Line 20: do not declare C-style arrays, use std::array<> instead
- Line 21: operator=() does not handle self-assignment properly
- Line 21: do not declare C-style arrays, use std::array<> instead
- Line 22: operator=() does not handle self-assignment properly
- Line 22: do not declare C-style arrays, use std::array<> instead
- Line 23: operator=() does not handle self-assignment properly
- Line 23: do not declare C-style arrays, use std::array<> instead
- Line 24: operator=() does not handle self-assignment properly
- Line 24: do not declare C-style arrays, use std::array<> instead
src/WIB1/conversions.cpp
Errors
- Line 0: The standard copyright message wasn't found.
- Line 23: C-style comment syntax detected; please use either C++ style "//" or Doxygen style
- Line 18: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 19: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 20: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 21: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 35: Namespace should be terminated with '// namespace itemconv '
- Line 40: Using C-style cast. Use static_cast
(...) instead - Line 40: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 42: Using C-style cast. Use static_cast
(...) instead - Line 43: Using C-style cast. Use static_cast
(...) instead - Line 43: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 45: Using C-style cast. Use static_cast
(...) instead - Line 45: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 48: Using C-style cast. Use static_cast
(...) instead - Line 48: Increment/decrement operator should appear alone on its line unless in a while/for loop head
- Line 59: #define appears to be used. Macros should generally be avoided if there's an alternative to them.
- Line 63: "wibmod/WIB1/conversions-impl.hh" already included at sourcecode/wibmod/src/WIB1/conversions.cpp:29
- Line 66: Use of NULL #define found; prefer using the nullptr keyword
- Line 52: Add #include
for string - Line 38: Add #include
for unordered_map<>
Warnings
- Line 11: inclusion of deprecated C++ header 'stdio.h'; consider using 'cstdio' instead
- Line 18: variadic macro 'ARGS' used; consider using a 'constexpr' variadic template function
- Line 19: function-like macro 'EXPAND' used; consider a 'constexpr' template function
- Line 20: function-like macro 'STRIP' used; consider a 'constexpr' template function
- Line 40: C-style casts are discouraged; use static_cast
- Line 42: C-style casts are discouraged; use static_cast
- Line 43: C-style casts are discouraged; use static_cast
- Line 45: C-style casts are discouraged; use static_cast
- Line 48: C-style casts are discouraged; use static_cast
- Line 66: use nullptr
- Line 19: use auto when initializing with a cast to avoid duplicating the type name
- Line 19: do not use C-style cast to convert between unrelated types
- Line 35: 'sscanf' used to convert a string to a floating-point value, but function will not report conversion errors; consider using 'strtod' instead
- Line 36: 'sscanf' used to convert a string to a floating-point value, but function will not report conversion errors; consider using 'strtod' instead
- Line 39: use auto when initializing with a cast to avoid duplicating the type name
- Line 39: do not use C-style cast to convert between unrelated types
- Line 55: 'sscanf' used to convert a string to an integer value, but function will not report conversion errors; consider using 'strtol' instead
- Line 65: do not declare C-style arrays, use std::array<> instead
- Line 70: do not use C-style cast to convert between unrelated types
- Line 80: do not use pointer arithmetic
src/WIBCommon.cpp
Warnings
- Line 17: nested namespaces can be concatenated
- Line 44: nested namespaces can be concatenated
- Line 54: do not declare C-style arrays, use std::array<> instead
- Line 61: declaration uses identifier '_CDFastCmd_default_instance_', which is a reserved identifier
- Line 64: declaration uses identifier '_CDPeek_default_instance_', which is a reserved identifier
- Line 67: declaration uses identifier '_CDPoke_default_instance_', which is a reserved identifier
- Line 70: declaration uses identifier '_CDRegValue_default_instance_', which is a reserved identifier
- Line 73: declaration uses identifier '_Calibrate_default_instance_', which is a reserved identifier
- Line 76: declaration uses identifier '_Command_default_instance_', which is a reserved identifier
- Line 79: declaration uses identifier '_ConfigurePower_default_instance_', which is a reserved identifier
- Line 82: declaration uses identifier '_ConfigureWIB_default_instance_', which is a reserved identifier
- Line 85: declaration uses identifier '_ConfigureWIB_ConfigureCOLDADC_default_instance_', which is a reserved identifier
- Line 88: declaration uses identifier '_ConfigureWIB_ConfigureFEMB_default_instance_', which is a reserved identifier
- Line 91: declaration uses identifier '_ConfigureWIB_ConfigureWIBPulser_default_instance_', which is a reserved identifier
- Line 94: declaration uses identifier '_Empty_default_instance_', which is a reserved identifier
- Line 97: declaration uses identifier '_GetSWVersion_default_instance_', which is a reserved identifier
- Line 100: declaration uses identifier '_GetSWVersion_Version_default_instance_', which is a reserved identifier
- Line 103: declaration uses identifier '_GetSensors_default_instance_', which is a reserved identifier
- Line 106: declaration uses identifier '_GetSensors_Sensors_default_instance_', which is a reserved identifier
- Line 109: declaration uses identifier '_GetTimestamp_default_instance_', which is a reserved identifier
- Line 112: declaration uses identifier '_GetTimestamp_Timestamp_default_instance_', which is a reserved identifier
- Line 115: declaration uses identifier '_GetTimingStatus_default_instance_', which is a reserved identifier
- Line 118: declaration uses identifier '_GetTimingStatus_TimingStatus_default_instance_', which is a reserved identifier
- Line 121: declaration uses identifier '_LogControl_default_instance_', which is a reserved identifier
- Line 124: declaration uses identifier '_LogControl_Log_default_instance_', which is a reserved identifier
- Line 127: declaration uses identifier '_Peek_default_instance_', which is a reserved identifier
- Line 130: declaration uses identifier '_Poke_default_instance_', which is a reserved identifier
- Line 133: declaration uses identifier '_PowerWIB_default_instance_', which is a reserved identifier
- Line 136: declaration uses identifier '_ReadDaqSpy_default_instance_', which is a reserved identifier
- Line 139: declaration uses identifier '_ReadDaqSpy_DaqSpy_default_instance_', which is a reserved identifier
- Line 142: declaration uses identifier '_ReadDaqSpy_DeframedDaqSpy_default_instance_', which is a reserved identifier
- Line 145: declaration uses identifier '_Reboot_default_instance_', which is a reserved identifier
- Line 148: declaration uses identifier '_Recompile_default_instance_', which is a reserved identifier
- Line 151: declaration uses identifier '_RegValue_default_instance_', which is a reserved identifier
- Line 154: declaration uses identifier '_ResetTiming_default_instance_', which is a reserved identifier
- Line 157: declaration uses identifier '_Script_default_instance_', which is a reserved identifier
- Line 160: declaration uses identifier '_SetFakeTime_default_instance_', which is a reserved identifier
- Line 163: declaration uses identifier '_StartFakeTime_default_instance_', which is a reserved identifier
- Line 166: declaration uses identifier '_Status_default_instance_', which is a reserved identifier
- Line 169: declaration uses identifier '_Update_default_instance_', which is a reserved identifier
- Line 171: nested namespaces can be concatenated
- Line 235: do not use reinterpret_cast
- Line 265: default arguments on virtual or override methods are prohibited
- Line 292: redundant access specifier has the same accessibility as the previous access specifier
- Line 330: declaration uses identifier '_Internal', which is a reserved identifier
- Line 399: do not use reinterpret_cast
- Line 429: default arguments on virtual or override methods are prohibited
- Line 456: redundant access specifier has the same accessibility as the previous access specifier
- Line 506: declaration uses identifier '_Internal', which is a reserved identifier
- Line 575: do not use reinterpret_cast
- Line 605: default arguments on virtual or override methods are prohibited
- Line 632: redundant access specifier has the same accessibility as the previous access specifier
- Line 699: declaration uses identifier '_Internal', which is a reserved identifier
- Line 769: do not use reinterpret_cast
- Line 799: default arguments on virtual or override methods are prohibited
- Line 826: redundant access specifier has the same accessibility as the previous access specifier
- Line 926: declaration uses identifier '_Internal', which is a reserved identifier
- Line 999: do not use reinterpret_cast
- Line 1029: default arguments on virtual or override methods are prohibited
- Line 1056: redundant access specifier has the same accessibility as the previous access specifier
- Line 1158: declaration uses identifier '_Internal', which is a reserved identifier
- Line 1232: do not use reinterpret_cast
- Line 1262: default arguments on virtual or override methods are prohibited
- Line 1289: redundant access specifier has the same accessibility as the previous access specifier
- Line 1377: declaration uses identifier '_Internal', which is a reserved identifier
- Line 1450: do not use reinterpret_cast
- Line 1480: default arguments on virtual or override methods are prohibited
- Line 1507: redundant access specifier has the same accessibility as the previous access specifier
- Line 1595: declaration uses identifier '_Internal', which is a reserved identifier
- Line 1668: do not use reinterpret_cast
- Line 1698: default arguments on virtual or override methods are prohibited
- Line 1725: redundant access specifier has the same accessibility as the previous access specifier
- Line 1939: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2023: do not use reinterpret_cast
- Line 2053: default arguments on virtual or override methods are prohibited
- Line 2080: redundant access specifier has the same accessibility as the previous access specifier
- Line 2190: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2265: do not use reinterpret_cast
- Line 2295: default arguments on virtual or override methods are prohibited
- Line 2322: redundant access specifier has the same accessibility as the previous access specifier
- Line 2407: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2479: do not use reinterpret_cast
- Line 2509: default arguments on virtual or override methods are prohibited
- Line 2536: redundant access specifier has the same accessibility as the previous access specifier
- Line 2668: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2743: do not use reinterpret_cast
- Line 2773: default arguments on virtual or override methods are prohibited
- Line 2784: redundant access specifier has the same accessibility as the previous access specifier
- Line 2806: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2868: do not use reinterpret_cast
- Line 2898: default arguments on virtual or override methods are prohibited
- Line 2909: redundant access specifier has the same accessibility as the previous access specifier
- Line 2931: declaration uses identifier '_Internal', which is a reserved identifier
- Line 2994: do not use reinterpret_cast
- Line 3024: default arguments on virtual or override methods are prohibited
- Line 3051: redundant access specifier has the same accessibility as the previous access specifier
- Line 3107: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3175: do not use reinterpret_cast
- Line 3205: default arguments on virtual or override methods are prohibited
- Line 3216: redundant access specifier has the same accessibility as the previous access specifier
- Line 3238: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3301: do not use reinterpret_cast
- Line 3331: default arguments on virtual or override methods are prohibited
- Line 3358: redundant access specifier has the same accessibility as the previous access specifier
- Line 3391: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3459: do not use reinterpret_cast
- Line 3489: default arguments on virtual or override methods are prohibited
- Line 3516: redundant access specifier has the same accessibility as the previous access specifier
- Line 3560: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3629: do not use reinterpret_cast
- Line 3659: default arguments on virtual or override methods are prohibited
- Line 3686: redundant access specifier has the same accessibility as the previous access specifier
- Line 3774: declaration uses identifier '_Internal', which is a reserved identifier
- Line 3847: do not use reinterpret_cast
- Line 3877: default arguments on virtual or override methods are prohibited
- Line 3904: redundant access specifier has the same accessibility as the previous access specifier
- Line 3981: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4053: do not use reinterpret_cast
- Line 4083: default arguments on virtual or override methods are prohibited
- Line 4110: redundant access specifier has the same accessibility as the previous access specifier
- Line 4143: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4211: do not use reinterpret_cast
- Line 4241: default arguments on virtual or override methods are prohibited
- Line 4268: redundant access specifier has the same accessibility as the previous access specifier
- Line 4532: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4612: do not use reinterpret_cast
- Line 4642: default arguments on virtual or override methods are prohibited
- Line 4653: redundant access specifier has the same accessibility as the previous access specifier
- Line 4677: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4740: do not use reinterpret_cast
- Line 4770: default arguments on virtual or override methods are prohibited
- Line 4797: redundant access specifier has the same accessibility as the previous access specifier
- Line 4896: declaration uses identifier '_Internal', which is a reserved identifier
- Line 4969: do not use reinterpret_cast
- Line 4999: default arguments on virtual or override methods are prohibited
- Line 5010: redundant access specifier has the same accessibility as the previous access specifier
- Line 5034: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5097: do not use reinterpret_cast
- Line 5127: default arguments on virtual or override methods are prohibited
- Line 5154: redundant access specifier has the same accessibility as the previous access specifier
- Line 5193: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5260: do not use reinterpret_cast
- Line 5290: default arguments on virtual or override methods are prohibited
- Line 5301: redundant access specifier has the same accessibility as the previous access specifier
- Line 5325: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5387: do not use reinterpret_cast
- Line 5417: default arguments on virtual or override methods are prohibited
- Line 5428: redundant access specifier has the same accessibility as the previous access specifier
- Line 5450: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5513: do not use reinterpret_cast
- Line 5543: default arguments on virtual or override methods are prohibited
- Line 5570: redundant access specifier has the same accessibility as the previous access specifier
- Line 5647: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5718: do not use reinterpret_cast
- Line 5748: default arguments on virtual or override methods are prohibited
- Line 5759: redundant access specifier has the same accessibility as the previous access specifier
- Line 5783: declaration uses identifier '_Internal', which is a reserved identifier
- Line 5846: do not use reinterpret_cast
- Line 5876: default arguments on virtual or override methods are prohibited
- Line 5903: redundant access specifier has the same accessibility as the previous access specifier
- Line 5936: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6003: do not use reinterpret_cast
- Line 6033: default arguments on virtual or override methods are prohibited
- Line 6044: redundant access specifier has the same accessibility as the previous access specifier
- Line 6066: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6129: do not use reinterpret_cast
- Line 6159: default arguments on virtual or override methods are prohibited
- Line 6186: redundant access specifier has the same accessibility as the previous access specifier
- Line 6225: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6293: do not use reinterpret_cast
- Line 6323: default arguments on virtual or override methods are prohibited
- Line 6350: redundant access specifier has the same accessibility as the previous access specifier
- Line 6407: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6476: do not use reinterpret_cast
- Line 6506: default arguments on virtual or override methods are prohibited
- Line 6517: redundant access specifier has the same accessibility as the previous access specifier
- Line 6539: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6602: do not use reinterpret_cast
- Line 6632: default arguments on virtual or override methods are prohibited
- Line 6659: redundant access specifier has the same accessibility as the previous access specifier
- Line 6709: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6778: do not use reinterpret_cast
- Line 6808: default arguments on virtual or override methods are prohibited
- Line 6835: redundant access specifier has the same accessibility as the previous access specifier
- Line 6879: declaration uses identifier '_Internal', which is a reserved identifier
- Line 6948: do not use reinterpret_cast
- Line 6978: default arguments on virtual or override methods are prohibited
- Line 7005: redundant access specifier has the same accessibility as the previous access specifier
- Line 7093: declaration uses identifier '_Internal', which is a reserved identifier
- Line 7139: do not use reinterpret_cast
- Line 7148: do not use reinterpret_cast
- Line 7150: do not use reinterpret_cast
- Line 7191: do not use reinterpret_cast
- Line 7204: do not use reinterpret_cast
- Line 7209: do not use reinterpret_cast
- Line 7218: do not use reinterpret_cast
- Line 8890: do not use reinterpret_cast
- Line 8899: do not use reinterpret_cast
- Line 8901: do not use reinterpret_cast
- Line 8942: do not use reinterpret_cast
- Line 8955: do not use reinterpret_cast
- Line 8960: do not use reinterpret_cast
- Line 8969: do not use reinterpret_cast
- Line 9030: do not use reinterpret_cast
- Line 9039: do not use reinterpret_cast
- Line 9041: do not use reinterpret_cast
- Line 9082: do not use reinterpret_cast
- Line 9095: do not use reinterpret_cast
- Line 9100: do not use reinterpret_cast
- Line 9109: do not use reinterpret_cast
- Line 21: nested namespaces can be concatenated
src/wibmod/WIBCommon.hpp
Errors
- Line 11: #ifndef header guard has wrong style, please use: WIBMOD_SRC_WIBMOD_WIBCOMMON_HPP_
- Line 72: #endif line should be "#endif // WIBMOD_SRC_WIBMOD_WIBCOMMON_HPP_"
- Line 17: Include the directory when naming .hh files
- Line 32: Single-parameter constructors should be marked explicit.