Use cached attributeInstances in TemplateParameterNameAttributeProvider. Relates #57

olsh · olsh · commit f85a4f5ed1b2 · 2022-02-04T19:12:21.000+03:00
diff --git a/src/ReSharper.Structured.Logging/Caching/TemplateParameterNameAttributeProvider.cs b/src/ReSharper.Structured.Logging/Caching/TemplateParameterNameAttributeProvider.cs
@@ -19,7 +19,7 @@ public TemplateParameterNameAttributeProvider(
 
         protected override string CalculateInfo(ITypeMember attributesOwner, IEnumerable<IAttributeInstance> attributeInstances)
         {
-            var templateFormatAttribute = attributesOwner.GetAttributeInstances(AttributesSource.All)
+            var templateFormatAttribute = attributeInstances
                 .FirstOrDefault(a => string.Equals(a.GetAttributeShortName(), "MessageTemplateFormatMethodAttribute", StringComparison.Ordinal));
 
             if (templateFormatAttribute != null)

Original file line number	Diff line number	Diff line change
`@@ -19,7 +19,7 @@ public TemplateParameterNameAttributeProvider(`
`19`	`19`
`20`	`20`	`protected override string CalculateInfo(ITypeMember attributesOwner, IEnumerable<IAttributeInstance> attributeInstances)`
`21`	`21`	`{`
`22`		`- var templateFormatAttribute = attributesOwner.GetAttributeInstances(AttributesSource.All)`
	`22`	`+ var templateFormatAttribute = attributeInstances`
`23`	`23`	`.FirstOrDefault(a => string.Equals(a.GetAttributeShortName(), "MessageTemplateFormatMethodAttribute", StringComparison.Ordinal));`
`24`	`24`
`25`	`25`	`if (templateFormatAttribute != null)`