Ticket #2818 (closed: fixed)

Opened 9 years ago

Last modified 5 years ago

Iteration 28: Compiler Warnings

Reported by: Stuart Campbell Owned by: Stuart Campbell
Priority: major Milestone: Iteration 28
Component: Mantid Keywords:
Cc: Blocked By:
Blocking: Tester: Michael Whitty

Description

General ticket for compiler warnings for iteration 28

Change History

comment:1 Changed 9 years ago by Stuart Campbell

(In [10748]) remove unused variables. refs #2818

comment:2 Changed 9 years ago by Stuart Campbell

(In [10755]) remove compiler warnings. refs #2818

comment:3 Changed 9 years ago by Stuart Campbell

(In [10756]) remove compiler warnings. refs #2818

comment:4 Changed 9 years ago by Stuart Campbell

(In [10760]) initialise variables. refs #2818

comment:5 Changed 9 years ago by Stuart Campbell

(In [10761]) fix signed/unsigned comparison. refs #2818

comment:6 Changed 9 years ago by Stuart Campbell

(In [10762]) remove compiler warnings. refs #2818

comment:7 Changed 9 years ago by Stuart Campbell

(In [10763]) fix signed/unsigned comparison. refs #2818

comment:8 Changed 9 years ago by Stuart Campbell

(In [10764]) remove some compiler warnings. refs #2818

comment:9 Changed 9 years ago by Stuart Campbell

(In [10942]) Remove compiler warnings for Nexus on Linux. refs #2818

comment:10 Changed 9 years ago by Stuart Campbell

(In [10943]) Remove compiler warnings for DataHandling (well LoadDAE) on Linux. refs #2818

comment:11 Changed 9 years ago by Stuart Campbell

(In [10944]) Remove compiler warnings for NexusTest on Linux. refs #2818

comment:12 Changed 9 years ago by Stuart Campbell

(In [10945]) Remove compiler warnings for DataHandlingTest on Linux. refs #2818

comment:13 Changed 9 years ago by Stuart Campbell

(In [10946]) Fix build. refs #2818

comment:14 Changed 9 years ago by Martyn Gigg

(In [10955]) Refs #2818. First round of commits for warning clearance on MSVC. There was a macro clash with muParser so I added a wrapper and replaced all includes with this one.

comment:15 Changed 9 years ago by Owen Arnold

(In [10956]) re #2818 gcc warnings hunt.

comment:16 Changed 9 years ago by Anders Markvardsen

(In [10957]) Notice what looks like ga ga in Atom.h

Declare attribute:

const NeutronAtom& neutron;

but construct it using from temporary local variable:

const NeutronAtom getNeutronNoExceptions(const uint16_t z, const uint16_t a) {

try {

return getNeutronAtom(z, a);

} catch (std::runtime_error & ) {

return NeutronAtom(z, a,

NAN, NAN, NAN, NAN, NAN, NAN, NAN, NAN); set to junk value

}

}

Atom::Atom(const string& symbol, const uint16_t z, const uint16_t a,

const double abundance, const double mass, const double density) : symbol(symbol), z_number(z), a_number(a), abundance(abundance), mass(mass), mass_density(density), number_density(density * N_A / mass), neutron(getNeutronNoExceptions(z, a))

{ }

Changed local attribute 'neutron' to be NeutronAtom instead of NeutronAtom&.

re #2818

comment:17 Changed 9 years ago by Martyn Gigg

(In [10958]) Refs #2818. Add a few warning options on gcc as there are some that show up on Windows that are useful, in particular conversions that lead to loss of data.

comment:18 Changed 9 years ago by Martyn Gigg

(In [10959]) Refs #2818. A couple of those gcc flags aren't valid for C++.

comment:19 Changed 9 years ago by Owen Arnold

(In [10960]) re #2818 gcc warnings hunt. Mostly unused argument warnings or init arg ordering.

comment:20 Changed 9 years ago by Anders Markvardsen

(In [10962]) Remove 90 something percent of Kernel warnings. re #2818

comment:21 Changed 9 years ago by Owen Arnold

(In [10963]) re #2818 gcc warnings hunt. Minor change to fix Vates warning while still using gcc.

comment:22 Changed 9 years ago by Roman Tolchenov

