Style (avoid new), work around bogus vc++ warning.

Author: evoskuil

include/bitcoin/system/impl/machine/interpreter.ipp

@@ -1649,7 +1649,6 @@ connect(const context& state, const transaction& tx,
     return error::script_success;
 }
 
-BC_PUSH_WARNING(NO_NEW_OR_DELETE)
 BC_PUSH_WARNING(NO_THROW_IN_NOEXCEPT)
 
 template <typename Stack>
@@ -1695,7 +1694,6 @@ code interpreter<Stack>::connect_embedded(const context& state,
     return error::script_success;
 }
 
-BC_POP_WARNING()
 BC_POP_WARNING()
 
 template <typename Stack>

src/chain/input.cpp

@@ -48,19 +48,14 @@ static_assert(max_script_size <
 // static/private
 const witness& input::no_witness() NOEXCEPT
 {
-    static const chain::witness empty_witness{};
-    return empty_witness;
+    static const chain::witness empty{};
+    return empty;
 }
 
 // static/private
 const witness::cptr& input::no_witness_cptr() NOEXCEPT
 {
-    BC_PUSH_WARNING(NO_NEW_OR_DELETE)
-    static const std::shared_ptr<const chain::witness> empty
-    {
-        new const chain::witness{}
-    };
-    BC_POP_WARNING()
+    static const auto empty = std::make_shared<const chain::witness>();
     return empty;
 }
 

src/chain/operation.cpp

@@ -45,32 +45,22 @@ static constexpr auto any_invalid = opcode::op_xor;
 // static/private
 const data_chunk& operation::no_data() NOEXCEPT
 {
-    static const data_chunk empty_data{};
-    return empty_data;
+    static const data_chunk empty{};
+    return empty;
 }
 
 // static/private
 const chunk_cptr& operation::no_data_cptr() NOEXCEPT
 {
-    BC_PUSH_WARNING(NO_NEW_OR_DELETE)
-    static const std::shared_ptr<const data_chunk> empty
-    {
-        new const data_chunk{}
-    };
-    BC_POP_WARNING()
+    static const auto empty = std::make_shared<const data_chunk>();
     return empty;
 }
 
 // static/private
 // Push data is not possible with an invalid code, combination is invalid.
 const chunk_cptr& operation::any_data_cptr() NOEXCEPT
 {
-    BC_PUSH_WARNING(NO_NEW_OR_DELETE)
-    static const std::shared_ptr<const data_chunk> any
-    {
-        new const data_chunk{ 0x42 }
-    };
-    BC_POP_WARNING()
+    static const auto any = to_shared(data_chunk{ 0x42 });
     return any;
 }
 

src/crypto/pseudo_random.cpp

@@ -30,7 +30,6 @@ namespace libbitcoin {
 namespace system {
 
 BC_PUSH_WARNING(NO_STATIC_CAST)
-BC_PUSH_WARNING(NO_NEW_OR_DELETE)
 BC_PUSH_WARNING(NO_THROW_IN_NOEXCEPT)
 BC_PUSH_WARNING(NO_CASTS_FOR_ARITHMETIC_CONVERSION)
 
@@ -104,7 +103,6 @@ steady_clock::duration pseudo_random::duration(
 BC_POP_WARNING()
 BC_POP_WARNING()
 BC_POP_WARNING()
-BC_POP_WARNING()
 
 } // namespace system
 } // namespace libbitcoin