Ticket #6475 (assigned)

Opened 8 years ago

Last modified 5 years ago

Turn off warning suppressions

Reported by: Peter Peterson Owned by: Peter Peterson
Priority: major Milestone: Backlog
Component: Framework Keywords: Maintenance
Cc: martyn.gigg@… Blocked By:
Blocking: Tester:

Description

In order to have a better idea how bad the projects we forked are, turn off suppressions for projects that we have forked into the repository.

Change History

comment:1 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.5 to Release 2.6

Moved to r2.6 at the end of r2.5

comment:2 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned
  • Owner set to Peter Peterson

Pete,

Before we do this we need to consider what forked projects the mean here, and if we would actually alter the code to correct errors or warnings.

If we would not actually do anything about the information I'm not so sure of it's value.

comment:3 Changed 7 years ago by Peter Peterson

  • Keywords Maintenance added

comment:4 Changed 7 years ago by Nick Draper

  • Status changed from assigned to new

comment:5 Changed 7 years ago by Nick Draper

  • Component changed from Mantid to Framework

comment:6 Changed 7 years ago by Nick Draper

  • Milestone changed from Release 2.6 to Backlog

Moved to backlog at the code freeze for R2.6

comment:7 Changed 7 years ago by Peter Peterson

The ones I'm especially suspicious of are in non-generated code. See gcc diagnostic pragmas for more information, but the thing to look for is things like

#pragma GCC diagnostic ignored "-Wformat"

comment:8 Changed 7 years ago by Peter Peterson

  • Milestone changed from Backlog to Release 3.0

comment:9 Changed 7 years ago by Peter Peterson

  • Blocked By 7679 added

The suppressions via pragmas look legitimate so this ticket is to change them to use the new header introduced by #7679.

comment:10 Changed 7 years ago by Michael Reuter

  • Blocked By 7679 removed

comment:11 Changed 7 years ago by Peter Peterson

  • Milestone changed from Release 3.0 to Backlog

comment:12 Changed 7 years ago by Nick Draper

  • Status changed from new to assigned

Bulk move to assigned at the introduction of the triage step

comment:13 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 7321

Note: See TracTickets for help on using tickets.