Ticket #8466 (closed: fixed)

Opened 7 years ago

Last modified 5 years ago

More CPPcheck cleanup

Reported by: Nick Draper Owned by: Nick Draper
Priority: major Milestone: Release 3.1
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description


Change History

comment:1 Changed 7 years ago by Nick Draper

  • Status changed from new to inprogress

re #8466 cppcheck warnings

Changeset: 796df28db73704bf4f9de35d90392ced249d2570

comment:2 Changed 7 years ago by Nick Draper

Further reduction of warnings. Addition of a CppCheck_Suppression.txt file to allow rules to be suppressed globally or for individual files.

comment:3 Changed 7 years ago by Nick Draper

  • Status changed from inprogress to verify
  • Resolution set to fixed

comment:4 Changed 7 years ago by Nick Draper

To Test:

  1. Check that the develop build passes with unit tests

​http://download.mantidproject.org/jenkins/job/is_inc_win7_develop/2828/

  1. Check that the system tests passed that night
  2. Code review (some large blocks look changed, that is just tab & space reformatting).

comment:5 Changed 7 years ago by Nick Draper

re #8466 clear compiler warning

Changeset: 1af4367efdf415540d69a9ff8aa37110abb33c57

comment:6 Changed 7 years ago by Martyn Gigg

  • Status changed from verify to verifying
  • Tester set to Martyn Gigg

comment:7 Changed 7 years ago by Martyn Gigg

Confirmed that the errors are now down to ones that need to be fixed by improved refactorings.

comment:8 Changed 7 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge remote-tracking branch 'origin/feature/8466_cppcheck_cleanup'

Full changeset: ​c52b9a53280fb8f9f0ef61c2738f457aa494154c

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue ​9310

Note: See TracTickets for help on using tickets.