disabled CompileTimeMethodInfos because of test errors. Not all tests…

… were covered with the optimization.
vescon · Oct 18, 2021 · a801966 · a801966
1 parent 4853cd5
commit a801966
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/src/MethodBoundaryAspect.Fody/ModuleWeaver.cs b/src/MethodBoundaryAspect.Fody/ModuleWeaver.cs
@@ -40,7 +40,7 @@ public ModuleWeaver()
             InitLogging();
         }
 
-        public bool DisableCompileTimeMethodInfos { get; set; }
+        public bool DisableCompileTimeMethodInfos { get; set; } = true;
 
         public int TotalWeavedTypes { get; private set; }
         public int TotalWeavedMethods { get; private set; }
@@ -52,12 +52,7 @@ public ModuleWeaver()
 
         public override void Execute()
         {
-            _methodInfoCompileTimeWeaver = new MethodInfoCompileTimeWeaver(ModuleDefinition);
-            _methodInfoCompileTimeWeaver.IsEnabled = !DisableCompileTimeMethodInfos;
-
             Execute(ModuleDefinition);
-
-            _methodInfoCompileTimeWeaver.Finish();
         }
 
         public override IEnumerable<string> GetAssembliesForScanning()
@@ -149,10 +144,17 @@ private void InitLogging()
 
         private void Execute(ModuleDefinition module)
         {
+            _methodInfoCompileTimeWeaver = new MethodInfoCompileTimeWeaver(module)
+            {
+                IsEnabled = !DisableCompileTimeMethodInfos
+            };
+
             var assemblyMethodBoundaryAspects = module.Assembly.CustomAttributes;
 
             foreach (var type in module.Types.ToList())
                 WeaveTypeAndNestedTypes(module, type, assemblyMethodBoundaryAspects);
+
+            _methodInfoCompileTimeWeaver.Finish();
         }
 
         private void WeaveTypeAndNestedTypes(ModuleDefinition module, TypeDefinition type,