Ticket #2818 (closed: fixed)
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:10 Changed 9 years ago by Stuart Campbell
comment:11 Changed 9 years ago by Stuart Campbell
comment:12 Changed 9 years ago by Stuart Campbell
comment:13 Changed 9 years ago by Stuart Campbell
comment:14 Changed 9 years ago by Martyn Gigg
comment:15 Changed 9 years ago by Owen Arnold
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
comment:18 Changed 9 years ago by Martyn Gigg
comment:19 Changed 9 years ago by Owen Arnold
comment:20 Changed 9 years ago by Anders Markvardsen
comment:21 Changed 9 years ago by Owen Arnold
comment:22 Changed 9 years ago by Roman Tolchenov
comment:23 Changed 9 years ago by Owen Arnold
comment:24 Changed 9 years ago by Martyn Gigg
comment:25 Changed 9 years ago by Owen Arnold
comment:26 Changed 9 years ago by Martyn Gigg
comment:27 Changed 9 years ago by Roman Tolchenov
comment:28 Changed 9 years ago by Owen Arnold
comment:29 Changed 9 years ago by Roman Tolchenov
comment:30 Changed 9 years ago by Owen Arnold
comment:31 Changed 9 years ago by Owen Arnold
comment:32 Changed 9 years ago by Martyn Gigg
comment:33 Changed 9 years ago by Martyn Gigg
comment:34 Changed 9 years ago by Martyn Gigg
comment:35 Changed 9 years ago by Martyn Gigg
comment:36 Changed 9 years ago by Steve Williams
comment:37 Changed 9 years ago by Steve Williams
comment:38 Changed 9 years ago by Martyn Gigg
comment:39 Changed 9 years ago by Martyn Gigg
comment:40 Changed 9 years ago by Martyn Gigg
comment:41 Changed 9 years ago by Steve Williams
comment:42 Changed 9 years ago by Anders Markvardsen
comment:43 Changed 9 years ago by Martyn Gigg
comment:44 Changed 9 years ago by Martyn Gigg
comment:45 Changed 9 years ago by Martyn Gigg
comment:46 Changed 9 years ago by Martyn Gigg
comment:47 Changed 9 years ago by Martyn Gigg
comment:48 Changed 9 years ago by Roman Tolchenov
comment:49 Changed 9 years ago by Steve Williams
comment:50 Changed 9 years ago by Roman Tolchenov
comment:51 Changed 9 years ago by Martyn Gigg
comment:52 Changed 9 years ago by Martyn Gigg
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
comment:55 Changed 9 years ago by Janik Zikovsky
comment:56 Changed 9 years ago by Janik Zikovsky
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
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:63 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 3665
(In [10748]) remove unused variables. refs #2818