Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ApiDesignGuidelines.BasicOverrideEqualsOnOverloadingOperatorEqualsAnalyzer throws NRE #70029

Closed
tmat opened this issue Sep 19, 2023 · 1 comment · Fixed by #70030
Closed

ApiDesignGuidelines.BasicOverrideEqualsOnOverloadingOperatorEqualsAnalyzer throws NRE #70029

tmat opened this issue Sep 19, 2023 · 1 comment · Fixed by #70030
Assignees
@mavasani
Labels
Area-Analyzers untriaged Issues and PRs which have not yet been triaged by a lead

Comments

@tmat
Copy link
Member

tmat commented Sep 19, 2023

While working in Roslyn.sln

Error (active)	AD0001	Analyzer 'Microsoft.CodeQuality.VisualBasic.Analyzers.ApiDesignGuidelines.BasicOverrideEqualsOnOverloadingOperatorEqualsAnalyzer' threw the following exception:
'Exception occurred with following context:
Compilation: Microsoft.CodeAnalysis.VisualBasic.Test.Utilities
ISymbol: VisualBasicSyntaxPrinter (NamedType)

System.NullReferenceException: Object reference not set to an instance of an object.
   at Microsoft.CodeQuality.VisualBasic.Analyzers.ApiDesignGuidelines.BasicOverrideEqualsOnOverloadingOperatorEqualsAnalyzer.HasEqualityOperator(INamedTypeSymbol type)
   at Microsoft.CodeQuality.VisualBasic.Analyzers.ApiDesignGuidelines.BasicOverrideEqualsOnOverloadingOperatorEqualsAnalyzer._Closure$__._Lambda$__7-0(SymbolAnalysisContext symbolContext)
   at Microsoft.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock[TArg](DiagnosticAnalyzer analyzer, Action`1 analyze, TArg argument, Nullable`1 info, CancellationToken cancellationToken)
-----
'.	Microsoft.CodeAnalysis.VisualBasic.Test.Utilities (net472), Microsoft.CodeAnalysis.VisualBasic.Test.Utilities (net7.0)	D:\R1\src\Compilers\Test\Utilities\VisualBasic\Microsoft.CodeAnalysis.VisualBasic.Test.Utilities.vbproj	1
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Analyzers untriaged Issues and PRs which have not yet been triaged by a lead labels Sep 19, 2023
@sharwell sharwell transferred this issue from dotnet/roslyn Sep 19, 2023
@mavasani
Copy link
Contributor

Duplicate of dotnet/roslyn-analyzers#6778. Roslyn repo needs to move to newer analyzer package. Moving back to Roslyn

@mavasani mavasani transferred this issue from dotnet/roslyn-analyzers Sep 20, 2023
@mavasani mavasani self-assigned this Sep 20, 2023
mavasani added a commit to mavasani/roslyn that referenced this issue Sep 20, 2023
@mavasani
Move to latest 8.0 preview version of MS.CA.NetAnalyzers package
9d9e931 
Closes dotnet#70029
@mavasani mavasani mentioned this issue Sep 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mavasani mavasani

Labels
Area-Analyzers untriaged Issues and PRs which have not yet been triaged by a lead
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Move to latest 8.0 preview version of MS.CA.NetAnalyzers package mavasani/roslyn
2 participants
@tmat @mavasani