research/cantera
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[Kinetics] Remove unnecessary copy constructors and assignment operators

Browse source 
The data members of C1, C2, C3, C_AnyN, and StoichManagerN are all copyable and
assignable, so it's better to just let the compiler supply these.
This commit is contained in:
Ray Speth 2014-11-01 00:13:52 +00:00
parent fd777096b2
commit 7eb726d3d9
1 changed files with 0 additions and 83 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

83
include/cantera/kinetics/StoichManager.h
View file

@ -155,19 +155,6 @@ public:
m_ic0(ic0) {
}
C1(const C1& right) :
m_rxn(right.m_rxn),
m_ic0(right.m_ic0) {
}
C1& operator=(const C1& right) {
if (this != &right) {
m_rxn = right.m_rxn;
m_ic0 = right.m_ic0;
}
return *this;
}
size_t data(std::vector<size_t>& ic) {
ic.resize(1);
ic[0] = m_ic0;
@ -247,21 +234,6 @@ public:
C2(size_t rxn = 0, size_t ic0 = 0, size_t ic1 = 0)
: m_rxn(rxn), m_ic0(ic0), m_ic1(ic1) {}
C2(const C2& right) :
m_rxn(right.m_rxn),
m_ic0(right.m_ic0),
m_ic1(right.m_ic1) {
}
C2& operator=(const C2& right) {
if (this != &right) {
m_rxn = right.m_rxn;
m_ic0 = right.m_ic0;
m_ic1 = right.m_ic1;
}
return *this;
}
size_t data(std::vector<size_t>& ic) {
ic.resize(2);
ic[0] = m_ic0;
@ -349,23 +321,6 @@ public:
C3(size_t rxn = 0, size_t ic0 = 0, size_t ic1 = 0, size_t ic2 = 0)
: m_rxn(rxn), m_ic0(ic0), m_ic1(ic1), m_ic2(ic2) {}
C3(const C3& right) :
m_rxn(right.m_rxn),
m_ic0(right.m_ic0),
m_ic1(right.m_ic1),
m_ic2(right.m_ic2) {
}
C3& operator=(const C3& right) {
if (this != &right) {
m_rxn = right.m_rxn;
m_ic0 = right.m_ic0;
m_ic1 = right.m_ic1;
m_ic2 = right.m_ic2;
}
return *this;
}
size_t data(std::vector<size_t>& ic) {
ic.resize(3);
ic[0] = m_ic0;
@ -473,26 +428,6 @@ public:
}
}
C_AnyN(const C_AnyN& right) :
m_n(right.m_n),
m_rxn(right.m_rxn),
m_ic(right.m_ic),
m_order(right.m_order),
m_stoich(right.m_stoich)
{
}
C_AnyN& operator=(const C_AnyN& right) {
if (this != &right) {
m_n = right.m_n;
m_rxn = right.m_rxn;
m_ic = right.m_ic;
m_order = right.m_order;
m_stoich = right.m_stoich;
}
return *this;
}
size_t data(std::vector<size_t>& ic) {
ic.resize(m_n);
for (size_t n = 0; n < m_n; n++) {
@ -790,24 +725,6 @@ public:
StoichManagerN() {
}
StoichManagerN(const StoichManagerN& right) :
m_c1_list(right.m_c1_list),
m_c2_list(right.m_c2_list),
m_c3_list(right.m_c3_list),
m_cn_list(right.m_cn_list)
{
}
StoichManagerN& operator=(const StoichManagerN& right) {
if (this != &right) {
m_c1_list = right.m_c1_list;
m_c2_list = right.m_c2_list;
m_c3_list = right.m_c3_list;
m_cn_list = right.m_cn_list;
}
return *this;
}
/**
* Add a single reaction to the list of reactions that this
* stoichiometric manager object handles.