Single key -= is identical to erase(), remove it

Author: d-frey

include/tao/json/object_operators.hh

@@ -24,15 +24,6 @@ namespace tao
          return v;
       }
 
-      template< template< typename ... > class Traits >
-      basic_value< Traits > & operator-= ( basic_value< Traits > & v, const std::string & k )
-      {
-         if ( v.get_object().erase( k ) == 0 ) {
-            throw std::runtime_error( "key not found: " + k );
-         }
-         return v;
-      }
-
       template< template< typename ... > class Traits >
       basic_value< Traits > & operator-= ( basic_value< Traits > & v, std::initializer_list< std::string > l )
       {

src/test/json/object_operators.cc

@@ -23,13 +23,13 @@ namespace tao
          TEST_THROWS( v -= { "what?" } );
          v -= {};
          TEST_ASSERT( v.get_object().size() == 3 );
-         v -= "bar";
-         TEST_THROWS( v -= "bar" );
+         v -= { "bar" };
          TEST_THROWS( v -= { "bar" } );
          TEST_ASSERT( v.get_object().size() == 2 );
          v -= { "foo", "baz" };
          TEST_ASSERT( v.get_object().size() == 0 );
-         TEST_THROWS( v -= "foo" );
+         TEST_THROWS( v -= { "foo" } );
+         TEST_THROWS( v -= { "bar" } );
          TEST_THROWS( v -= { "baz" } );
       }