Ticket #10527 (closed: invalid)
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: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).
Note: See
TracTickets for help on using
tickets.
Refs #10527 - Fix Clang compiler warnings.
Changeset: 5dde572049ba4ab8c807d498636a8a5e540b3503