Removed Contains

src/Swift.Runtime/src/Metadata/ITypeMetadataCache.cs

@@ -12,27 +12,20 @@ namespace Swift.Runtime;
 public interface ITypeMetadataCache
 {
     /// <summary>
-    /// Returns true if and only if the cache contains an entry for t
-    /// </summary>
-    /// <param name="type">The type to look up in the cache</param>
-    /// <returns>true if and only if the cache contains t, false otherwise</returns>
-    bool Contains(Type type);
-
-    /// <summary>
-    /// Returns true if the cache contains an entry for t and sets metadata to the resulting value,
+    /// Returns true if the cache contains an entry for type and sets metadata to the resulting value,
     /// otherwise it returns false and metadata will be null.
     /// </summary>
-    /// <param name="type"></param>
-    /// <param name="metadata"></param>
+    /// <param name="type">The type to look up in the cache</param>
+    /// <param name="metadata">The resulting metadata if found</param>
     /// <returns>true if the lookup was successful, false otherwise</returns>
     bool TryGet(Type type, [NotNullWhen(true)]out TypeMetadata? metadata);
 
     /// <summary>
-    /// Gets the TypeMetadata for the given Type t or if it is not present,
+    /// Gets the TypeMetadata for the given Type type or if it is not present,
     /// adds it to the cache using the given factory to generate the value.
     /// </summary>
-    /// <param name="type"></param>
-    /// <param name="metadataFactory"></param>
-    /// <returns>The TypeMetadata associated with the give Type t</returns>
+    /// <param name="type">The type to look up in the cache</param>
+    /// <param name="metadataFactory">a factory to generate the TypeMetadata if not present</param>
+    /// <returns>The TypeMetadata associated with the give Type type</returns>
     TypeMetadata GetOrAdd(Type type, Func<Type, TypeMetadata> metadataFactory);
 }

src/Swift.Runtime/src/Metadata/TypeMetadataCache.cs

@@ -35,23 +35,12 @@ public TypeMetadataCache(IEnumerable<(Type, TypeMetadata)> initialValues)
         }
     }
 
-
     /// <summary>
-    /// Returns true if and only if the cache contains an entry for t.
-    /// </summary>
-    /// <param name="type">The type to look up in the cache</param>
-    /// <returns>true if and only if the cache contains t, false otherwise</returns>
-    public bool Contains(Type type)
-    {
-        return cache.ContainsKey(type);
-    }
-
-    /// <summary>
-    /// Returns true if the cache contains an entry for t and sets metadata to the resulting value,
+    /// Returns true if the cache contains an entry for type and sets metadata to the resulting value,
     /// otherwise it returns false and metadata will be null.
     /// </summary>
-    /// <param name="type"></param>
-    /// <param name="metadata"></param>
+    /// <param name="type">The type to look up in the cache</param>
+    /// <param name="metadata">The resulting metadata if found</param>
     /// <returns>true if the lookup was successful, false otherwise</returns>
     public bool TryGet(Type type, [NotNullWhen(true)] out TypeMetadata? metadata)
     {
@@ -68,12 +57,12 @@ public bool TryGet(Type type, [NotNullWhen(true)] out TypeMetadata? metadata)
     }
 
     /// <summary>
-    /// Gets the TypeMetadata for the given Type t or if it is not present,
+    /// Gets the TypeMetadata for the given Type type or if it is not present,
     /// adds it to the cache using the given factory to generate the value.
     /// </summary>
-    /// <param name="type"></param>
-    /// <param name="metadataFactory"></param>
-    /// <returns>The TypeMetadata associated with the give Type t</returns>
+    /// <param name="type">The type to look up in the cache</param>
+    /// <param name="metadataFactory">a factory to generate the TypeMetadata if not present</param>
+    /// <returns>The TypeMetadata associated with the give Type type</returns>
     public TypeMetadata GetOrAdd(Type type, Func<Type, TypeMetadata> metadataFactory)
     {
         return cache.GetOrAdd(type, metadataFactory);

src/Swift.Runtime/tests/TypeMetadataTests/TypeMetadataTests.cs

@@ -37,12 +37,6 @@ static TypeMetadata MakePhonyMetadata(int value)
         return (TypeMetadata)(ci.Invoke(new object[] { p }));
     }
 
-    [Fact]
-    public static void CacheDoesntContain()
-    {
-        Assert.False(TypeMetadata.Cache.Contains(typeof(System.EventArgs)));
-    }
-
     [Fact]
     public static void CacheWorks()
     {
@@ -51,7 +45,7 @@ public static void CacheWorks()
         {
             return fakeMeta;
         });
-        Assert.True(TypeMetadata.Cache.Contains(typeof(System.Convert)));
+        Assert.True(TypeMetadata.Cache.TryGet(typeof(System.Convert), out var result));
     }
 
     [Fact]