Ticket #10527 (closed: invalid)

Opened 6 years ago

Last modified 5 years ago

Clang Warnings

Reported by: Peter Parker Owned by: Peter Parker
Priority: trivial Milestone: Release 3.3
Component: Indirect Inelastic Keywords:
Cc: hahnse@… Blocked By:
Blocking: Tester: Dan Nixon

Description

Slack message from Steven Hahn:

Some of your code in Refs #5308 (https://github.com/mantidproject/mantid/commit/9c4606692a4d4f60cc3fb2cb9bc591fb23837bf3) is generating warning in osx with the clang compiler. Would you please rework the following lines?

/Users/svh/Documents/MantidProject/mantid/Code/Mantid/MantidQt/CustomInterfaces/src/CalcCorr.cpp:292:5: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result]
   runPythonCode(pyInput).trimmed();
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
[1005/1462] Building CXX object Mantid...s/CustomInterfaces.dir/src/Elwin.cpp.o
/Users/svh/Documents/MantidProject/mantid/Code/Mantid/MantidQt/CustomInterfaces/src/Elwin.cpp:122:5: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result]
   runPythonCode(pyInput).trimmed();
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.
[1010/1462] Building CXX object Mantid.../CustomInterfaces.dir/src/MSDFit.cpp.o
/Users/svh/Documents/MantidProject/mantid/Code/Mantid/MantidQt/CustomInterfaces/src/MSDFit.cpp:86:5: warning: ignoring return value of function declared with warn_unused_result attribute [-Wunused-result]
   runPythonCode(pyInput).trimmed();
   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1 warning generated.

Change History

comment:1 Changed 6 years ago by Peter Parker

Refs #10527 - Fix Clang compiler warnings.

Changeset: 5dde572049ba4ab8c807d498636a8a5e540b3503

comment:2 Changed 6 years ago by Peter Parker

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

After getting some merge conflicts during a checkbuild, it became apparent that these warnings have actually already been fixed as part of #10527 in the following commit:

https://github.com/mantidproject/mantid/commit/cac2596d72c579c7cb29d26c7587604442662030

They'll see their way into master once that ticket has been tested.

Closing this ticket as invalid and deleting the branch.

comment:3 Changed 6 years ago by Dan Nixon

I added a fix for this in #10368 (specifically 7cce79a844a4ddfa9a4810faa584808a9e6e1592).

Last edited 6 years ago by Dan Nixon (previous) (diff)

comment:4 Changed 6 years ago by Dan Nixon

  • Status changed from verify to verifying
  • Tester set to Dan Nixon

comment:5 Changed 6 years ago by Dan Nixon

  • Status changed from verifying to closed

comment:6 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 11369

Note: See TracTickets for help on using tickets.