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.
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
Remove unused diagnostic methods. #30539
Conversation
@@ -108,49 +108,8 @@ public bool IsDiagnosticSuppressed(Diagnostic diagnostic, out AttributeData supp | |||
return false; | |||
} | |||
|
|||
private bool IsDiagnosticSuppressed(Diagnostic diagnostic, out SuppressMessageInfo info, ISymbol symbolOpt = null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there are no callers of this method that pass in "symbolOpt". So symbolOpt is always null. So nothing ever called IsDiagnosticSuppressed(diagnostic.Id, symbolOpt, out info)
.
return parent != null && IsDiagnosticSuppressed(id, parent, out info); | ||
} | ||
private bool IsDiagnosticSuppressed(Diagnostic diagnostic, out SuppressMessageInfo info) | ||
=> IsDiagnosticSuppressed(diagnostic.Id, diagnostic.Location, out info); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is just the original method, with the optional symbol (that was never used) removed.
Tagging @jcouv |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM Thanks (iteration 1)
test windows_debug_spanish_unit32_prtest please |
Thanks! |
…rop-refactoring-naming-alignment * origin/master: (1888 commits) Remove unused diagnostic methods. (#30539) Added missing underscores in resource name (#30545) Move Spanish job to Azure DevOps Add when clause case to the TypeInferrer Add unit test Workaround for missing ZipDirectory task in msbuild 15.7 Build Roslyn_Preview.zip from a project Fixed typo ‘should descent’ Test loading of analyzer built against netstandard1.3 Add links to workitems/test plans (#30476) Use the IsAccessor extension for ISymbol Add unit test Update test. Simplify. Don't double massage actions. Only look at the nested actions, ensuring that this form doesn't change unexpectedly. Set getInnermostNodeForTie argument to true Fix for UseLocalFunction breaking semantics Rename Update name. ... # Conflicts: # src/Features/Core/Portable/InitializeParameter/AbstractInitializeMemberFromParameterCodeRefactoringProviderMemberCreation.cs
No description provided.