[lmi-commits] [lmi] master 66944d8 2/3: Use make_shared in the most obvious cases

[Greg Chicares]

branch: master
commit 66944d897233d5734bd75c6025b9ae1a1427a46b
Author: Gregory W. Chicares <gchicares@sbcglobal.net>
Commit: Gregory W. Chicares <gchicares@sbcglobal.net>

    Use make_shared in the most obvious cases
    
    Here, "most obvious" means identified by:
      git grep '_ptr.*new'
---
 database.cpp         | 2 +-
 input_test.cpp       | 2 +-
 ledger.cpp           | 4 ++--
 premium_tax_test.cpp | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/database.cpp b/database.cpp
index 4a21079..8fc29a6 100644
--- a/database.cpp
+++ b/database.cpp
@@ -226,7 +226,7 @@ namespace
 
 std::shared_ptr<DBDictionary const> antediluvian_db()
 {
-    std::shared_ptr<DBDictionary> z(new DBDictionary);
+    auto z = std::make_shared<DBDictionary>();
     z->InitAntediluvian();
     LMI_ASSERT(z);
     return z;
diff --git a/input_test.cpp b/input_test.cpp
index 394631e..3487192 100644
--- a/input_test.cpp
+++ b/input_test.cpp
@@ -101,7 +101,7 @@ void input_test::test_product_database()
     Input input;
     yare_input yi(input);
     product_database db(yi);
-    std::shared_ptr<DBDictionary> x(new DBDictionary);
+    auto x = std::make_shared<DBDictionary>();
     db.db_ = x;
     DBDictionary& dictionary = *x;
     dictionary.Add({DB_ChildRiderMinAmt, 25000});
diff --git a/ledger.cpp b/ledger.cpp
index d14a6ac..83ceadc 100644
--- a/ledger.cpp
+++ b/ledger.cpp
@@ -43,8 +43,8 @@
 //   Ledger Ledger::Clone() const
 //   {
 //       Ledger new_ledger(*this);
-//       new_ledger.ledger_map_       = std::shared_ptr<ledger_map_holder>(new 
ledger_map_holder(*ledger_map_));
-//       new_ledger.ledger_invariant_ = std::shared_ptr<LedgerInvariant>(new 
LedgerInvariant(*ledger_invariant_));
+//       new_ledger.ledger_map_       = 
std::make_shared<ledger_map_holder>(*ledger_map_);
+//       new_ledger.ledger_invariant_ = 
std::make_shared_ptr<LedgerInvariant>(*ledger_invariant_);
 //       return new_ledger;
 //   }
 // that would make unshared copies? If not, then Clone() would be
diff --git a/premium_tax_test.cpp b/premium_tax_test.cpp
index 1f101bf..67c56d7 100644
--- a/premium_tax_test.cpp
+++ b/premium_tax_test.cpp
@@ -99,7 +99,7 @@ void premium_tax_test::test_rates()
     // A uniform but nonzero load would elicit a runtime error,
     // because the tiered load is not zero.
     {
-    std::shared_ptr<DBDictionary> x(new DBDictionary);
+    auto x = std::make_shared<DBDictionary>();
     db.db_ = x;
     DBDictionary& dictionary = *x;