Skip to content

Commit f0fec81

Browse files
Avoid static symbol cache for nullable context (#288)
1 parent 4dbb0b0 commit f0fec81

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

CheckedExceptions/CheckedExceptionsAnalyzer.Nullability.cs

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,10 @@ namespace Sundstrom.CheckedExceptions;
55

66
partial class CheckedExceptionsAnalyzer
77
{
8-
static INamedTypeSymbol? argumentNullExceptionTypeSymbol;
9-
108
private static IEnumerable<ExceptionInfo> ProcessNullable(
119
Compilation compilation, SemanticModel semanticModel, SyntaxNode node, IMethodSymbol methodSymbol, IEnumerable<ExceptionInfo> exceptionInfos)
1210
{
13-
if (argumentNullExceptionTypeSymbol is null)
14-
{
15-
argumentNullExceptionTypeSymbol = compilation.GetTypeByMetadataName("System.ArgumentNullException");
16-
}
11+
var argumentNullExceptionTypeSymbol = compilation.GetTypeByMetadataName("System.ArgumentNullException");
1712

1813
var isCompilationNullableEnabled = compilation.Options.NullableContextOptions is NullableContextOptions.Enable;
1914

@@ -59,10 +54,7 @@ private static IEnumerable<ExceptionInfo> ProcessNullable(
5954

6055
private static IEnumerable<INamedTypeSymbol> ProcessNullable(Compilation compilation, SemanticModel semanticModel, SyntaxNode node, IMethodSymbol methodSymbol, IEnumerable<INamedTypeSymbol> exceptions)
6156
{
62-
if (argumentNullExceptionTypeSymbol is null)
63-
{
64-
argumentNullExceptionTypeSymbol = compilation.GetTypeByMetadataName("System.ArgumentNullException");
65-
}
57+
var argumentNullExceptionTypeSymbol = compilation.GetTypeByMetadataName("System.ArgumentNullException");
6658

6759
var isCompilationNullableEnabled = compilation.Options.NullableContextOptions is NullableContextOptions.Enable;
6860

0 commit comments

Comments
 (0)