Assign nullptr to the smart pointers #203

Author: kaorut

library/json/test/src/test_tetengo.json.json_parser.cpp

@@ -152,8 +152,7 @@ BOOST_AUTO_TEST_CASE(construction)
         const tetengo::json::json_parser parser{ std::move(p_reader), 42 };
     }
     {
-        BOOST_CHECK_THROW(
-            const tetengo::json::json_parser parser{ std::unique_ptr<tetengo::json::reader>{} }, std::invalid_argument);
+        BOOST_CHECK_THROW(const tetengo::json::json_parser parser{ nullptr }, std::invalid_argument);
     }
 
     {

library/lattice/c/src/tetengo_lattice_vocabulary.cpp

@@ -78,7 +78,7 @@ tetengo_lattice_vocabulary_t* tetengo_lattice_vocabulary_createUnorderedMapVocab
                 BOOST_SCOPE_EXIT_END;
                 std::any cpp_entry_value{ entry.p_value };
                 cpp_entry_values.emplace_back(
-                    p_cpp_entry_key ? p_cpp_entry_key->cpp_input().clone() : std::unique_ptr<tetengo::lattice::input>{},
+                    p_cpp_entry_key ? p_cpp_entry_key->cpp_input().clone() : nullptr,
                     std::move(cpp_entry_value),
                     entry.cost);
             }
@@ -107,12 +107,10 @@ tetengo_lattice_vocabulary_t* tetengo_lattice_vocabulary_createUnorderedMapVocab
             BOOST_SCOPE_EXIT_END;
             std::any cpp_to_value{ connection_element.p_to->p_value };
             auto     cpp_key = std::make_pair(
-                tetengo::lattice::entry{ p_cpp_from_key ? p_cpp_from_key->cpp_input().clone() :
-                                                              std::unique_ptr<tetengo::lattice::input>{},
+                tetengo::lattice::entry{ p_cpp_from_key ? p_cpp_from_key->cpp_input().clone() : nullptr,
                                          std::move(cpp_from_value),
                                          connection_element.p_from->cost },
-                tetengo::lattice::entry{ p_cpp_to_key ? p_cpp_to_key->cpp_input().clone() :
-                                                            std::unique_ptr<tetengo::lattice::input>{},
+                tetengo::lattice::entry{ p_cpp_to_key ? p_cpp_to_key->cpp_input().clone() : nullptr,
                                          std::move(cpp_to_value),
                                          connection_element.p_to->cost });
 

library/lattice/cpp/src/tetengo.lattice.entry.cpp

@@ -16,7 +16,7 @@ namespace tetengo::lattice
 {
     const entry& entry::bos_eos()
     {
-        static const entry singleton{ std::unique_ptr<input>{}, std::any{}, 0 };
+        static const entry singleton{ nullptr, std::any{}, 0 };
         return singleton;
     }
 
@@ -27,13 +27,13 @@ namespace tetengo::lattice
     {}
 
     entry::entry(const entry_view& view) :
-    m_p_key{ view.p_key() ? view.p_key()->clone() : std::unique_ptr<input>{} },
+    m_p_key{ view.p_key() ? view.p_key()->clone() : nullptr },
     m_value{ *view.value() },
     m_cost{ view.cost() }
     {}
 
     entry::entry(const entry& another) :
-    m_p_key{ another.m_p_key ? another.m_p_key->clone() : std::unique_ptr<tetengo::lattice::input>{} },
+    m_p_key{ another.m_p_key ? another.m_p_key->clone() : nullptr },
     m_value{ another.m_value },
     m_cost{ another.m_cost }
     {}

library/lattice/test/src/test_tetengo.lattice.input.cpp

@@ -193,7 +193,7 @@ BOOST_AUTO_TEST_CASE(append)
     concrete_input input_{};
 
     input_.append(std::make_unique<concrete_input>());
-    BOOST_CHECK_THROW(input_.append(std::unique_ptr<concrete_input>{}), std::invalid_argument);
+    BOOST_CHECK_THROW(input_.append(nullptr), std::invalid_argument);
     BOOST_CHECK_THROW(input_.append(std::make_unique<concrete_input2>()), std::invalid_argument);
 }
 

library/lattice/test/src/test_tetengo.lattice.string_input.cpp

@@ -513,7 +513,7 @@ BOOST_AUTO_TEST_CASE(append)
     {
         tetengo::lattice::string_input input{ "hoge" };
 
-        BOOST_CHECK_THROW(input.append(std::unique_ptr<tetengo::lattice::string_input>{}), std::invalid_argument);
+        BOOST_CHECK_THROW(input.append(nullptr), std::invalid_argument);
         BOOST_CHECK_THROW(input.append(std::make_unique<another_input>()), std::invalid_argument);
     }
 

library/trie/c/src/tetengo_trie_storage.cpp

@@ -112,7 +112,7 @@ namespace
         }
         catch (const boost::interprocess::interprocess_exception&)
         {
-            return std::unique_ptr<boost::interprocess::file_mapping>{};
+            return nullptr;
         }
     }
 

library/trie/cpp/include/tetengo/trie/trie.hpp

@@ -468,7 +468,7 @@ namespace tetengo::trie
             }();
             if (!p_trie_impl)
             {
-                return std::unique_ptr<trie>{};
+                return nullptr;
             }
             std::unique_ptr<trie> p_trie{ new trie{ std::move(p_trie_impl), m_key_serializer } };
             return p_trie;

library/trie/cpp/src/tetengo.trie.double_array.cpp

@@ -112,8 +112,7 @@ namespace tetengo::trie
         std::unique_ptr<double_array> subtrie(const std::string_view& key_prefix) const
         {
             const auto o_index = traverse(key_prefix);
-            return o_index ? std::make_unique<double_array>(m_p_storage->clone(), *o_index) :
-                             std::unique_ptr<double_array>{};
+            return o_index ? std::make_unique<double_array>(m_p_storage->clone(), *o_index) : nullptr;
         }
 
         const storage& get_storage() const

library/trie/cpp/src/tetengo.trie.trie.cpp

@@ -139,7 +139,7 @@ namespace tetengo::trie
             auto p_subtrie = m_p_double_array->subtrie(key_prefix);
             if (!p_subtrie)
             {
-                return std::unique_ptr<trie_impl>{};
+                return nullptr;
             }
             return std::make_unique<trie_impl>(std::move(p_subtrie));
         }