Remove dead code

Author: MitchRazga

src/Rocks.Analysis.Tests/Extensions/AttributeDataExtensionsTests.cs

@@ -58,31 +58,6 @@ public MyTestAttribute(string a, double b, int c, uint d, Type e, int[] f, MyVal
 
 	public static class AttributeDataExtensionsTests
 	{
-		[Test]
-		public static void GetNamespaces()
-		{
-			var (attributes, compilation) = AttributeDataExtensionsTests.GetAttributes(
-$$"""
-using {{typeof(TypeOfThis).Namespace}};
-using {{typeof(MethodAttribute).Namespace}};
-
-public interface IA
-{
-	[Method(typeof(TypeOfThis))]
-	void Foo();
-}
-""");
-
-			var namespaces = attributes[0].GetNamespaces();
-
-			Assert.Multiple(() =>
-			{
-				Assert.That(namespaces, Has.Count.EqualTo(2));
-				Assert.That(namespaces.Any(_ => _.Name == typeof(TypeOfThis).Namespace), Is.True);
-				Assert.That(namespaces.Any(_ => _.Name == typeof(MethodAttribute).Namespace), Is.True);
-			});
-		}
-
 		[Test]
 		public static void GetDescriptionWithSpecialCharactersInString()
 		{

src/Rocks.Analysis.Tests/Extensions/IEventSymbolGetNamespacesExtensionsTests.cs

@@ -74,26 +74,6 @@ public static void GetDefaultValue(string code, string expectedResult)
 		Assert.That(parameter.ExplicitDefaultValue.GetDefaultValue(parameter.Type, compilation), Is.EqualTo(expectedResult));
 	}
 
-	[Test]
-	public static void FormatValue() =>
-		Assert.That(33.FormatValue(), Is.EqualTo("33"));
-
-	[Test]
-	public static void FormatEnumValue() =>
-		Assert.That(ObjectExtensionValues.Some.FormatValue(), Is.EqualTo("Rocks.Analysis.Tests.Extensions.ObjectExtensionValues.Some"));
-
-	[Test]
-	public static void FormatEnumValueWhenValueIsIncorrect() =>
-		Assert.That(((ObjectExtensionValues)22).FormatValue(), Is.EqualTo("(Rocks.Analysis.Tests.Extensions.ObjectExtensionValues)22"));
-
-	[Test]
-	public static void FormatNullValue() =>
-		Assert.That((null as string).FormatValue(), Is.EqualTo("null"));
-
-	[Test]
-	public static void FormatCollectionValue() =>
-		Assert.That((new int[] { 1, 2, 3 }).FormatValue(), Is.EqualTo("System.Int32[], Count = 3"));
-
 	private static (IParameterSymbol, Compilation) GetParameterSymbol(string source)
 	{
 		var syntaxTree = CSharpSyntaxTree.ParseText(source);

src/Rocks.Analysis.Tests/Extensions/IMethodSymbolExtensionsGetNamespacesTests.cs

@@ -8,40 +8,6 @@ namespace Rocks.Analysis.Extensions;
 
 internal static class AttributeDataExtensions
 {
-	internal static ImmutableHashSet<INamespaceSymbol> GetNamespaces(this AttributeData self)
-	{
-		static IEnumerable<INamespaceSymbol> GetNamespacesForValue(TypedConstant value)
-		{
-			if (value.Kind == TypedConstantKind.Primitive || value.Kind == TypedConstantKind.Enum)
-			{
-				yield return value.Type!.ContainingNamespace;
-			}
-			else if (value.Kind == TypedConstantKind.Type)
-			{
-				yield return ((INamedTypeSymbol)value.Value!).ContainingNamespace;
-			}
-			else if (value.Kind == TypedConstantKind.Array)
-			{
-				yield return value.Type!.ContainingNamespace;
-
-				foreach (var arrayValue in value.Values)
-				{
-					foreach (var arrayValueNamespace in GetNamespacesForValue(arrayValue))
-					{
-						yield return arrayValueNamespace;
-					}
-				}
-			}
-		}
-
-		return
-		[
-			self.AttributeClass!.ContainingNamespace,
-			.. self.ConstructorArguments.SelectMany(_ => GetNamespacesForValue(_)),
-			.. self.NamedArguments.SelectMany(_ => GetNamespacesForValue(_.Value)),
-		];
-	}
-
 	internal static string GetDescription(this AttributeData self, Compilation compilation)
 	{
 		static string GetTypedConstantValue(TypedConstant value, Compilation compilation) =>

src/Rocks.Analysis.Tests/Extensions/ObjectExtensionsTests.cs

@@ -14,7 +14,4 @@ internal static bool CanBeSeenByContainingAssembly(this IEventSymbol self, IAsse
 		Compilation compilation) =>
 		((ISymbol)self).CanBeSeenByContainingAssembly(assembly, compilation) &&
 			self.Type.CanBeSeenByContainingAssembly(assembly, compilation);
-
-   internal static ImmutableHashSet<INamespaceSymbol> GetNamespaces(this IEventSymbol self) => 
-		[.. self.Type.GetNamespaces(), .. self.GetAttributes().SelectMany(_ => _.GetNamespaces())];
 }

src/Rocks.Analysis/Extensions/AttributeDataExtensions.cs

@@ -108,35 +108,6 @@ internal static EquatableArray<Constraints> GetConstraints(this IMethodSymbol se
 		return [.. constraints];
 	}
 
-	internal static ImmutableHashSet<INamespaceSymbol> GetNamespaces(this IMethodSymbol self)
-	{
-		static IEnumerable<INamespaceSymbol> GetParameterNamespaces(IParameterSymbol parameter)
-		{
-			foreach (var parameterTypeNamespace in parameter.Type.GetNamespaces())
-			{
-				yield return parameterTypeNamespace;
-			}
-
-			foreach (var attributeNamespace in parameter.GetAttributes().SelectMany(_ => _.GetNamespaces()))
-			{
-				yield return attributeNamespace;
-			}
-		}
-
-		var namespaces = new HashSet<INamespaceSymbol>();
-
-		if (!self.ReturnsVoid)
-		{
-			namespaces.AddRange(self.ReturnType.GetNamespaces());
-			namespaces.AddRange(self.GetReturnTypeAttributes().SelectMany(_ => _.GetNamespaces()));
-		}
-
-		namespaces.AddRange(self.GetAttributes().SelectMany(_ => _.GetNamespaces()));
-		namespaces.AddRange(self.Parameters.SelectMany(_ => GetParameterNamespaces(_)));
-
-		return [.. namespaces];
-	}
-
 	internal static MethodMatch Match(this IMethodSymbol self, IMethodSymbol other, Compilation compilation)
 	{
 		if (self.Name != other.Name)

src/Rocks.Analysis/Extensions/IEventSymbolExtensions.cs

@@ -32,34 +32,6 @@ internal static ImmutableArray<AttributeData> GetAllAttributes(this IPropertySym
 		return [.. attributes];
 	}
 
-	internal static ImmutableHashSet<INamespaceSymbol> GetNamespaces(this IPropertySymbol self)
-	{
-		static IEnumerable<INamespaceSymbol> GetParameterNamespaces(IParameterSymbol parameter)
-		{
-			foreach (var parameterTypeNamespaces in parameter.Type.GetNamespaces())
-			{
-				yield return parameterTypeNamespaces;
-			}
-
-			foreach (var attributeNamespace in parameter.GetAttributes().SelectMany(_ => _.GetNamespaces()))
-			{
-				yield return attributeNamespace;
-			}
-		}
-
-		var namespaces = new HashSet<INamespaceSymbol>();
-
-		namespaces.AddRange(self.GetAllAttributes().SelectMany(_ => _.GetNamespaces()));
-		namespaces.AddRange(self.Type.GetNamespaces());
-
-		if (self.IsIndexer)
-		{
-			namespaces.AddRange(self.Parameters.SelectMany(_ => GetParameterNamespaces(_)));
-		}
-
-		return [.. namespaces];
-	}
-
 	internal static bool IsUnsafe(this IPropertySymbol self) =>
 		self.IsIndexer ? (self.Parameters.Any(_ => _.Type.IsPointer()) || self.Type.IsPointer()) : self.Type.IsPointer();
 

src/Rocks.Analysis/Extensions/IMethodSymbolExtensions.cs

@@ -8,39 +8,6 @@ namespace Rocks.Analysis.Extensions;
 /// </summary>
 public static class ObjectExtensions
 {
-	// TODO: If I could project this code like I do for RefStructArgument,
-	// I might be able to do a FormatValue<T>(this T self) where T : allows ref struct.
-
-	/// <summary>
-	/// This gets a stringified version of a value
-	/// that will be put into the call site of an emitted method.
-	/// </summary>
-	public static string FormatValue(this object? self)
-	{
-		if (self is Enum)
-		{
-			var selfType = self.GetType();
-			var selfName = Enum.GetName(selfType, self);
-			var selfValue = selfName is not null ?
-				$"{selfType.FullName}.{selfName}" :
-				$"({selfType.FullName}){self}";
-
-			return selfValue;
-		}
-		else if (self is ICollection c)
-		{
-			return $"{c}, Count = {c.Count}";
-		}
-		else if (self is null)
-		{
-			return "null";
-		}
-		else
-		{
-			return self.ToString();
-		}
-	}
-
 	/// <summary>
 	/// This should only be used to get a stringified version of a default value
 	/// that will be put into the call site of an emitted method.