Merge pull request #1104 from Geode-solutions/feat/conversion-from-uv

fix(Basic): add missing functions in SmallSet

Author: MelchiorSchuh

include/geode/basic/small_set.hpp

@@ -55,16 +55,26 @@ namespace geode
             return container_.end();
         }
 
+        auto contains( const Type& element ) const
+        {
+            return absl::c_contains( container_, element );
+        }
+
         auto insert( const Type& element )
         {
-            if( absl::c_contains( container_, element ) )
+            if( contains( element ) )
             {
                 return false;
             }
             container_.push_back( element );
             return true;
         }
 
+        auto erase( const Type& element )
+        {
+            return container_.erase( absl::c_find( container_, element ) );
+        }
+
         auto at( index_t index ) const
         {
             return container_.at( index );

tests/basic/test-small-set.cpp

@@ -37,6 +37,9 @@ void test()
     OPENGEODE_EXCEPTION( !set.insert( 0 ), "[Test] Insert not allow" );
     OPENGEODE_EXCEPTION( !set.insert( 1 ), "[Test] Insert not allow" );
     OPENGEODE_EXCEPTION( set.size(), "[Test] Set size should be 2" );
+    set.erase( 0 );
+    OPENGEODE_EXCEPTION( set.size(), "[Test] Set size should be 1" );
+    OPENGEODE_EXCEPTION( set.at( 0 ) == 1, "[Test] Wrong value in set" );
 }
 
 OPENGEODE_TEST( "small-set" )