Ticket #10838 (closed: fixed)

Opened 6 years ago

Last modified 5 years ago

suppress clang warnings on OSX

Reported by: Steven Hahn Owned by: Steven Hahn
Priority: major Milestone: Release 3.3
Component: Framework Keywords:
Cc: Blocked By:
Blocking: Tester: Martyn Gigg

Description (last modified by Steven Hahn) (diff)

Additional warning options were added to the osx+clang build in ticket #10685. This is to temporarily suppress these warnings on OSX.

Ticket #10839 will systematically enable these additional clang/gcc warning options on OSX and fix any issues they uncover.

Change History

comment:1 Changed 6 years ago by Steven Hahn

  • Description modified (diff)

comment:2 Changed 6 years ago by Steven Hahn

Refs #10838. Suppress additional gcc warnings. Fix remaining warning.

Changeset: a3654ae8435198867a98c7b8282eef263bb2ce07

comment:3 Changed 6 years ago by Steven Hahn

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

This is being verified as pull request #113.

comment:4 Changed 6 years ago by Alex Buts

Re #10838 fixed old approach to update_defaults_from_instrument

but it is still too complicated

Changeset: 3b0973d934bd426e909363b2db45317bd657ef89

comment:5 Changed 6 years ago by Martyn Gigg

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

comment:6 Changed 6 years ago by Martyn Gigg

Merge pull request #113 from mantidproject/feature/10838_suppress_osx_warnings

Refs #10838. Suppress new clang warnings. Fix remaining warning.

Changeset: 7648880e1c1d3b979c76b6dd3e27f2d8fc5777d0

comment:7 Changed 6 years ago by Martyn Gigg

  • Status changed from verifying to closed

Merge pull request #113 from mantidproject/feature/10838_suppress_osx_warnings

Refs #10838. Suppress new clang warnings. Fix remaining warning.

Full changeset: 7648880e1c1d3b979c76b6dd3e27f2d8fc5777d0

comment:8 Changed 6 years ago by Alex Buts

Re #10838 fixed old approach to update_defaults_from_instrument

but it is still too complicated

Changeset: d780d8dfa6913ab67556a6e4a7304f860a32acc1

comment:9 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11680

Note: See TracTickets for help on using tickets.