User story #181
Address v0.2 cppcheck warnings
0%
Description
CppCheck has issued a series of warning that shall be fixed.
unchanged Source/Core/L0Portability/OperatingSystem/Linux/BasicConsoleOS.cpp 167 error fflushOnInputStream false fflush() called on input stream 'stdin' results in undefined behaviour.
unchanged Source/Core/L0Portability/OperatingSystem/Windows/BasicConsoleOS.cpp 204 style variableScope false The scope of the variable 'stdConsoleColumns' can be reduced.
unchanged Source/Core/L0Portability/OperatingSystem/Windows/BasicConsoleOS.cpp 205 style variableScope false The scope of the variable 'stdConsoleRows' can be reduced.
unchanged Source/Core/L0Portability/OperatingSystem/Windows/MemoryOS.cpp 68 style unreadVariable false Variable 'ok' is assigned a value that is never used.
unchanged Source/Core/L0Portability/ProcessorType.cpp 1 information ConfigurationNotChecked false Skipping configuration 'ProcessorTypeDefaultCPUs' since the value of 'ProcessorTypeDefaultCPUs' is unknown. Use -D if you want to check it. You can use -U to skip it explicitly.
unchanged Source/Core/L0Portability/StringPortable.cpp 353 style variableScope false The scope of the variable 'end2' can be reduced.
unchanged Source/Core/L0Portability/StringPortable.cpp 384 style variableScope false The scope of the variable 'end2' can be reduced.
unchanged Source/Core/L0Portability/StringPortable.cpp 469 style variableScope false The scope of the variable 'end' can be reduced.
Everything as been corrected. Added the parameter -DProcessorTypeDefaultCPUs=1 to jenkins.
History
#1 Updated by Riccardo Vitelli over 9 years ago
- Status changed from New to Code: Impl
#2 Updated by Riccardo Vitelli over 9 years ago
- Assignee set to Giuseppe Ferro
#3 Updated by Giuseppe Ferro over 9 years ago
- Status changed from Code: Impl to Arch: Rev
- Assignee deleted (
Giuseppe Ferro)
Code implementation done: a3189ef
#4 Updated by Giuseppe Ferro over 9 years ago
- Status changed from Arch: Rev to Code: Rev
#5 Updated by André Neto over 9 years ago
- Assignee set to André Neto
#6 Updated by Riccardo Vitelli over 9 years ago
- Status changed from Code: Rev to Closed
#7 Updated by André Neto over 9 years ago
- Description updated (diff)
- Assignee deleted (
André Neto)