[Java.Interop.Tools.Cecil] Unroll TypeDefinitionRocks iterators.

Author: jpobst

src/Java.Interop.Tools.Cecil/Java.Interop.Tools.Cecil.csproj

@@ -2,7 +2,7 @@
 
   <PropertyGroup>
     <TargetFramework>netstandard2.0</TargetFramework>
-    <LangVersion>8.0</LangVersion>
+    <LangVersion>9.0</LangVersion>
     <Nullable>enable</Nullable>
     <SignAssembly>true</SignAssembly>
     <AssemblyOriginatorKeyFile>..\..\product.snk</AssemblyOriginatorKeyFile>

src/Java.Interop.Tools.Cecil/Java.Interop.Tools.Cecil/MethodDefinitionRocks.cs

@@ -20,7 +20,9 @@ public static MethodDefinition GetBaseDefinition (this MethodDefinition method,
 			if (method.IsStatic || method.IsNewSlot || !method.IsVirtual)
 				return method;
 
-			foreach (var baseType in method.DeclaringType.GetBaseTypes (resolver)) {
+			TypeDefinition? baseType = method.DeclaringType;
+
+			while ((baseType = baseType.GetBaseType (resolver)) != null) {
 				foreach (var m in baseType.Methods) {
 					if (!m.IsConstructor &&
 							m.Name == method.Name &&

src/Java.Interop.Tools.Cecil/Java.Interop.Tools.Cecil/TypeDefinitionRocks.cs

@@ -65,14 +65,19 @@ public static bool IsAssignableFrom (this TypeReference type, TypeReference c, I
 			var d = resolver.Resolve (c);
 			if (d == null)
 				return false;
-			foreach (var t in d.GetTypeAndBaseTypes (resolver)) {
+
+			TypeDefinition? t = d;
+
+			while (t is not null) {
 				if (type.FullName == t.FullName)
 					return true;
 				foreach (var ifaceImpl in t.Interfaces) {
 					var i   = ifaceImpl.InterfaceType;
 					if (IsAssignableFrom (type, i, resolver))
 						return true;
 				}
+
+				t = t.GetBaseType (resolver);
 			}
 			return false;
 		}
@@ -84,20 +89,26 @@ public static bool IsSubclassOf (this TypeDefinition type, string typeName, Type
 			IsSubclassOf (type, typeName, (IMetadataResolver) cache);
 		public static bool IsSubclassOf (this TypeDefinition type, string typeName, IMetadataResolver resolver)
 		{
-			foreach (var t in type.GetTypeAndBaseTypes (resolver)) {
+			TypeDefinition? t = type;
+
+			while (t is not null) {
 				if (t.FullName == typeName) {
 					return true;
 				}
+
+				t = t.GetBaseType (resolver);
 			}
 			return false;
 		}
 
 		public static bool HasJavaPeer (this TypeDefinition type, IMetadataResolver resolver)
 		{
-			if (type.IsInterface && type.ImplementsInterface ("Java.Interop.IJavaPeerable", resolver))
-				return true;
+			if (type.IsInterface)
+				return type.ImplementsInterface ("Java.Interop.IJavaPeerable", resolver);
 
-			foreach (var t in type.GetTypeAndBaseTypes (resolver)) {
+			TypeDefinition? t = type;
+
+			while (t is not null) {
 				switch (t.FullName) {
 					case "Java.Lang.Object":
 					case "Java.Lang.Throwable":
@@ -107,6 +118,8 @@ public static bool HasJavaPeer (this TypeDefinition type, IMetadataResolver reso
 					default:
 						break;
 				}
+
+				t = t.GetBaseType (resolver);
 			}
 			return false;
 		}
@@ -119,12 +132,16 @@ public static bool ImplementsInterface (this TypeDefinition type, string interfa
 
 		public static bool ImplementsInterface (this TypeDefinition type, string interfaceName, IMetadataResolver resolver)
 		{
-			foreach (var t in type.GetTypeAndBaseTypes (resolver)) {
+			TypeDefinition? t = type;
+
+			while (t is not null) {
 				foreach (var i in t.Interfaces) {
 					if (i.InterfaceType.FullName == interfaceName) {
 						return true;
 					}
 				}
+
+				t = t.GetBaseType (resolver);
 			}
 			return false;
 		}

src/Java.Interop.Tools.JavaCallableWrappers/Java.Interop.Tools.JavaCallableWrappers.Adapters/CecilImporter.cs

@@ -124,7 +124,9 @@ static CallableWrapperType CreateType (TypeDefinition type, IMetadataResolver re
 			type,
 		};
 
-		foreach (var bt in type.GetBaseTypes (resolver)) {
+		TypeDefinition? bt = type;
+
+		while ((bt = bt.GetBaseType (resolver)) != null) {
 			ctorTypes.Add (bt);
 			var rattr = CecilExtensions.GetTypeRegistrationAttributes (bt).FirstOrDefault ();
 

src/Java.Interop.Tools.TypeNameMappings/Java.Interop.Tools.TypeNameMappings/JavaNativeTypeManager.cs

@@ -438,7 +438,7 @@ public static bool IsApplication (TypeDefinition type, TypeDefinitionCache cache
 
 		public static bool IsApplication (TypeDefinition type, IMetadataResolver resolver)
 		{
-			return type.GetBaseTypes (resolver).Any (b => b.FullName == "Android.App.Application");
+			return type.IsSubclassOf ("Android.App.Application", resolver);
 		}
 
 		[Obsolete ("Use the TypeDefinitionCache overload for better performance.", error: true)]
@@ -449,7 +449,7 @@ public static bool IsInstrumentation (TypeDefinition type, TypeDefinitionCache c
 
 		public static bool IsInstrumentation (TypeDefinition type, IMetadataResolver resolver)
 		{
-			return type.GetBaseTypes (resolver).Any (b => b.FullName == "Android.App.Instrumentation");
+			return type.IsSubclassOf ("Android.App.Instrumentation", resolver);
 		}
 
 		// moved from JavaTypeInfo
@@ -734,9 +734,10 @@ internal static bool IsNonStaticInnerClass (TypeDefinition? type, IMetadataResol
 			if (!type.DeclaringType.HasJavaPeer (cache))
 				return false;
 
-			foreach (var baseType in type.GetBaseTypes (cache)) {
-				if (baseType == null)
-					continue;
+			TypeDefinition? baseType = type;
+
+			while ((baseType = baseType.GetBaseType (cache)) != null) {
+
 				if (!baseType.AnyCustomAttributes (typeof (RegisterAttribute)))
 					continue;