Skip to content

Commit 1f8488e

Browse files
committed
Remove special case MicrosoftCodeAnalysisVersionForPerfSensitiveAnalyzersTests
1 parent 6e2f18a commit 1f8488e

File tree

3 files changed

+8
-4
lines changed

3 files changed

+8
-4
lines changed

eng/Versions.props

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,6 @@
6565
<MicrosoftCodeAnalysisVersionForPublicApiAnalyzers>1.2.1</MicrosoftCodeAnalysisVersionForPublicApiAnalyzers>
6666
<MicrosoftCodeAnalysisVersionForBannedApiAnalyzers>3.11.0</MicrosoftCodeAnalysisVersionForBannedApiAnalyzers>
6767
<MicrosoftCodeAnalysisVersionForPerfSensitiveAnalyzers>3.11.0</MicrosoftCodeAnalysisVersionForPerfSensitiveAnalyzers>
68-
<MicrosoftCodeAnalysisVersionForPerfSensitiveAnalyzersTests>3.11.0</MicrosoftCodeAnalysisVersionForPerfSensitiveAnalyzersTests>
6968
<MicrosoftCodeAnalysisVersionForResxSourceGenerators>4.0.1</MicrosoftCodeAnalysisVersionForResxSourceGenerators>
7069
<MicrosoftCodeAnalysisVersionForNetAnalyzers>3.11.0</MicrosoftCodeAnalysisVersionForNetAnalyzers>
7170
<MicrosoftCodeAnalysisVersionForTextAnalyzers>3.11.0</MicrosoftCodeAnalysisVersionForTextAnalyzers>

src/PerformanceSensitiveAnalyzers/Core/ExplicitAllocationAnalyzer.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
using System.Collections.Immutable;
55
using Analyzer.Utilities;
66
using Analyzer.Utilities.Extensions;
7+
using Analyzer.Utilities.Lightup;
78
using Microsoft.CodeAnalysis.Diagnostics;
89
using Microsoft.CodeAnalysis.Operations;
910

@@ -89,6 +90,12 @@ protected override void AnalyzeNode(OperationAnalysisContext context, in Perform
8990

9091
if (context.Operation is IObjectCreationOperation or ITypeParameterObjectCreationOperation)
9192
{
93+
if (context.Operation.Parent?.Kind == OperationKindEx.Attribute)
94+
{
95+
// Don't report attribute usage as creating a new instance
96+
return;
97+
}
98+
9299
if (context.Operation.Type?.IsReferenceType == true)
93100
{
94101
context.ReportDiagnostic(context.Operation.CreateDiagnostic(ObjectCreationRule, EmptyMessageArgs));

src/PerformanceSensitiveAnalyzers/UnitTests/Microsoft.CodeAnalysis.PerformanceSensitiveAnalyzers.UnitTests.csproj

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,7 @@
99
<NoWarn>$(NoWarn);CA2007</NoWarn>
1010

1111
<ExcludeFromSourceBuild>true</ExcludeFromSourceBuild>
12-
<!-- These unit tests fail when run agains MicrosoftCodeAnalysisVersionForTests (4.6.0-final.1).
13-
https://github.com/dotnet/roslyn-analyzers/issues/6672. -->
14-
<MicrosoftCodeAnalysisVersion>$(MicrosoftCodeAnalysisVersionForPerfSensitiveAnalyzersTests)</MicrosoftCodeAnalysisVersion>
12+
<MicrosoftCodeAnalysisVersion>$(MicrosoftCodeAnalysisVersionForTests)</MicrosoftCodeAnalysisVersion>
1513
</PropertyGroup>
1614
<ItemGroup>
1715
<PackageReference Include="Microsoft.CodeAnalysis" Version="$(MicrosoftCodeAnalysisVersion)" />

0 commit comments

Comments
 (0)