Merge pull request #171 from mono/shortcut-appdomain

Don't use AppDomain codepath for CurrentDomain
mono · Dec 6, 2023 · 6ec6c31 · 6ec6c31
2 parents d5fb8e6 + 145334b
commit 6ec6c31
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/Mono.TextTemplating/Mono.TextTemplating/CompiledTemplate.cs b/Mono.TextTemplating/Mono.TextTemplating/CompiledTemplate.cs
@@ -82,7 +82,10 @@ internal void SetTemplateContentForAppDomain (string content)
 		TemplateProcessor CreateTemplateProcessor ()
 		{
 			var domain = host.ProvideTemplatingAppDomain (templateContentForAppDomain);
-			if (domain == null) {
+
+			// hosts are supposed to return null of they don't want to use a domain
+			// but check for CurrentDomain too so we can optimize if they do that
+			if (domain == null || domain == AppDomain.CurrentDomain) {
 				return new TemplateProcessor ();
 			}