(In [10964]) Added the XIntegrationControl ( re #2134 ) and removed a few compiler warnings in MantidPlot ( re #2818 ).

comment:23 Changed 9 years ago by Owen Arnold

(In [10965]) re #2818 VS warnings hunt. Fix MDDataObject build for windows.

comment:24 Changed 9 years ago by Martyn Gigg

(In [10966]) Refs #2818. MSVC warnings clearance for MDGeometry + related.

comment:25 Changed 9 years ago by Owen Arnold

(In [10967]) re #2818 Fix broken test and fix warning.

comment:26 Changed 9 years ago by Martyn Gigg

(In [10968]) Refs #2818. Clear a further large stack of warnings from Geometry.

comment:27 Changed 9 years ago by Roman Tolchenov

(In [10969]) More warnings cleared from MantidPlot. re #2818

comment:28 Changed 9 years ago by Owen Arnold

(In [10970]) re #2818 Remove all warnings possible from VATES project under windows after geometry-level type changes.

comment:29 Changed 9 years ago by Roman Tolchenov

(In [10971]) The build server wasn't happy. re #2818

comment:30 Changed 9 years ago by Owen Arnold

(In [10972]) re #2818 Remove from MDDataObjects/MDDataAlgorithms under windows. some use of static_casts required. Should refactor out later.

comment:31 Changed 9 years ago by Owen Arnold

(In [10973]) re #2818 Want to verify things work in MDDataObjects.

comment:32 Changed 9 years ago by Martyn Gigg

(In [10974]) Refs #2818. Geometry (and tests) now compile cleanly on MSVC. About to lose the will to live...

comment:33 Changed 9 years ago by Martyn Gigg

(In [10975]) Refs #2818. Clear Python API compiler warnings on MSVC. Had to define a global boost/python.hpp wrapper as alot of warnings came out of there. Not 100% happy but can't think of another way.

comment:34 Changed 9 years ago by Martyn Gigg

(In [10976]) Refs #2818. Missed a new file.

comment:35 Changed 9 years ago by Martyn Gigg

(In [10977]) Refs #2818. Fix linux include.

comment:36 Changed 9 years ago by Steve Williams

(In [10978]) Cleared the matlab compiler warnings re #2818

comment:37 Changed 9 years ago by Steve Williams

(In [10979]) Partial clearing of Datahandling warnings, many remain re #2818

comment:38 Changed 9 years ago by Martyn Gigg

(In [10985]) Remove the final few warnings from Kernel on Linux. Re #2818

comment:39 Changed 9 years ago by Martyn Gigg

(In [10986]) Fix Mac. Re #2818

comment:40 Changed 9 years ago by Martyn Gigg

(In [10987]) Fix DateAndTime test. Re #2818

comment:41 Changed 9 years ago by Steve Williams

(In [10988]) Last matlab compiler warnings from mwdebg.c re #2818

comment:42 Changed 9 years ago by Anders Markvardsen

(In [10989]) Remove last Kernel warnings. re #2818

comment:43 Changed 9 years ago by Martyn Gigg

(In [10990]) Refs #2818. Cleared gcc warnings for Geometry. Some should have been picked up with MSVC, not sure why they weren't. Anyhow, back to some real work...

comment:44 Changed 9 years ago by Martyn Gigg

(In [10991]) Refs #2818. Missed a file deletion.

comment:45 Changed 9 years ago by Martyn Gigg

(In [10993]) Refs #2818. Easy win, 85 warnings gone.

comment:46 Changed 9 years ago by Martyn Gigg

(In [10994]) Refs #2818. Better fix for the future. Now the generated files won't need changing at all.

comment:47 Changed 9 years ago by Martyn Gigg

(In [10995]) Refs #2818. Missed 2 file adds.

comment:48 Changed 9 years ago by Roman Tolchenov

(In [11002]) Cleared a few linux warnings. re #2818

comment:49 Changed 9 years ago by Steve Williams

(In [11005]) Cleared all the Datahandling warnings on my machine re #2818

comment:50 Changed 9 years ago by Roman Tolchenov

(In [11019]) More warnings cleared from MantidPlot. re #2818

comment:51 Changed 9 years ago by Martyn Gigg

(In [11036]) Refs #2818. Stop warnings from the Qt libraries headers from pervading our code.

comment:52 Changed 9 years ago by Martyn Gigg

(In [11041]) Fix the python script generation for the dynamic Load algorithm. Fixes #2465. Also cleared a few more warnings. Refs #2818

comment:53 Changed 9 years ago by Janik Zikovsky

(In [11065]) - Refs #2835: Added a MDDimensionStats object that calculates mean and approximate variance of positions.

  • Refs #2835: Implemented a binary-tree style splitting for MDEventWorkspaces (MDSplitBox) as an experiment to replace MDGridBox.
  • Added Utils::NestedForLoop routines.
  • Refs #2711: Improved MDEventWorkspace binning speed by approximately 20 through better scheduling. Memory use is also much lower.
  • Refs #2818: Fixed compiler warnings for MDEvents and MDEventsTest (as of last ~Monday with fewer warnings flags).
  • More flags added to class_maker.py

comment:54 Changed 9 years ago by Janik Zikovsky

(In [11066]) Refs #2818: Fixed some of the new warnings in MDEvents

comment:55 Changed 9 years ago by Janik Zikovsky

(In [11067]) Refs #2818: Fixed all remaining compiler warnings in MDEvents.

comment:56 Changed 9 years ago by Janik Zikovsky

(In [11068]) Refs #2818: Fixed compiler warning on Linus for MDAlgorithms.

comment:57 Changed 9 years ago by Nick Draper

  • Milestone changed from Iteration 28 to Iteration 29

Bulk move of tickets at the end of iteration 28

comment:58 Changed 9 years ago by Peter Peterson

(In [11131]) Frixes #2502. Refs #2818.

comment:59 Changed 9 years ago by Stuart Campbell

  • Status changed from new to accepted
  • Owner set to Stuart Campbell

comment:60 Changed 9 years ago by Stuart Campbell

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

Closing for end of iteration.

comment:61 Changed 9 years ago by Michael Whitty

  • Status changed from verify to verifying
  • Tester set to Michael Whitty
  • Milestone changed from Iteration 29 to Iteration 28

comment:62 Changed 9 years ago by Michael Whitty

  • Status changed from verifying to closed

comment:63 Changed 5 years ago by Stuart Campbell

This ticket has been transferred to github issue 3665

Note: See TracTickets for help on using tickets.