Project

General

Profile

User story #181

Address v0.2 cppcheck warnings

Added by André Neto over 9 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Target version:
Start date:
31.07.2015
Due date:
% Done:

0%

Estimated time:
Git branch (link):
Git merge to develop (link):
SVN commit (link/?p=rev):

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)

Also available in: Atom PDF