[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[lmi-commits] [lmi] master f9af151: Use C++11 default keyword instead of
|
From: |
Greg Chicares |
|
Subject: |
[lmi-commits] [lmi] master f9af151: Use C++11 default keyword instead of empty ctor/dtor bodies (VZ) |
|
Date: |
Tue, 10 Jan 2017 12:30:42 +0000 (UTC) |
branch: master
commit f9af151e08690d630226dbeac3db514030a8590f
Author: Gregory W. Chicares <address@hidden>
Commit: Gregory W. Chicares <address@hidden>
Use C++11 default keyword instead of empty ctor/dtor bodies (VZ)
Give compiler more opportunities for optimization by explicitly marking
trivial ctor/dtor implementations as being default.
Changes in this commit were mostly done by clang-tidy with its
modernize-use-default check and prettied up later with
perl -0777 -i'' -pe 's/\n= default;/ = default;/gs'
to put "= default" on the same line, instead of on a new one and some
manual alignment fixes.
The only significant change was the removal of the default ctor of
database_impl in rate_table.cpp as it was confusing to have an empty
ctor which wasn't really equivalent to the default implementation due
to the use of const member field. Clarify the intention by leaving
only the ctor taking a path and calling it explicitly with an empty
path when necessary.
[GWC notes: Several empty assignment-operator bodies have similarly
been replaced by '= default;'. The 'rate_table.cpp' change causes a
unit-test problem--see:
http://lists.nongnu.org/archive/html/lmi/2017-01/msg00035.html
which requires investigation.]
---
about_dialog.cpp | 4 +---
accountvalue.cpp | 4 +---
actuarial_table.cpp | 4 +---
antediluvian_stubs.cpp | 27 +++++++++------------------
any_entity.hpp | 2 +-
any_member.hpp | 12 ++++--------
any_member_test.cpp | 2 +-
authenticity.cpp | 3 +--
basicvalues.cpp | 4 +---
cache_file_reads_test.cpp | 3 +--
callback_test.cpp | 4 +---
census_document.cpp | 4 +---
commutation_functions.cpp | 8 ++------
configurable_settings.cpp | 3 +--
currency.hpp | 4 +---
database.cpp | 4 +---
database_document.cpp | 4 +---
database_view.cpp | 6 ++----
database_view_editor.cpp | 4 +---
datum_base.cpp | 4 +---
datum_boolean.cpp | 4 +---
datum_sequence.cpp | 4 +---
datum_sequence.hpp | 10 +++++-----
datum_string.cpp | 8 ++------
dbdict.cpp | 4 +---
dbvalue.cpp | 4 +---
death_benefits.cpp | 4 +---
default_view.cpp | 8 ++------
docmanager_ex.cpp | 4 +---
emit_ledger.cpp | 3 +--
fund_data.cpp | 12 +++---------
getopt.cpp | 4 +---
global_settings.cpp | 3 +--
gpt_commutation_functions.cpp | 6 ++----
gpt_document.cpp | 4 +---
gpt_input.cpp | 4 +---
gpt_server.cpp | 4 +---
gpt_state.cpp | 4 +---
gpt_view.cpp | 12 +++---------
gpt_xml_document.cpp | 4 +---
group_quote_pdf_gen.cpp | 8 ++------
group_values.cpp | 8 ++------
icon_monger.cpp | 4 +---
ihs_acctval.cpp | 4 +---
ihs_basicval.cpp | 4 +---
ihs_irc7702.cpp | 4 +---
ihs_irc7702a.cpp | 4 +---
illustration_document.cpp | 4 +---
illustration_view.cpp | 4 +---
illustrator.cpp | 4 +---
input.cpp | 4 +---
input_sequence.cpp | 4 +---
interest_rates.cpp | 4 +---
ledger.cpp | 4 +---
ledger_base.cpp | 4 +---
ledger_text_formats.cpp | 8 ++------
ledger_variant.cpp | 8 ++------
ledgervalues.cpp | 4 +---
loads.cpp | 4 +---
loads_test.cpp | 2 +-
mec_document.cpp | 4 +---
mec_input.cpp | 4 +---
mec_server.cpp | 4 +---
mec_state.cpp | 4 +---
mec_view.cpp | 12 +++---------
mec_xml_document.cpp | 4 +---
multidimgrid_any.cpp | 4 +---
multidimgrid_any.hpp | 6 +++---
multidimgrid_tools.cpp | 4 +---
multiple_cell_document.cpp | 4 +---
mvc_controller.cpp | 4 +---
mvc_model.cpp | 8 ++------
mvc_view.cpp | 8 ++------
name_value_pairs.cpp | 3 +--
null_stream.cpp | 3 +--
obstruct_slicing.hpp | 4 ++--
obstruct_slicing_test.cpp | 10 +++++-----
outlay.cpp | 4 +---
policy_document.cpp | 4 +---
policy_view.cpp | 4 +---
preferences_model.cpp | 4 +---
preferences_view.cpp | 8 ++------
premium_tax.cpp | 3 +--
previewframe_ex.cpp | 4 +---
product_data.cpp | 10 +++-------
product_editor.cpp | 12 +++---------
progress_meter.cpp | 4 +---
progress_meter_cgi.cpp | 4 +---
progress_meter_cli.cpp | 4 +---
progress_meter_wx.cpp | 4 +---
rate_table.cpp | 7 +------
rounding_document.cpp | 4 +---
rounding_rules.cpp | 8 ++------
rounding_view.cpp | 4 +---
rounding_view_editor.cpp | 4 +---
rtti_lmi.hpp | 2 +-
safely_dereference_as_test.cpp | 2 +-
single_cell_document.cpp | 4 +---
single_choice_popup_menu.cpp | 4 +---
skeleton.cpp | 4 +---
stratified_charges.cpp | 12 +++---------
test_coding_rules.cpp | 4 ++--
text_doc.cpp | 4 +---
text_view.cpp | 4 +---
tier_document.cpp | 4 +---
tier_view.cpp | 6 ++----
tier_view_editor.cpp | 8 ++------
tier_view_editor.hpp | 4 +---
timer.cpp | 4 +---
tn_range.hpp | 8 ++++----
transferor.cpp | 4 +---
uncopyable_lmi.hpp | 2 +-
view_ex.cpp | 4 +---
wx_test_case.hpp | 2 +-
xml_lmi.cpp | 6 ++----
yare_input.cpp | 4 +---
116 files changed, 170 insertions(+), 426 deletions(-)
diff --git a/about_dialog.cpp b/about_dialog.cpp
index 355baba..9867e1e 100644
--- a/about_dialog.cpp
+++ b/about_dialog.cpp
@@ -48,9 +48,7 @@ AboutDialog::AboutDialog(wxWindow* parent)
{
}
-AboutDialog::~AboutDialog()
-{
-}
+AboutDialog::~AboutDialog() = default;
/// This virtual function calls its base-class namesake explicitly.
diff --git a/accountvalue.cpp b/accountvalue.cpp
index b8789d7..edd7edf 100644
--- a/accountvalue.cpp
+++ b/accountvalue.cpp
@@ -128,9 +128,7 @@ AccountValue::AccountValue(Input const& input)
}
//============================================================================
-AccountValue::~AccountValue()
-{
-}
+AccountValue::~AccountValue() = default;
//============================================================================
boost::shared_ptr<Ledger const> AccountValue::ledger_from_av() const
diff --git a/actuarial_table.cpp b/actuarial_table.cpp
index d9d6737..bce3193 100644
--- a/actuarial_table.cpp
+++ b/actuarial_table.cpp
@@ -105,9 +105,7 @@ actuarial_table::actuarial_table(std::string const&
filename, int table_number)
parse_table();
}
-actuarial_table::~actuarial_table()
-{
-}
+actuarial_table::~actuarial_table() = default;
/// Read a given number of values for a given issue age.
diff --git a/antediluvian_stubs.cpp b/antediluvian_stubs.cpp
index fea0c3f..2d14604 100644
--- a/antediluvian_stubs.cpp
+++ b/antediluvian_stubs.cpp
@@ -37,11 +37,9 @@ namespace
void authenticate_system()
{}
-glossed_string::glossed_string()
-{}
+glossed_string::glossed_string() = default;
-glossed_string::~glossed_string()
-{}
+glossed_string::~glossed_string() = default;
bool is_antediluvian_fork()
{
@@ -51,8 +49,7 @@ bool is_antediluvian_fork()
gpt_server::gpt_server(mcenum_emission)
{}
-gpt_server::~gpt_server()
-{}
+gpt_server::~gpt_server() = default;
bool gpt_server::operator()(fs::path const&)
{
@@ -61,8 +58,7 @@ bool gpt_server::operator()(fs::path const&)
template class xml_serializable<gpt_state>;
-gpt_state::gpt_state()
-{}
+gpt_state::gpt_state() = default;
gpt_state::gpt_state(gpt_state const&)
:obstruct_slicing <gpt_state>()
@@ -70,8 +66,7 @@ gpt_state::gpt_state(gpt_state const&)
,MemberSymbolTable <gpt_state>()
{}
-gpt_state::~gpt_state()
-{}
+gpt_state::~gpt_state() = default;
int gpt_state::class_version() const
{
@@ -91,8 +86,7 @@ bool gpt_state::is_detritus(std::string const&) const
mec_server::mec_server(mcenum_emission)
{}
-mec_server::~mec_server()
-{}
+mec_server::~mec_server() = default;
bool mec_server::operator()(fs::path const&)
{
@@ -101,8 +95,7 @@ bool mec_server::operator()(fs::path const&)
template class xml_serializable<mec_state>;
-mec_state::mec_state()
-{}
+mec_state::mec_state() = default;
mec_state::mec_state(mec_state const&)
:obstruct_slicing <mec_state>()
@@ -110,8 +103,7 @@ mec_state::mec_state(mec_state const&)
,MemberSymbolTable <mec_state>()
{}
-mec_state::~mec_state()
-{}
+mec_state::~mec_state() = default;
int mec_state::class_version() const
{
@@ -131,8 +123,7 @@ bool mec_state::is_detritus(std::string const&) const
product_data::product_data(std::string const&)
{}
-product_data::~product_data()
-{}
+product_data::~product_data() = default;
int product_data::class_version() const
{
diff --git a/any_entity.hpp b/any_entity.hpp
index 5f31235..dba21e8 100644
--- a/any_entity.hpp
+++ b/any_entity.hpp
@@ -37,7 +37,7 @@
class any_entity
{
public:
- virtual ~any_entity() {}
+ virtual ~any_entity() = default;
any_entity& operator=(std::string const& s) {return assign(s);}
diff --git a/any_member.hpp b/any_member.hpp
index 3a35ae6..87638cc 100644
--- a/any_member.hpp
+++ b/any_member.hpp
@@ -106,8 +106,7 @@ class placeholder
// Implementation of class placeholder.
-inline placeholder::~placeholder()
-{}
+inline placeholder::~placeholder() = default;
// Definition of class holder.
@@ -149,8 +148,7 @@ holder<ClassType,ValueType>::holder(ClassType* object,
ValueType const& value)
{}
template<typename ClassType, typename ValueType>
-holder<ClassType,ValueType>::~holder()
-{}
+holder<ClassType,ValueType>::~holder() = default;
template<typename ClassType, typename ValueType>
holder<ClassType,ValueType>& holder<ClassType,ValueType>::assign
@@ -561,12 +559,10 @@ class MemberSymbolTable
// Implementation of class MemberSymbolTable.
template<typename ClassType>
-MemberSymbolTable<ClassType>::MemberSymbolTable()
-{}
+MemberSymbolTable<ClassType>::MemberSymbolTable() = default;
template<typename ClassType>
-MemberSymbolTable<ClassType>::~MemberSymbolTable()
-{}
+MemberSymbolTable<ClassType>::~MemberSymbolTable() = default;
// operator[]() returns a known member; unlike std::map::operator[](),
// it never adds a new pair to the map, and it complains if such an
diff --git a/any_member_test.cpp b/any_member_test.cpp
index 8817332..d83e318 100644
--- a/any_member_test.cpp
+++ b/any_member_test.cpp
@@ -35,7 +35,7 @@
struct base_datum
{
base_datum() :sane(7) {}
- virtual ~base_datum() {} // Just to make it polymorphic.
+ virtual ~base_datum() = default; // Just to make it polymorphic.
virtual int virtual_function() = 0; // Just to make it abstract.
bool base_function()
{
diff --git a/authenticity.cpp b/authenticity.cpp
index 1828094..6af72fd 100644
--- a/authenticity.cpp
+++ b/authenticity.cpp
@@ -58,8 +58,7 @@ Authenticity::Authenticity()
{
}
-Authenticity::~Authenticity()
-{}
+Authenticity::~Authenticity() = default;
Authenticity& Authenticity::Instance()
{
diff --git a/basicvalues.cpp b/basicvalues.cpp
index 3ba4774..a06c867 100644
--- a/basicvalues.cpp
+++ b/basicvalues.cpp
@@ -78,9 +78,7 @@ BasicValues::BasicValues(Input const& input)
}
//============================================================================
-BasicValues::~BasicValues()
-{
-}
+BasicValues::~BasicValues() = default;
//============================================================================
void BasicValues::Init()
diff --git a/cache_file_reads_test.cpp b/cache_file_reads_test.cpp
index 395ca0d..0ce3577 100644
--- a/cache_file_reads_test.cpp
+++ b/cache_file_reads_test.cpp
@@ -36,8 +36,7 @@ class X
:public cache_file_reads<X>
{
public:
- X()
- {}
+ X() = default;
X(std::string const& filename)
{
std::ifstream ifs(filename, ios_in_binary());
diff --git a/callback_test.cpp b/callback_test.cpp
index 5e1d318..f83a66e 100644
--- a/callback_test.cpp
+++ b/callback_test.cpp
@@ -48,9 +48,7 @@ namespace shared_library
class adder
{
public:
- adder()
- {
- }
+ adder() = default;
double operator()(int x,float y,double z) const
{
diff --git a/census_document.cpp b/census_document.cpp
index 7848025..a146f61 100644
--- a/census_document.cpp
+++ b/census_document.cpp
@@ -40,9 +40,7 @@ CensusDocument::CensusDocument()
{
}
-CensusDocument::~CensusDocument()
-{
-}
+CensusDocument::~CensusDocument() = default;
wxDataViewCtrl& CensusDocument::PredominantViewWindow() const
{
diff --git a/commutation_functions.cpp b/commutation_functions.cpp
index 58526e3..80c332d 100644
--- a/commutation_functions.cpp
+++ b/commutation_functions.cpp
@@ -94,9 +94,7 @@ OLCommFns::OLCommFns
std::partial_sum(c.rbegin(), c.rend(), m.rbegin());
}
-OLCommFns::~OLCommFns()
-{
-}
+OLCommFns::~OLCommFns() = default;
/// Interest- and mortality-rate vectors --> commutation functions.
///
@@ -198,7 +196,5 @@ ULCommFns::ULCommFns
std::partial_sum(kc.rbegin(), kc.rend(), km.rbegin());
}
-ULCommFns::~ULCommFns()
-{
-}
+ULCommFns::~ULCommFns() = default;
diff --git a/configurable_settings.cpp b/configurable_settings.cpp
index 88808c5..5ed167f 100644
--- a/configurable_settings.cpp
+++ b/configurable_settings.cpp
@@ -173,8 +173,7 @@ configurable_settings::configurable_settings()
}
}
-configurable_settings::~configurable_settings()
-{}
+configurable_settings::~configurable_settings() = default;
configurable_settings& configurable_settings::instance()
{
diff --git a/currency.hpp b/currency.hpp
index eb52eb4..e8b81b3 100644
--- a/currency.hpp
+++ b/currency.hpp
@@ -70,9 +70,7 @@ class currency
/// No ctor-initializer-list is needed because the lone data
/// member has a member-initializer at its point of declaration.
- currency()
- {
- }
+ currency() = default;
/// Constructor from a positive number of dollars and cents.
///
diff --git a/database.cpp b/database.cpp
index d362eb9..dd3236c 100644
--- a/database.cpp
+++ b/database.cpp
@@ -72,9 +72,7 @@ product_database::product_database(yare_input const& input)
initialize(input.ProductName);
}
-product_database::~product_database()
-{
-}
+product_database::~product_database() = default;
/// Number of years to maturity for default lookup index.
///
diff --git a/database_document.cpp b/database_document.cpp
index 3792a71..1805b66 100644
--- a/database_document.cpp
+++ b/database_document.cpp
@@ -41,9 +41,7 @@ DatabaseDocument::DatabaseDocument()
db_.InitDB();
}
-DatabaseDocument::~DatabaseDocument()
-{
-}
+DatabaseDocument::~DatabaseDocument() = default;
database_entity& DatabaseDocument::GetTDBValue(e_database_key index)
{
diff --git a/database_view.cpp b/database_view.cpp
index 41470c8..dae4292 100644
--- a/database_view.cpp
+++ b/database_view.cpp
@@ -107,7 +107,7 @@ class database_tree_item_data
{
public:
database_tree_item_data(db_names const&);
- ~database_tree_item_data() override {}
+ ~database_tree_item_data() override = default;
db_names const& db_name() const {return db_names_;}
@@ -151,9 +151,7 @@ DatabaseView::DatabaseView()
{
}
-DatabaseView::~DatabaseView()
-{
-}
+DatabaseView::~DatabaseView() = default;
wxTreeCtrl* DatabaseView::CreateTreeCtrl(wxWindow* parent)
{
diff --git a/database_view_editor.cpp b/database_view_editor.cpp
index 572222b..890e618 100644
--- a/database_view_editor.cpp
+++ b/database_view_editor.cpp
@@ -110,9 +110,7 @@ DatabaseTableAdapter::DatabaseTableAdapter(database_entity*
db_value)
indexes_.resize(eda_max);
}
-DatabaseTableAdapter::~DatabaseTableAdapter()
-{
-}
+DatabaseTableAdapter::~DatabaseTableAdapter() = default;
bool DatabaseTableAdapter::DoApplyAxisAdjustment
(MultiDimAxisAny& axis
diff --git a/datum_base.cpp b/datum_base.cpp
index cc31e37..bfe4f70 100644
--- a/datum_base.cpp
+++ b/datum_base.cpp
@@ -28,9 +28,7 @@ datum_base::datum_base()
{
}
-datum_base::~datum_base()
-{
-}
+datum_base::~datum_base() = default;
void datum_base::enable(bool b)
{
diff --git a/datum_boolean.cpp b/datum_boolean.cpp
index a7b6c79..6a748e3 100644
--- a/datum_boolean.cpp
+++ b/datum_boolean.cpp
@@ -28,9 +28,7 @@ datum_boolean::datum_boolean(bool value)
{
}
-datum_boolean::~datum_boolean()
-{
-}
+datum_boolean::~datum_boolean() = default;
datum_boolean& datum_boolean::operator=(bool b)
{
diff --git a/datum_sequence.cpp b/datum_sequence.cpp
index 2c2ab41..f14a390 100644
--- a/datum_sequence.cpp
+++ b/datum_sequence.cpp
@@ -46,9 +46,7 @@ datum_sequence::datum_sequence(std::string const& s)
assert_sanity();
}
-datum_sequence::~datum_sequence()
-{
-}
+datum_sequence::~datum_sequence() = default;
datum_sequence& datum_sequence::operator=(std::string const& s)
{
diff --git a/datum_sequence.hpp b/datum_sequence.hpp
index e6336a6..09f9e17 100644
--- a/datum_sequence.hpp
+++ b/datum_sequence.hpp
@@ -117,7 +117,7 @@ class numeric_sequence
,private boost::equality_comparable<numeric_sequence,numeric_sequence>
{
public:
- numeric_sequence() {}
+ numeric_sequence() = default;
explicit numeric_sequence(std::string const& s) : datum_sequence(s) {}
numeric_sequence& operator=(std::string const&);
@@ -148,7 +148,7 @@ class payment_sequence
,private boost::equality_comparable<payment_sequence,payment_sequence>
{
public:
- payment_sequence() {}
+ payment_sequence() = default;
explicit payment_sequence(std::string const& s) : datum_sequence(s) {}
payment_sequence& operator=(std::string const&);
@@ -179,7 +179,7 @@ class mode_sequence
,private boost::equality_comparable<mode_sequence,mode_sequence>
{
public:
- mode_sequence() {}
+ mode_sequence() = default;
explicit mode_sequence(std::string const& s) : datum_sequence(s) {}
mode_sequence& operator=(std::string const&);
@@ -211,7 +211,7 @@ class specamt_sequence
,private boost::equality_comparable<specamt_sequence,specamt_sequence>
{
public:
- specamt_sequence() {}
+ specamt_sequence() = default;
explicit specamt_sequence(std::string const& s) : datum_sequence(s) {}
specamt_sequence& operator=(std::string const&);
@@ -242,7 +242,7 @@ class dbo_sequence
,private boost::equality_comparable<dbo_sequence,dbo_sequence>
{
public:
- dbo_sequence() {}
+ dbo_sequence() = default;
explicit dbo_sequence(std::string const& s) : datum_sequence(s) {}
dbo_sequence& operator=(std::string const&);
diff --git a/datum_string.cpp b/datum_string.cpp
index 6436e94..2159e1b 100644
--- a/datum_string.cpp
+++ b/datum_string.cpp
@@ -25,18 +25,14 @@
#include "facets.hpp"
-datum_string::datum_string()
-{
-}
+datum_string::datum_string() = default;
datum_string::datum_string(std::string const& value)
:value_(value)
{
}
-datum_string::~datum_string()
-{
-}
+datum_string::~datum_string() = default;
datum_string& datum_string::operator=(std::string const& s)
{
diff --git a/dbdict.cpp b/dbdict.cpp
index 0cd70c9..c278897 100644
--- a/dbdict.cpp
+++ b/dbdict.cpp
@@ -95,9 +95,7 @@ DBDictionary::DBDictionary(std::string const& filename)
Init(filename);
}
-DBDictionary::~DBDictionary()
-{
-}
+DBDictionary::~DBDictionary() = default;
database_entity const& DBDictionary::datum(std::string const& name) const
{
diff --git a/dbvalue.cpp b/dbvalue.cpp
index 53f2384..9a70497 100644
--- a/dbvalue.cpp
+++ b/dbvalue.cpp
@@ -103,9 +103,7 @@ database_entity::database_entity
assert_invariants();
}
-database_entity::~database_entity()
-{
-}
+database_entity::~database_entity() = default;
#if 0
/// Deliberately undefined copy assignment operator.
diff --git a/death_benefits.cpp b/death_benefits.cpp
index abfe7ed..ae73b09 100644
--- a/death_benefits.cpp
+++ b/death_benefits.cpp
@@ -51,9 +51,7 @@ death_benefits::death_benefits(int length, yare_input const&
yi)
}
//============================================================================
-death_benefits::~death_benefits()
-{
-}
+death_benefits::~death_benefits() = default;
//============================================================================
void death_benefits::set_specamt(double z, int from_year, int to_year)
diff --git a/default_view.cpp b/default_view.cpp
index 251c477..c161d37 100644
--- a/default_view.cpp
+++ b/default_view.cpp
@@ -25,13 +25,9 @@
#include "configurable_settings.hpp"
-DefaultView::DefaultView()
-{
-}
+DefaultView::DefaultView() = default;
-DefaultView::~DefaultView()
-{
-}
+DefaultView::~DefaultView() = default;
char const* DefaultView::DoBookControlName() const
{
diff --git a/docmanager_ex.cpp b/docmanager_ex.cpp
index 05e052e..cf8e904 100644
--- a/docmanager_ex.cpp
+++ b/docmanager_ex.cpp
@@ -70,9 +70,7 @@ DocManagerEx::DocManagerEx(long int flags, bool initialize)
{
}
-DocManagerEx::~DocManagerEx()
-{
-}
+DocManagerEx::~DocManagerEx() = default;
// WX !! Perhaps xrc could support a menu-use-file-history tag,
// and this stuff could be made available by default in the library.
diff --git a/emit_ledger.cpp b/emit_ledger.cpp
index 4269b34..cd9e591 100644
--- a/emit_ledger.cpp
+++ b/emit_ledger.cpp
@@ -73,8 +73,7 @@ ledger_emitter::ledger_emitter
}
}
-ledger_emitter::~ledger_emitter()
-{}
+ledger_emitter::~ledger_emitter() = default;
/// Perform initial case-level steps such as writing headers.
diff --git a/fund_data.cpp b/fund_data.cpp
index 4c114a7..3b8ee52 100644
--- a/fund_data.cpp
+++ b/fund_data.cpp
@@ -58,9 +58,7 @@ FundInfo::FundInfo
}
//============================================================================
-FundInfo::~FundInfo()
-{
-}
+FundInfo::~FundInfo() = default;
namespace xml_serialize
{
@@ -90,9 +88,7 @@ template<> struct xml_io<FundInfo>
} // namespace xml_serialize
//============================================================================
-FundData::FundData()
-{
-}
+FundData::FundData() = default;
//============================================================================
FundData::FundData(std::string const& a_Filename)
@@ -101,9 +97,7 @@ FundData::FundData(std::string const& a_Filename)
}
//============================================================================
-FundData::~FundData()
-{
-}
+FundData::~FundData() = default;
namespace
{
diff --git a/getopt.cpp b/getopt.cpp
index ed764b2..17219e8 100644
--- a/getopt.cpp
+++ b/getopt.cpp
@@ -162,9 +162,7 @@ GetOpt::GetOpt (int argc, char** argv, char const*
optstring,
Initialize (noptstring);
}
-GetOpt::~GetOpt()
-{
-}
+GetOpt::~GetOpt() = default;
void
GetOpt::Initialize (std::string const& a_optstring)
diff --git a/global_settings.cpp b/global_settings.cpp
index a4ac6f4..68f5db2 100644
--- a/global_settings.cpp
+++ b/global_settings.cpp
@@ -53,8 +53,7 @@ global_settings::global_settings()
,prospicience_date_ (last_yyyy_date())
{}
-global_settings::~global_settings()
-{}
+global_settings::~global_settings() = default;
global_settings& global_settings::instance()
{
diff --git a/gpt_commutation_functions.cpp b/gpt_commutation_functions.cpp
index 73e8965..f070ffc 100644
--- a/gpt_commutation_functions.cpp
+++ b/gpt_commutation_functions.cpp
@@ -161,8 +161,7 @@ gpt_commfns::gpt_commfns
LMI_ASSERT(0.0 < *std::min_element(D_net_exc_.begin(), D_net_exc_.end()));
}
-gpt_commfns::~gpt_commfns()
-{}
+gpt_commfns::~gpt_commfns() = default;
/// Calculate GLP or GSP.
///
@@ -213,8 +212,7 @@ gpt_cf_triad::gpt_cf_triad
{
}
-gpt_cf_triad::~gpt_cf_triad()
-{}
+gpt_cf_triad::~gpt_cf_triad() = default;
/// Calculate GLP or GSP.
///
diff --git a/gpt_document.cpp b/gpt_document.cpp
index 7e030f1..839b391 100644
--- a/gpt_document.cpp
+++ b/gpt_document.cpp
@@ -38,9 +38,7 @@ gpt_document::gpt_document()
{
}
-gpt_document::~gpt_document()
-{
-}
+gpt_document::~gpt_document() = default;
gpt_view& gpt_document::PredominantView() const
{
diff --git a/gpt_input.cpp b/gpt_input.cpp
index 1f3ec9e..1efe3fa 100644
--- a/gpt_input.cpp
+++ b/gpt_input.cpp
@@ -136,9 +136,7 @@ gpt_input::gpt_input(gpt_input const& z)
DoAdaptExternalities();
}
-gpt_input::~gpt_input()
-{
-}
+gpt_input::~gpt_input() = default;
gpt_input& gpt_input::operator=(gpt_input const& z)
{
diff --git a/gpt_server.cpp b/gpt_server.cpp
index d693035..c2da9ed 100644
--- a/gpt_server.cpp
+++ b/gpt_server.cpp
@@ -518,9 +518,7 @@ gpt_server::gpt_server(mcenum_emission emission)
{
}
-gpt_server::~gpt_server()
-{
-}
+gpt_server::~gpt_server() = default;
bool gpt_server::operator()(fs::path const& file_path)
{
diff --git a/gpt_state.cpp b/gpt_state.cpp
index 9b95385..e2f405d 100644
--- a/gpt_state.cpp
+++ b/gpt_state.cpp
@@ -59,9 +59,7 @@ gpt_state::gpt_state(gpt_state const& z)
MemberSymbolTable<gpt_state>::assign(z);
}
-gpt_state::~gpt_state()
-{
-}
+gpt_state::~gpt_state() = default;
gpt_state& gpt_state::operator=(gpt_state const& z)
{
diff --git a/gpt_view.cpp b/gpt_view.cpp
index 4f567c8..6dd84cc 100644
--- a/gpt_view.cpp
+++ b/gpt_view.cpp
@@ -35,13 +35,9 @@
#include <wx/html/htmprint.h>
#include <wx/xrc/xmlres.h>
-gpt_mvc_view::gpt_mvc_view()
-{
-}
+gpt_mvc_view::gpt_mvc_view() = default;
-gpt_mvc_view::~gpt_mvc_view()
-{
-}
+gpt_mvc_view::~gpt_mvc_view() = default;
char const* gpt_mvc_view::DoBookControlName() const
{
@@ -91,9 +87,7 @@ gpt_view::gpt_view()
{
}
-gpt_view::~gpt_view()
-{
-}
+gpt_view::~gpt_view() = default;
inline gpt_input& gpt_view::input_data()
{
diff --git a/gpt_xml_document.cpp b/gpt_xml_document.cpp
index f415c60..7ef49a1 100644
--- a/gpt_xml_document.cpp
+++ b/gpt_xml_document.cpp
@@ -52,9 +52,7 @@ gpt_xml_document::gpt_xml_document(std::string const&
filename)
}
//============================================================================
-gpt_xml_document::~gpt_xml_document()
-{
-}
+gpt_xml_document::~gpt_xml_document() = default;
//============================================================================
std::string const& gpt_xml_document::xml_root_name() const
diff --git a/group_quote_pdf_gen.cpp b/group_quote_pdf_gen.cpp
index b9da3ec..500e734 100644
--- a/group_quote_pdf_gen.cpp
+++ b/group_quote_pdf_gen.cpp
@@ -45,11 +45,7 @@ boost::shared_ptr<group_quote_pdf_generator>
group_quote_pdf_generator::create()
return group_quote_pdf_generator_create_callback()();
}
-group_quote_pdf_generator::group_quote_pdf_generator()
-{
-}
+group_quote_pdf_generator::group_quote_pdf_generator() = default;
-group_quote_pdf_generator::~group_quote_pdf_generator()
-{
-}
+group_quote_pdf_generator::~group_quote_pdf_generator() = default;
diff --git a/group_values.cpp b/group_values.cpp
index 1b1c4b3..0681ae9 100644
--- a/group_values.cpp
+++ b/group_values.cpp
@@ -663,13 +663,9 @@ census_run_result run_census_in_parallel::operator()
return result;
}
-run_census::run_census()
-{
-}
+run_census::run_census() = default;
-run_census::~run_census()
-{
-}
+run_census::~run_census() = default;
census_run_result run_census::operator()
(fs::path const& file
diff --git a/icon_monger.cpp b/icon_monger.cpp
index d477be4..ab5f948 100644
--- a/icon_monger.cpp
+++ b/icon_monger.cpp
@@ -100,9 +100,7 @@ icon_monger::icon_monger()
lmi_specific_icon_names_.insert("write-spreadsheet" );
}
-icon_monger::~icon_monger()
-{
-}
+icon_monger::~icon_monger() = default;
namespace
{
diff --git a/ihs_acctval.cpp b/ihs_acctval.cpp
index bcb176c..1749fd7 100644
--- a/ihs_acctval.cpp
+++ b/ihs_acctval.cpp
@@ -181,9 +181,7 @@ AccountValue::AccountValue(Input const& input)
}
//============================================================================
-AccountValue::~AccountValue()
-{
-}
+AccountValue::~AccountValue() = default;
//============================================================================
boost::shared_ptr<Ledger const> AccountValue::ledger_from_av() const
diff --git a/ihs_basicval.cpp b/ihs_basicval.cpp
index 116a714..fbda111 100644
--- a/ihs_basicval.cpp
+++ b/ihs_basicval.cpp
@@ -131,9 +131,7 @@ BasicValues::BasicValues
}
//============================================================================
-BasicValues::~BasicValues()
-{
-}
+BasicValues::~BasicValues() = default;
//============================================================================
void BasicValues::Init()
diff --git a/ihs_irc7702.cpp b/ihs_irc7702.cpp
index 2ba1c13..9047ee4 100644
--- a/ihs_irc7702.cpp
+++ b/ihs_irc7702.cpp
@@ -203,9 +203,7 @@ Irc7702::Irc7702
}
//============================================================================
-Irc7702::~Irc7702()
-{
-}
+Irc7702::~Irc7702() = default;
//============================================================================
void Irc7702::ProcessGptPmt
diff --git a/ihs_irc7702a.cpp b/ihs_irc7702a.cpp
index 9ca31a5..11c076a 100644
--- a/ihs_irc7702a.cpp
+++ b/ihs_irc7702a.cpp
@@ -171,9 +171,7 @@ Irc7702A::Irc7702A
}
//============================================================================
-Irc7702A::~Irc7702A()
-{
-}
+Irc7702A::~Irc7702A() = default;
/// Set initial values on issue or inforce date.
///
diff --git a/illustration_document.cpp b/illustration_document.cpp
index 8936159..423dff0 100644
--- a/illustration_document.cpp
+++ b/illustration_document.cpp
@@ -39,9 +39,7 @@ IllustrationDocument::IllustrationDocument()
{
}
-IllustrationDocument::~IllustrationDocument()
-{
-}
+IllustrationDocument::~IllustrationDocument() = default;
IllustrationView& IllustrationDocument::PredominantView() const
{
diff --git a/illustration_view.cpp b/illustration_view.cpp
index f3bc8a3..95ec372 100644
--- a/illustration_view.cpp
+++ b/illustration_view.cpp
@@ -103,9 +103,7 @@ IllustrationView::IllustrationView()
{
}
-IllustrationView::~IllustrationView()
-{
-}
+IllustrationView::~IllustrationView() = default;
inline Input& IllustrationView::input_data()
{
diff --git a/illustrator.cpp b/illustrator.cpp
index 799e50f..140fe1b 100644
--- a/illustrator.cpp
+++ b/illustrator.cpp
@@ -53,9 +53,7 @@ illustrator::illustrator(mcenum_emission emission)
{
}
-illustrator::~illustrator()
-{
-}
+illustrator::~illustrator() = default;
bool illustrator::operator()(fs::path const& file_path)
{
diff --git a/input.cpp b/input.cpp
index 4b39181..69b7d73 100644
--- a/input.cpp
+++ b/input.cpp
@@ -247,9 +247,7 @@ Input::Input(Input const& z)
DoAdaptExternalities();
}
-Input::~Input()
-{
-}
+Input::~Input() = default;
Input& Input::operator=(Input const& z)
{
diff --git a/input_sequence.cpp b/input_sequence.cpp
index b87a85b..5b69bfb 100644
--- a/input_sequence.cpp
+++ b/input_sequence.cpp
@@ -262,9 +262,7 @@ InputSequence::InputSequence
realize_vector();
}
-InputSequence::~InputSequence()
-{
-}
+InputSequence::~InputSequence() = default;
/*
// TODO ?? Want a similar function for std::vector<string>, for enumerators?
diff --git a/interest_rates.cpp b/interest_rates.cpp
index 0b8b65c..6341bd5 100644
--- a/interest_rates.cpp
+++ b/interest_rates.cpp
@@ -248,9 +248,7 @@ bool need_loan_rates(yare_input const& yi)
#endif // 0
} // Unnamed namespace.
-InterestRates::~InterestRates()
-{
-}
+InterestRates::~InterestRates() = default;
InterestRates::InterestRates(BasicValues const& v)
:Length_ (v.GetLength())
diff --git a/ledger.cpp b/ledger.cpp
index b5744ca..2248c00 100644
--- a/ledger.cpp
+++ b/ledger.cpp
@@ -80,9 +80,7 @@ Ledger::Ledger
}
//============================================================================
-Ledger::~Ledger()
-{
-}
+Ledger::~Ledger() = default;
//============================================================================
void Ledger::SetRunBases(int length)
diff --git a/ledger_base.cpp b/ledger_base.cpp
index d5d74d4..273e646 100644
--- a/ledger_base.cpp
+++ b/ledger_base.cpp
@@ -64,9 +64,7 @@ LedgerBase& LedgerBase::operator=(LedgerBase const& obj)
}
//============================================================================
-LedgerBase::~LedgerBase()
-{
-}
+LedgerBase::~LedgerBase() = default;
//============================================================================
void LedgerBase::Alloc()
diff --git a/ledger_text_formats.cpp b/ledger_text_formats.cpp
index c071c1d..14aad6b 100644
--- a/ledger_text_formats.cpp
+++ b/ledger_text_formats.cpp
@@ -208,9 +208,7 @@ calculation_summary_formatter::calculation_summary_formatter
}
}
-calculation_summary_formatter::~calculation_summary_formatter()
-{
-}
+calculation_summary_formatter::~calculation_summary_formatter() = default;
std::string calculation_summary_formatter::top_note
(std::string const& line_break
@@ -920,9 +918,7 @@ FlatTextLedgerPrinter::FlatTextLedgerPrinter
{
}
-FlatTextLedgerPrinter::~FlatTextLedgerPrinter()
-{
-}
+FlatTextLedgerPrinter::~FlatTextLedgerPrinter() = default;
void FlatTextLedgerPrinter::Print() const
{
diff --git a/ledger_variant.cpp b/ledger_variant.cpp
index 1bb9895..e6483c6 100644
--- a/ledger_variant.cpp
+++ b/ledger_variant.cpp
@@ -361,18 +361,14 @@ void LedgerVariant::Spew(std::ostream& os) const
LedgerBase::Spew(os);
}
-ledger_map_holder::ledger_map_holder()
-{
-}
+ledger_map_holder::ledger_map_holder() = default;
ledger_map_holder::ledger_map_holder(ledger_map_t const& z)
:held_(z)
{
}
-ledger_map_holder::~ledger_map_holder()
-{
-}
+ledger_map_holder::~ledger_map_holder() = default;
ledger_map_t const& ledger_map_holder::held() const
{
diff --git a/ledgervalues.cpp b/ledgervalues.cpp
index adc9795..c69bdcb 100644
--- a/ledgervalues.cpp
+++ b/ledgervalues.cpp
@@ -34,9 +34,7 @@ IllusVal::IllusVal(std::string const& filename)
{
}
-IllusVal::~IllusVal()
-{
-}
+IllusVal::~IllusVal() = default;
double IllusVal::run(Input const& input)
{
diff --git a/loads.cpp b/loads.cpp
index dcf6b6f..120301d 100644
--- a/loads.cpp
+++ b/loads.cpp
@@ -36,9 +36,7 @@
/// Ctor for unit testing.
-Loads::Loads()
-{
-}
+Loads::Loads() = default;
/// Ctor for production branch.
diff --git a/loads_test.cpp b/loads_test.cpp
index 9b2bfce..b5eda5a 100644
--- a/loads_test.cpp
+++ b/loads_test.cpp
@@ -50,7 +50,7 @@ std::vector<double> BasicValues::GetGuarSpecAmtLoadTable()
const {return dummy_v
#include "database.hpp"
product_database::product_database(int length) :length_(length) {}
-product_database::~product_database() {}
+product_database::~product_database() = default;
int product_database::length() const {return length_;}
void product_database::Query(std::vector<double>& v, e_database_key) const
{v.resize(length_);}
double product_database::Query(e_database_key) const {return 0.0;}
diff --git a/mec_document.cpp b/mec_document.cpp
index 4fa2f6d..ad2d0a3 100644
--- a/mec_document.cpp
+++ b/mec_document.cpp
@@ -38,9 +38,7 @@ mec_document::mec_document()
{
}
-mec_document::~mec_document()
-{
-}
+mec_document::~mec_document() = default;
mec_view& mec_document::PredominantView() const
{
diff --git a/mec_input.cpp b/mec_input.cpp
index f17dc96..f8a1446 100644
--- a/mec_input.cpp
+++ b/mec_input.cpp
@@ -126,9 +126,7 @@ mec_input::mec_input(mec_input const& z)
DoAdaptExternalities();
}
-mec_input::~mec_input()
-{
-}
+mec_input::~mec_input() = default;
mec_input& mec_input::operator=(mec_input const& z)
{
diff --git a/mec_server.cpp b/mec_server.cpp
index ebefede..3923266 100644
--- a/mec_server.cpp
+++ b/mec_server.cpp
@@ -500,9 +500,7 @@ mec_server::mec_server(mcenum_emission emission)
{
}
-mec_server::~mec_server()
-{
-}
+mec_server::~mec_server() = default;
bool mec_server::operator()(fs::path const& file_path)
{
diff --git a/mec_state.cpp b/mec_state.cpp
index 76681b0..aa30841 100644
--- a/mec_state.cpp
+++ b/mec_state.cpp
@@ -59,9 +59,7 @@ mec_state::mec_state(mec_state const& z)
MemberSymbolTable<mec_state>::assign(z);
}
-mec_state::~mec_state()
-{
-}
+mec_state::~mec_state() = default;
mec_state& mec_state::operator=(mec_state const& z)
{
diff --git a/mec_view.cpp b/mec_view.cpp
index 70c0c15..ade9070 100644
--- a/mec_view.cpp
+++ b/mec_view.cpp
@@ -35,13 +35,9 @@
#include <wx/html/htmprint.h>
#include <wx/xrc/xmlres.h>
-mec_mvc_view::mec_mvc_view()
-{
-}
+mec_mvc_view::mec_mvc_view() = default;
-mec_mvc_view::~mec_mvc_view()
-{
-}
+mec_mvc_view::~mec_mvc_view() = default;
char const* mec_mvc_view::DoBookControlName() const
{
@@ -91,9 +87,7 @@ mec_view::mec_view()
{
}
-mec_view::~mec_view()
-{
-}
+mec_view::~mec_view() = default;
inline mec_input& mec_view::input_data()
{
diff --git a/mec_xml_document.cpp b/mec_xml_document.cpp
index c53a5fe..e235e83 100644
--- a/mec_xml_document.cpp
+++ b/mec_xml_document.cpp
@@ -52,9 +52,7 @@ mec_xml_document::mec_xml_document(std::string const&
filename)
}
//============================================================================
-mec_xml_document::~mec_xml_document()
-{
-}
+mec_xml_document::~mec_xml_document() = default;
//============================================================================
std::string const& mec_xml_document::xml_root_name() const
diff --git a/multidimgrid_any.cpp b/multidimgrid_any.cpp
index 5eb93bc..040f356 100644
--- a/multidimgrid_any.cpp
+++ b/multidimgrid_any.cpp
@@ -140,9 +140,7 @@ inline MultiDimGridGrid::MultiDimGridGrid
{
}
-MultiDimGridGrid::~MultiDimGridGrid()
-{
-}
+MultiDimGridGrid::~MultiDimGridGrid() = default;
wxSize MultiDimGridGrid::DoGetBestSize() const
{
diff --git a/multidimgrid_any.hpp b/multidimgrid_any.hpp
index f45d41b..5cd499f 100644
--- a/multidimgrid_any.hpp
+++ b/multidimgrid_any.hpp
@@ -193,7 +193,7 @@ class MultiDimAxisAny
{
public:
MultiDimAxisAny(std::string const& name);
- virtual ~MultiDimAxisAny() {}
+ virtual ~MultiDimAxisAny() = default;
std::string const& GetName() const;
@@ -290,8 +290,8 @@ class MultiDimTableAny
typedef boost::shared_ptr<MultiDimAxisAny> AxisAnyPtr;
typedef std::vector<AxisAnyPtr> AxesAny;
- MultiDimTableAny() {}
- virtual ~MultiDimTableAny() {}
+ MultiDimTableAny() = default;
+ virtual ~MultiDimTableAny() = default;
/// Return the number of dimensions in this table.
unsigned int GetDimension() const;
diff --git a/multidimgrid_tools.cpp b/multidimgrid_tools.cpp
index abbb238..252a666 100644
--- a/multidimgrid_tools.cpp
+++ b/multidimgrid_tools.cpp
@@ -73,9 +73,7 @@ AutoResizingTreeCtrl::AutoResizingTreeCtrl
{
}
-AutoResizingTreeCtrl::~AutoResizingTreeCtrl()
-{
-}
+AutoResizingTreeCtrl::~AutoResizingTreeCtrl() = default;
wxSize AutoResizingTreeCtrl::DoGetBestSize() const
{
diff --git a/multiple_cell_document.cpp b/multiple_cell_document.cpp
index bfd3268..68b5053 100644
--- a/multiple_cell_document.cpp
+++ b/multiple_cell_document.cpp
@@ -59,9 +59,7 @@ multiple_cell_document::multiple_cell_document(std::string
const& filename)
}
//============================================================================
-multiple_cell_document::~multiple_cell_document()
-{
-}
+multiple_cell_document::~multiple_cell_document() = default;
/// Verify invariants.
///
diff --git a/mvc_controller.cpp b/mvc_controller.cpp
index 23ffadb..e8d175d 100644
--- a/mvc_controller.cpp
+++ b/mvc_controller.cpp
@@ -153,9 +153,7 @@ MvcController::MvcController
);
}
-MvcController::~MvcController()
-{
-}
+MvcController::~MvcController() = default;
/// Make the Model consistent, and change the View to comport with it.
///
diff --git a/mvc_model.cpp b/mvc_model.cpp
index 6fffac2..41ed622 100644
--- a/mvc_model.cpp
+++ b/mvc_model.cpp
@@ -64,13 +64,9 @@ void ComplainAboutAnyDiscrepancies
}
} // Unnamed namespace.
-MvcModel::MvcModel()
-{
-}
+MvcModel::MvcModel() = default;
-MvcModel::~MvcModel()
-{
-}
+MvcModel::~MvcModel() = default;
datum_base const* MvcModel::BaseDatumPointer(std::string const& name) const
{
diff --git a/mvc_view.cpp b/mvc_view.cpp
index 96772c2..730526b 100644
--- a/mvc_view.cpp
+++ b/mvc_view.cpp
@@ -23,13 +23,9 @@
#include "mvc_view.hpp"
-MvcView::MvcView()
-{
-}
+MvcView::MvcView() = default;
-MvcView::~MvcView()
-{
-}
+MvcView::~MvcView() = default;
char const* MvcView::BookControlName() const
{
diff --git a/name_value_pairs.cpp b/name_value_pairs.cpp
index 0746838..f6f588d 100644
--- a/name_value_pairs.cpp
+++ b/name_value_pairs.cpp
@@ -49,8 +49,7 @@ name_value_pairs::name_value_pairs(std::string const&
filename)
}
}
-name_value_pairs::~name_value_pairs()
-{}
+name_value_pairs::~name_value_pairs() = default;
std::string const& name_value_pairs::string_value(std::string const& key) const
{
diff --git a/null_stream.cpp b/null_stream.cpp
index ff68c3c..7fdcac7 100644
--- a/null_stream.cpp
+++ b/null_stream.cpp
@@ -48,8 +48,7 @@ class dev_null_stream_buffer
{
setp(buffer_, buffer_ + buffer_size_);
}
- ~dev_null_stream_buffer() override
- {}
+ ~dev_null_stream_buffer() override = default;
private:
int_type overflow(int_type c) override
diff --git a/obstruct_slicing.hpp b/obstruct_slicing.hpp
index 8838f50..65eb471 100644
--- a/obstruct_slicing.hpp
+++ b/obstruct_slicing.hpp
@@ -94,8 +94,8 @@ template<typename T>
class obstruct_slicing
{
protected:
- obstruct_slicing() {}
- obstruct_slicing(obstruct_slicing const&) {}
+ obstruct_slicing() = default;
+ obstruct_slicing(obstruct_slicing const&) = default;
};
#endif // obstruct_slicing_hpp
diff --git a/obstruct_slicing_test.cpp b/obstruct_slicing_test.cpp
index bb746b2..0217bb2 100644
--- a/obstruct_slicing_test.cpp
+++ b/obstruct_slicing_test.cpp
@@ -37,7 +37,7 @@
class no_derive0
{
protected:
- no_derive0() {}
+ no_derive0() = default;
};
class A0
@@ -55,8 +55,8 @@ template<typename T>
class no_derive1
{
protected:
- no_derive1() {}
- no_derive1(no_derive1 const&) {}
+ no_derive1() = default;
+ no_derive1(no_derive1 const&) = default;
};
class B0
@@ -78,8 +78,8 @@ class C0_no_derive
friend class C0;
private:
- C0_no_derive() {}
- C0_no_derive(C0_no_derive const&) {}
+ C0_no_derive() = default;
+ C0_no_derive(C0_no_derive const&) = default;
};
class C0
diff --git a/outlay.cpp b/outlay.cpp
index f5d26ce..c625265 100644
--- a/outlay.cpp
+++ b/outlay.cpp
@@ -41,9 +41,7 @@ modal_outlay::modal_outlay(yare_input const& yi)
{
}
-modal_outlay::~modal_outlay()
-{
-}
+modal_outlay::~modal_outlay() = default;
void modal_outlay::set_ee_modal_premiums(double z, int from_year, int to_year)
{
diff --git a/policy_document.cpp b/policy_document.cpp
index 3e3c0ec..5dd7012 100644
--- a/policy_document.cpp
+++ b/policy_document.cpp
@@ -83,9 +83,7 @@ PolicyDocument::PolicyDocument()
values_["GuarMortalityFootnote" ] = &product_data_.GuarMortalityFootnote
.datum_;
}
-PolicyDocument::~PolicyDocument()
-{
-}
+PolicyDocument::~PolicyDocument() = default;
PolicyView& PolicyDocument::PredominantView() const
{
diff --git a/policy_view.cpp b/policy_view.cpp
index 4f0ab2c..8615741 100644
--- a/policy_view.cpp
+++ b/policy_view.cpp
@@ -40,9 +40,7 @@ PolicyView::PolicyView()
{
}
-PolicyView::~PolicyView()
-{
-}
+PolicyView::~PolicyView() = default;
wxWindow* PolicyView::CreateChildWindow()
{
diff --git a/preferences_model.cpp b/preferences_model.cpp
index 7a1f7f9..4a540bb 100644
--- a/preferences_model.cpp
+++ b/preferences_model.cpp
@@ -95,9 +95,7 @@ PreferencesModel::PreferencesModel()
Load();
}
-PreferencesModel::~PreferencesModel()
-{
-}
+PreferencesModel::~PreferencesModel() = default;
void PreferencesModel::AscribeMembers()
{
diff --git a/preferences_view.cpp b/preferences_view.cpp
index 5fffa53..91510a4 100644
--- a/preferences_view.cpp
+++ b/preferences_view.cpp
@@ -23,13 +23,9 @@
#include "preferences_view.hpp"
-PreferencesView::PreferencesView()
-{
-}
+PreferencesView::PreferencesView() = default;
-PreferencesView::~PreferencesView()
-{
-}
+PreferencesView::~PreferencesView() = default;
char const* PreferencesView::DoBookControlName() const
{
diff --git a/premium_tax.cpp b/premium_tax.cpp
index 250ff4b..5b0d70e 100644
--- a/premium_tax.cpp
+++ b/premium_tax.cpp
@@ -178,8 +178,7 @@ premium_tax::premium_tax
levy_rate_ = db.Query(DB_PremTaxRate, index);
}
-premium_tax::~premium_tax()
-{}
+premium_tax::~premium_tax() = default;
/// Test consistency of premium-tax loads.
///
diff --git a/previewframe_ex.cpp b/previewframe_ex.cpp
index 9c726be..ed18ee1 100644
--- a/previewframe_ex.cpp
+++ b/previewframe_ex.cpp
@@ -59,9 +59,7 @@ PreviewFrameEx::PreviewFrameEx
}
}
-PreviewFrameEx::~PreviewFrameEx()
-{
-}
+PreviewFrameEx::~PreviewFrameEx() = default;
void PreviewFrameEx::Initialize()
{
diff --git a/product_data.cpp b/product_data.cpp
index 6f6c933..a3cbcf9 100644
--- a/product_data.cpp
+++ b/product_data.cpp
@@ -88,8 +88,7 @@ template<> glossed_string
value_cast<glossed_string>(std::string const& z)
return glossed_string(z);
}
-glossed_string::glossed_string()
-{}
+glossed_string::glossed_string() = default;
glossed_string::glossed_string
(std::string const& datum
@@ -99,8 +98,7 @@ glossed_string::glossed_string
,gloss_(gloss)
{}
-glossed_string::~glossed_string()
-{}
+glossed_string::~glossed_string() = default;
glossed_string& glossed_string::operator=(std::string const& s)
{
@@ -155,9 +153,7 @@ product_data::product_data(std::string const& product_name)
load(AddDataDir(path.string()));
}
-product_data::~product_data()
-{
-}
+product_data::~product_data() = default;
/// Principal string datum (without gloss) for named member.
diff --git a/product_editor.cpp b/product_editor.cpp
index 7d82ddd..ed7d07e 100644
--- a/product_editor.cpp
+++ b/product_editor.cpp
@@ -38,9 +38,7 @@ ProductEditorDocument::ProductEditorDocument()
{
}
-ProductEditorDocument::~ProductEditorDocument()
-{
-}
+ProductEditorDocument::~ProductEditorDocument() = default;
ProductEditorView& ProductEditorDocument::PredominantView() const
{
@@ -120,9 +118,7 @@ ProductEditorView::ProductEditorView()
{
}
-ProductEditorView::~ProductEditorView()
-{
-}
+ProductEditorView::~ProductEditorView() = default;
TreeGridViewBase::TreeGridViewBase()
:ProductEditorView()
@@ -132,9 +128,7 @@ TreeGridViewBase::TreeGridViewBase()
{
}
-TreeGridViewBase::~TreeGridViewBase()
-{
-}
+TreeGridViewBase::~TreeGridViewBase() = default;
wxTreeCtrl& TreeGridViewBase::tree() const
{
diff --git a/progress_meter.cpp b/progress_meter.cpp
index 5425481..5805316 100644
--- a/progress_meter.cpp
+++ b/progress_meter.cpp
@@ -77,9 +77,7 @@ progress_meter::progress_meter
{
}
-progress_meter::~progress_meter()
-{
-}
+progress_meter::~progress_meter() = default;
void progress_meter::dawdle(int seconds)
{
diff --git a/progress_meter_cgi.cpp b/progress_meter_cgi.cpp
index ecb9393..aedf30f 100644
--- a/progress_meter_cgi.cpp
+++ b/progress_meter_cgi.cpp
@@ -59,9 +59,7 @@ concrete_progress_meter::concrete_progress_meter
{
}
-concrete_progress_meter::~concrete_progress_meter()
-{
-}
+concrete_progress_meter::~concrete_progress_meter() = default;
std::string concrete_progress_meter::progress_message() const
{
diff --git a/progress_meter_cli.cpp b/progress_meter_cli.cpp
index 083e9f7..c76f586 100644
--- a/progress_meter_cli.cpp
+++ b/progress_meter_cli.cpp
@@ -103,9 +103,7 @@ concrete_progress_meter::concrete_progress_meter
os_ << title << std::flush;
}
-concrete_progress_meter::~concrete_progress_meter()
-{
-}
+concrete_progress_meter::~concrete_progress_meter() = default;
std::string concrete_progress_meter::progress_message() const
{
diff --git a/progress_meter_wx.cpp b/progress_meter_wx.cpp
index 2ce816a..c64c70d 100644
--- a/progress_meter_wx.cpp
+++ b/progress_meter_wx.cpp
@@ -102,9 +102,7 @@ concrete_progress_meter::concrete_progress_meter
}
}
-concrete_progress_meter::~concrete_progress_meter()
-{
-}
+concrete_progress_meter::~concrete_progress_meter() = default;
/// Sleep for the number of seconds given in the argument.
///
diff --git a/rate_table.cpp b/rate_table.cpp
index 7c6d4c4..0106078 100644
--- a/rate_table.cpp
+++ b/rate_table.cpp
@@ -2361,7 +2361,6 @@ class database_impl
return fs::change_extension(path, ".dat");
}
- database_impl();
explicit database_impl(fs::path const& path);
database_impl(std::istream& index_is, shared_ptr<std::istream> data_is);
@@ -2474,10 +2473,6 @@ class database_impl
shared_ptr<std::istream> data_is_;
};
-database_impl::database_impl()
-{
-}
-
database_impl::database_impl(fs::path const& path)
:path_(path)
{
@@ -3000,7 +2995,7 @@ bool database::exists(fs::path const& path)
}
database::database()
- :impl_(new database_impl())
+ :impl_(new database_impl(fs::path()))
{
}
diff --git a/rounding_document.cpp b/rounding_document.cpp
index e5af984..47c8588 100644
--- a/rounding_document.cpp
+++ b/rounding_document.cpp
@@ -68,9 +68,7 @@ RoundingDocument::RoundingDocument()
values_["min_init_premium" ] = &rounding_rules_.round_min_init_premium_ ;
}
-RoundingDocument::~RoundingDocument()
-{
-}
+RoundingDocument::~RoundingDocument() = default;
void RoundingDocument::ReadDocument(std::string const& filename)
{
diff --git a/rounding_rules.cpp b/rounding_rules.cpp
index bfd93c7..434629b 100644
--- a/rounding_rules.cpp
+++ b/rounding_rules.cpp
@@ -110,9 +110,7 @@ rounding_parameters::rounding_parameters
{
}
-rounding_parameters::~rounding_parameters()
-{
-}
+rounding_parameters::~rounding_parameters() = default;
bool rounding_parameters::operator==(rounding_parameters const& z) const
{
@@ -203,9 +201,7 @@ rounding_rules::rounding_rules(std::string const& filename)
LMI_ASSERT(r_not_at_all == round_min_init_premium_ .style() || r_upward
== round_min_init_premium_ .style());
}
-rounding_rules::~rounding_rules()
-{
-}
+rounding_rules::~rounding_rules() = default;
/// Member datum nominated by the given name.
diff --git a/rounding_view.cpp b/rounding_view.cpp
index c46d7d0..6126b21 100644
--- a/rounding_view.cpp
+++ b/rounding_view.cpp
@@ -40,9 +40,7 @@ RoundingView::RoundingView()
{
}
-RoundingView::~RoundingView()
-{
-}
+RoundingView::~RoundingView() = default;
wxWindow* RoundingView::CreateChildWindow()
{
diff --git a/rounding_view_editor.cpp b/rounding_view_editor.cpp
index a9aa7ec..207f326 100644
--- a/rounding_view_editor.cpp
+++ b/rounding_view_editor.cpp
@@ -250,9 +250,7 @@ wxSize RoundingButtons::CalculateMinimumTextControlSize
return size;
}
-RoundingButtons::~RoundingButtons()
-{
-}
+RoundingButtons::~RoundingButtons() = default;
bool RoundingButtons::IsModified() const
{
diff --git a/rtti_lmi.hpp b/rtti_lmi.hpp
index 0b481f7..f011f9b 100644
--- a/rtti_lmi.hpp
+++ b/rtti_lmi.hpp
@@ -127,7 +127,7 @@ class TypeInfo
public:
TypeInfo(std::type_info const& z): ti_(&z) {}
- ~TypeInfo() {}
+ ~TypeInfo() = default;
bool operator==(TypeInfo const& z) const {return *z.ti_ == *ti_;}
bool operator<(TypeInfo const& z) const {return ti_->before(*z.ti_);}
diff --git a/safely_dereference_as_test.cpp b/safely_dereference_as_test.cpp
index c6d43a9..e1ff6a8 100644
--- a/safely_dereference_as_test.cpp
+++ b/safely_dereference_as_test.cpp
@@ -27,7 +27,7 @@
struct B
{
- virtual ~B(){}
+ virtual ~B()= default;
};
struct D
diff --git a/single_cell_document.cpp b/single_cell_document.cpp
index d3eeb51..c965a77 100644
--- a/single_cell_document.cpp
+++ b/single_cell_document.cpp
@@ -60,9 +60,7 @@ single_cell_document::single_cell_document(std::string const&
filename)
}
//============================================================================
-single_cell_document::~single_cell_document()
-{
-}
+single_cell_document::~single_cell_document() = default;
/// Backward-compatibility serial number of this class's xml version.
///
diff --git a/single_choice_popup_menu.cpp b/single_choice_popup_menu.cpp
index d014e1e..22cd709 100644
--- a/single_choice_popup_menu.cpp
+++ b/single_choice_popup_menu.cpp
@@ -60,9 +60,7 @@ SingleChoicePopupMenu::SingleChoicePopupMenu
);
}
-SingleChoicePopupMenu::~SingleChoicePopupMenu()
-{
-}
+SingleChoicePopupMenu::~SingleChoicePopupMenu() = default;
// WX !! Can't be const because PopupMenu() isn't.
int SingleChoicePopupMenu::Choose()
diff --git a/skeleton.cpp b/skeleton.cpp
index 068a855..0e2a13b 100644
--- a/skeleton.cpp
+++ b/skeleton.cpp
@@ -214,9 +214,7 @@ Skeleton::Skeleton()
timer_.Start(100);
}
-Skeleton::~Skeleton()
-{
-}
+Skeleton::~Skeleton() = default;
wxMDIChildFrame* Skeleton::CreateChildFrame
(wxDocument* doc
diff --git a/stratified_charges.cpp b/stratified_charges.cpp
index 3064f93..ab5634d 100644
--- a/stratified_charges.cpp
+++ b/stratified_charges.cpp
@@ -84,9 +84,7 @@ template<> stratified_entity
value_cast<stratified_entity>(std::string const&)
// Class stratified_entity implementation.
-stratified_entity::stratified_entity()
-{
-}
+stratified_entity::stratified_entity() = default;
stratified_entity::stratified_entity
(std::vector<double> const& limits
@@ -100,9 +98,7 @@ stratified_entity::stratified_entity
assert_validity();
}
-stratified_entity::~stratified_entity()
-{
-}
+stratified_entity::~stratified_entity() = default;
bool stratified_entity::operator==(stratified_entity const& z) const
{
@@ -190,9 +186,7 @@ stratified_charges::stratified_charges(stratified_charges
const& z)
MemberSymbolTable<stratified_charges>::assign(z);
}
-stratified_charges::~stratified_charges()
-{
-}
+stratified_charges::~stratified_charges() = default;
stratified_charges& stratified_charges::operator=(stratified_charges const& z)
{
diff --git a/test_coding_rules.cpp b/test_coding_rules.cpp
index 41948b6..72010eb 100644
--- a/test_coding_rules.cpp
+++ b/test_coding_rules.cpp
@@ -87,7 +87,7 @@ class file
{
public:
explicit file(std::string const& file_path);
- ~file() {}
+ ~file() = default;
bool is_of_phylum(enum_phylum ) const;
bool is_of_phylum(enum_kingdom) const;
@@ -984,7 +984,7 @@ class statistics
{
public:
statistics() : files_(0), lines_(0), defects_(0) {}
- ~statistics() {}
+ ~statistics() = default;
statistics& operator+=(statistics const&);
diff --git a/text_doc.cpp b/text_doc.cpp
index 2a83a7b..6d3339a 100644
--- a/text_doc.cpp
+++ b/text_doc.cpp
@@ -44,9 +44,7 @@ TextEditDocument::TextEditDocument()
{
}
-TextEditDocument::~TextEditDocument()
-{
-}
+TextEditDocument::~TextEditDocument() = default;
wxTextCtrl& TextEditDocument::PredominantViewWindow() const
{
diff --git a/text_view.cpp b/text_view.cpp
index 797ae41..5bceab7 100644
--- a/text_view.cpp
+++ b/text_view.cpp
@@ -45,9 +45,7 @@ TextEditView::TextEditView()
{
}
-TextEditView::~TextEditView()
-{
-}
+TextEditView::~TextEditView() = default;
wxWindow* TextEditView::CreateChildWindow()
{
diff --git a/tier_document.cpp b/tier_document.cpp
index 1c04536..1ada783 100644
--- a/tier_document.cpp
+++ b/tier_document.cpp
@@ -43,9 +43,7 @@ TierDocument::TierDocument()
initialize_charges();
}
-TierDocument::~TierDocument()
-{
-}
+TierDocument::~TierDocument() = default;
void TierDocument::initialize_charges()
{
diff --git a/tier_view.cpp b/tier_view.cpp
index 0bcf857..3f7cdf3 100644
--- a/tier_view.cpp
+++ b/tier_view.cpp
@@ -69,7 +69,7 @@ class tier_tree_item_data
{
public:
tier_tree_item_data(tier_entity_info const&);
- ~tier_tree_item_data() override {}
+ ~tier_tree_item_data() override = default;
tier_entity_info const& entity_info() const;
@@ -102,9 +102,7 @@ TierView::TierView()
{
}
-TierView::~TierView()
-{
-}
+TierView::~TierView() = default;
wxTreeCtrl* TierView::CreateTreeCtrl(wxWindow* parent)
{
diff --git a/tier_view_editor.cpp b/tier_view_editor.cpp
index 0ba2e67..c44c624 100644
--- a/tier_view_editor.cpp
+++ b/tier_view_editor.cpp
@@ -191,13 +191,9 @@ MultiDimTableAny::AxesAny TierTableAdapter::DoGetAxesAny()
// TierEditorGrid implementation
// ---------------------------
-TierEditorGrid::TierEditorGrid()
-{
-}
+TierEditorGrid::TierEditorGrid() = default;
-TierEditorGrid::~TierEditorGrid()
-{
-}
+TierEditorGrid::~TierEditorGrid() = default;
TierEditorGrid::TierEditorGrid
(wxWindow* parent
diff --git a/tier_view_editor.hpp b/tier_view_editor.hpp
index a4e47c5..b90cd7c 100644
--- a/tier_view_editor.hpp
+++ b/tier_view_editor.hpp
@@ -254,9 +254,7 @@ inline
TierTableAdapter::TierTableAdapter(tier_entity_adapter entity)
{
SetTierEntity(entity);
}
-inline TierTableAdapter::~TierTableAdapter()
-{
-}
+inline TierTableAdapter::~TierTableAdapter() = default;
inline tier_entity_adapter TierTableAdapter::GetTierEntity() const
{
diff --git a/timer.cpp b/timer.cpp
index 54eb547..e565201 100644
--- a/timer.cpp
+++ b/timer.cpp
@@ -79,9 +79,7 @@ Timer::Timer()
start();
}
-Timer::~Timer()
-{
-}
+Timer::~Timer() = default;
/// Set elapsed time to zero, and restart timer.
diff --git a/tn_range.hpp b/tn_range.hpp
index d67f831..df8b8d8 100644
--- a/tn_range.hpp
+++ b/tn_range.hpp
@@ -83,7 +83,7 @@ class trammel_base
friend class tn_range_test;
public:
- virtual ~trammel_base() {}
+ virtual ~trammel_base() = default;
void assert_sanity () const;
T minimum_minimorum () const;
@@ -91,9 +91,9 @@ class trammel_base
T maximum_maximorum () const;
protected:
- trammel_base() {}
- trammel_base(trammel_base const&) {}
- trammel_base& operator=(trammel_base const&) {return *this;}
+ trammel_base() = default;
+ trammel_base(trammel_base const&) = default;
+ trammel_base& operator=(trammel_base const&) = default;
private:
virtual T nominal_minimum() const = 0;
diff --git a/transferor.cpp b/transferor.cpp
index fdd4b38..f264536 100644
--- a/transferor.cpp
+++ b/transferor.cpp
@@ -91,9 +91,7 @@ Transferor::Transferor(std::string& data, std::string const&
name)
{
}
-Transferor::~Transferor()
-{
-}
+Transferor::~Transferor() = default;
wxObject* Transferor::Clone() const
{
diff --git a/uncopyable_lmi.hpp b/uncopyable_lmi.hpp
index b674683..d53cb78 100644
--- a/uncopyable_lmi.hpp
+++ b/uncopyable_lmi.hpp
@@ -96,7 +96,7 @@ template<typename T>
class uncopyable
{
protected:
- uncopyable() {}
+ uncopyable() = default;
~uncopyable()
{
// Double parentheses: don't parse comma as a macro parameter
separator.
diff --git a/view_ex.cpp b/view_ex.cpp
index 857396f..18e16b3 100644
--- a/view_ex.cpp
+++ b/view_ex.cpp
@@ -64,9 +64,7 @@ ViewEx::ViewEx()
{
}
-ViewEx::~ViewEx()
-{
-}
+ViewEx::~ViewEx() = default;
DocManagerEx& ViewEx::DocManager() const
{
diff --git a/wx_test_case.hpp b/wx_test_case.hpp
index 1fdbac4..afbc8a9 100644
--- a/wx_test_case.hpp
+++ b/wx_test_case.hpp
@@ -45,7 +45,7 @@ class wx_base_test_case
// Only required to fix g++ warning about a class having virtual functions
// but a non-virtual dtor, as this class is not used polymorphically the
// dtor doesn't really need to be virtual.
- virtual ~wx_base_test_case() { }
+ virtual ~wx_base_test_case() = default;
/// Skip the test if the specified file is not supported.
///
diff --git a/xml_lmi.cpp b/xml_lmi.cpp
index 9752c2f..cca5fb5 100644
--- a/xml_lmi.cpp
+++ b/xml_lmi.cpp
@@ -130,8 +130,7 @@ xml_lmi::dom_parser::dom_parser(std::istream const& is)
/// Throws: nothing unless member parser_'s destructor does.
-xml_lmi::dom_parser::~dom_parser()
-{}
+xml_lmi::dom_parser::~dom_parser() = default;
/// Return the parsed document.
///
@@ -201,8 +200,7 @@ xml_lmi::xml_document::xml_document(std::string const&
root_node_name)
{
}
-xml_lmi::xml_document::~xml_document()
-{}
+xml_lmi::xml_document::~xml_document() = default;
xml::element& xml_lmi::xml_document::root_node()
{
diff --git a/yare_input.cpp b/yare_input.cpp
index d7d6705..00e1371 100644
--- a/yare_input.cpp
+++ b/yare_input.cpp
@@ -202,9 +202,7 @@ yare_input::yare_input(Input const& z)
CorporationPaymentStrategy = convert_vector_type<mcenum_pmt_strategy
>(z.CorporationPaymentStrategyRealized_ );
}
-yare_input::~yare_input()
-{
-}
+yare_input::~yare_input() = default;
bool is_policy_rated(yare_input const& z)
{
| [Prev in Thread] |
Current Thread |
[Next in Thread] |
- [lmi-commits] [lmi] master f9af151: Use C++11 default keyword instead of empty ctor/dtor bodies (VZ),
Greg Chicares <=