From cdcfce1b057cc55ef1b357555418f89f36d7e66f Mon Sep 17 00:00:00 2001 From: Cyrus Najmabadi Date: Thu, 9 Jan 2025 11:23:17 -0800 Subject: [PATCH] Remove pointless xml doc comment tags --- .../CSharpUseSystemThreadingLockDiagnosticAnalyzer.cs | 1 - .../CSharp/CodeFixes/ConvertToRecord/ConvertToRecordHelpers.cs | 1 - ...bstractRemoveUnusedParametersAndValuesDiagnosticAnalyzer.cs | 2 -- .../UseAutoProperty/AbstractUseAutoPropertyAnalyzer.cs | 1 - .../Core/CodeFixes/AddParameter/AddParameterService.cs | 1 - src/Dependencies/PooledObjects/PooledStringBuilder.cs | 1 - .../Core/InlineRename/IEditorInlineRenameService.cs | 1 - .../Core/Shared/Extensions/ITextBufferEditExtensions.cs | 1 - src/EditorFeatures/Core/Shared/Extensions/SpanExtensions.cs | 1 - src/EditorFeatures/Test2/Peek/PeekTests.vb | 1 - .../TestUtilities/Workspaces/EditorTestWorkspace.cs | 1 - .../ConvertLinq/CSharpConvertLinqQueryToForEachProvider.cs | 1 - .../ConvertLinq/ConvertForEachToLinqQuery/AbstractConverter.cs | 3 --- .../CSharp/Portable/Copilot/CSharpCopilotCodeFixProvider.cs | 2 -- .../Portable/ChangeSignature/IChangeSignatureOptionsService.cs | 1 - .../Core/Portable/Completion/CharacterSetModificationRule.cs | 2 -- src/Features/Core/Portable/Completion/CompletionItem.cs | 1 - src/Features/Core/Portable/Completion/CompletionItemRules.cs | 3 --- src/Features/Core/Portable/Completion/CompletionList.cs | 1 - src/Features/Core/Portable/Copilot/ICopilotOptionsService.cs | 1 - .../Portable/EmbeddedLanguages/StackFrame/StackFrameLexer.cs | 1 - .../Core/Portable/ExtractMethod/MethodExtractor.Analyzer.cs | 1 - src/Features/Core/Portable/ExtractMethod/SelectionResult.cs | 1 - .../Portable/GenerateFromMembers/GenerateFromMembersHelpers.cs | 1 - src/Features/Core/Portable/QuickInfo/IndentationHelper.cs | 1 - .../Contracts/IVisualDiagnosticsLanguageService.cs | 1 - .../IBrokeredServiceBridgeManifest.cs | 1 - .../Microsoft.CodeAnalysis.LanguageServer/DotnetCliHelper.cs | 1 - src/LanguageServer/Protocol/Extensions/Extensions.cs | 1 - src/Scripting/Core/ScriptOptions.cs | 1 - src/Test/Perf/Utilities/PerfTestBase.cs | 1 - src/Test/Perf/Utilities/RelativeDirectory.cs | 1 - src/Tools/BuildBoss/CompilerNuGetCheckerUtil.cs | 1 - src/VisualStudio/Core/Def/CodeMarkers/ManagedCodeMarkers.cs | 1 - .../Core/Def/ProjectSystem/VisualStudioWorkspaceImpl.cs | 1 - .../InProcess/AutomationElementExtensions.cs | 2 -- .../Impl/Venus/ContainedLanguageStaticEventBinding.vb | 1 - .../Xaml/Impl/Features/InlineRename/IXamlRenameInfo.cs | 1 - .../Portable/Rename/CSharpRenameRewriterLanguageService.cs | 1 - .../Portable/Simplification/Simplifiers/NameSimplifier.cs | 1 - src/Workspaces/Core/MSBuild/MSBuild/ProjectMap.cs | 1 - src/Workspaces/Core/Portable/Editing/SymbolEditor.cs | 2 -- src/Workspaces/Core/Portable/Editing/SyntaxGenerator.cs | 2 -- src/Workspaces/Core/Portable/FindSymbols/SymbolFinder.cs | 1 - .../Recommendations/AbstractRecommendationServiceRunner.cs | 1 - .../Core/Portable/Rename/IRenameRewriterLanguageService.cs | 2 -- .../Core/Portable/Serialization/SerializableSourceText.cs | 1 - .../Utilities/Documentation/XmlDocumentationProvider.cs | 1 - .../Core/Portable/Workspace/Host/HostWorkspaceServices.cs | 1 - src/Workspaces/Core/Portable/Workspace/Solution/AssetPath.cs | 2 -- .../Core/Portable/Workspace/Solution/TextAndVersion.cs | 2 -- .../Core/Portable/Workspace/Solution/TextDocumentStates.cs | 1 - .../Compiler/CSharp/Extensions/SyntaxTokenExtensions.cs | 1 - ...mbolUsageAnalysis.DataFlowAnalyzer.FlowGraphAnalysisData.cs | 1 - .../Compiler/Core/Utilities/AbstractSpeculationAnalyzer.cs | 1 - .../Compiler/Core/Utilities/SpecializedTasks.cs | 1 - .../Workspace/Core/Extensions/SyntaxGeneratorExtensions.cs | 1 - 57 files changed, 70 deletions(-) diff --git a/src/Analyzers/CSharp/Analyzers/UseSystemThreadingLock/CSharpUseSystemThreadingLockDiagnosticAnalyzer.cs b/src/Analyzers/CSharp/Analyzers/UseSystemThreadingLock/CSharpUseSystemThreadingLockDiagnosticAnalyzer.cs index b3190e2a98316..84b0b251f3ee1 100644 --- a/src/Analyzers/CSharp/Analyzers/UseSystemThreadingLock/CSharpUseSystemThreadingLockDiagnosticAnalyzer.cs +++ b/src/Analyzers/CSharp/Analyzers/UseSystemThreadingLock/CSharpUseSystemThreadingLockDiagnosticAnalyzer.cs @@ -46,7 +46,6 @@ internal sealed class CSharpUseSystemThreadingLockDiagnosticAnalyzer() /// /// A method body edit anywhere in a type will force us to reanalyze the whole type. /// - /// public override DiagnosticAnalyzerCategory GetAnalyzerCategory() => DiagnosticAnalyzerCategory.SemanticDocumentAnalysis; diff --git a/src/Analyzers/CSharp/CodeFixes/ConvertToRecord/ConvertToRecordHelpers.cs b/src/Analyzers/CSharp/CodeFixes/ConvertToRecord/ConvertToRecordHelpers.cs index 7778a8ec53176..a1c1e8c0839bd 100644 --- a/src/Analyzers/CSharp/CodeFixes/ConvertToRecord/ConvertToRecordHelpers.cs +++ b/src/Analyzers/CSharp/CodeFixes/ConvertToRecord/ConvertToRecordHelpers.cs @@ -489,7 +489,6 @@ private static bool IsSafeAssignment(IOperation operation) /// /// /// the symbol of the equals method - /// private static ImmutableArray GetEqualizedFields( IMethodBodyOperation operation, IMethodSymbol methodSymbol) diff --git a/src/Analyzers/Core/Analyzers/RemoveUnusedParametersAndValues/AbstractRemoveUnusedParametersAndValuesDiagnosticAnalyzer.cs b/src/Analyzers/Core/Analyzers/RemoveUnusedParametersAndValues/AbstractRemoveUnusedParametersAndValuesDiagnosticAnalyzer.cs index 42846f4bc11e2..39e86f986bfd6 100644 --- a/src/Analyzers/Core/Analyzers/RemoveUnusedParametersAndValues/AbstractRemoveUnusedParametersAndValuesDiagnosticAnalyzer.cs +++ b/src/Analyzers/Core/Analyzers/RemoveUnusedParametersAndValues/AbstractRemoveUnusedParametersAndValuesDiagnosticAnalyzer.cs @@ -128,7 +128,6 @@ protected virtual bool ShouldBailOutFromRemovableAssignmentAnalysis(IOperation u /// Indicates if the given expression statement operation has an explicit "Call" statement syntax indicating explicit discard. /// For example, VB "Call" statement. /// - /// protected abstract bool IsCallStatement(IExpressionStatementOperation expressionStatement); /// @@ -147,7 +146,6 @@ protected virtual bool ShouldBailOutFromRemovableAssignmentAnalysis(IOperation u /// and hence can be removed completely. For example, if the assigned value is a constant or a reference /// to a local/parameter, then it has no side effects, but if it is method invocation, it may have side effects. /// - /// private static PropertiesMap CreatePropertiesMap() { var builder = ImmutableDictionary.CreateBuilder<(UnusedValuePreference preference, bool isUnusedLocalAssignment, bool isRemovableAssignment), diff --git a/src/Analyzers/Core/Analyzers/UseAutoProperty/AbstractUseAutoPropertyAnalyzer.cs b/src/Analyzers/Core/Analyzers/UseAutoProperty/AbstractUseAutoPropertyAnalyzer.cs index 811855d577c41..50584ae2493dc 100644 --- a/src/Analyzers/Core/Analyzers/UseAutoProperty/AbstractUseAutoPropertyAnalyzer.cs +++ b/src/Analyzers/Core/Analyzers/UseAutoProperty/AbstractUseAutoPropertyAnalyzer.cs @@ -69,7 +69,6 @@ private static void ClearAndFree(ConcurrentDictionary /// A method body edit anywhere in a type will force us to reanalyze the whole type. /// - /// public override DiagnosticAnalyzerCategory GetAnalyzerCategory() => DiagnosticAnalyzerCategory.SemanticDocumentAnalysis; diff --git a/src/Analyzers/Core/CodeFixes/AddParameter/AddParameterService.cs b/src/Analyzers/Core/CodeFixes/AddParameter/AddParameterService.cs index 0288de4014e4c..a025fa1d41876 100644 --- a/src/Analyzers/Core/CodeFixes/AddParameter/AddParameterService.cs +++ b/src/Analyzers/Core/CodeFixes/AddParameter/AddParameterService.cs @@ -74,7 +74,6 @@ public static bool HasCascadingDeclarations(IMethodSymbol method) /// Adds a parameter to a method. /// /// to add as the final parameter - /// public static async Task AddParameterAsync( Document invocationDocument, IMethodSymbol method, diff --git a/src/Dependencies/PooledObjects/PooledStringBuilder.cs b/src/Dependencies/PooledObjects/PooledStringBuilder.cs index 6e1db4495d331..41ca484c2115d 100644 --- a/src/Dependencies/PooledObjects/PooledStringBuilder.cs +++ b/src/Dependencies/PooledObjects/PooledStringBuilder.cs @@ -77,7 +77,6 @@ public string ToStringAndFree(int startIndex, int length) /// If someone need to create a private pool /// /// The size of the pool. - /// public static ObjectPool CreatePool(int size = 32) { ObjectPool? pool = null; diff --git a/src/EditorFeatures/Core/InlineRename/IEditorInlineRenameService.cs b/src/EditorFeatures/Core/InlineRename/IEditorInlineRenameService.cs index 3a696307a8dcb..be4c7530fcfda 100644 --- a/src/EditorFeatures/Core/InlineRename/IEditorInlineRenameService.cs +++ b/src/EditorFeatures/Core/InlineRename/IEditorInlineRenameService.cs @@ -271,7 +271,6 @@ internal interface IEditorInlineRenameService : ILanguageService /// /// /// - /// Task>> GetRenameContextAsync( IInlineRenameInfo inlineRenameInfo, IInlineRenameLocationSet inlineRenameLocationSet, diff --git a/src/EditorFeatures/Core/Shared/Extensions/ITextBufferEditExtensions.cs b/src/EditorFeatures/Core/Shared/Extensions/ITextBufferEditExtensions.cs index b2d38dafb2390..790d35e11486e 100644 --- a/src/EditorFeatures/Core/Shared/Extensions/ITextBufferEditExtensions.cs +++ b/src/EditorFeatures/Core/Shared/Extensions/ITextBufferEditExtensions.cs @@ -17,7 +17,6 @@ internal static class ITextBufferEditExtensions /// Logs exceptions thrown during as we look for issues. /// /// - /// public static ITextSnapshot ApplyAndLogExceptions(this ITextBufferEdit edit) { try diff --git a/src/EditorFeatures/Core/Shared/Extensions/SpanExtensions.cs b/src/EditorFeatures/Core/Shared/Extensions/SpanExtensions.cs index d76293ba1933d..cf553a888e088 100644 --- a/src/EditorFeatures/Core/Shared/Extensions/SpanExtensions.cs +++ b/src/EditorFeatures/Core/Shared/Extensions/SpanExtensions.cs @@ -16,7 +16,6 @@ internal static class SpanExtensions /// Convert the editor Span instance to the corresponding TextSpan instance /// /// - /// public static TextSpan ToTextSpan(this Span span) => new(span.Start, span.Length); diff --git a/src/EditorFeatures/Test2/Peek/PeekTests.vb b/src/EditorFeatures/Test2/Peek/PeekTests.vb index 52b88f3db2613..fe764dd1a3e3b 100644 --- a/src/EditorFeatures/Test2/Peek/PeekTests.vb +++ b/src/EditorFeatures/Test2/Peek/PeekTests.vb @@ -463,7 +463,6 @@ public partial class D ''' Returns the text of the identifier line, starting at the identifier and ending at end of the line. ''' ''' - ''' Friend Function GetRemainingIdentifierLineTextOnDisk(index As Integer) As String Dim documentResult = DirectCast(Items(index), IDocumentPeekResult) Dim textBufferService = _workspace.GetService(Of ITextBufferFactoryService) diff --git a/src/EditorFeatures/TestUtilities/Workspaces/EditorTestWorkspace.cs b/src/EditorFeatures/TestUtilities/Workspaces/EditorTestWorkspace.cs index d9b54b3d39d54..e45633a941cff 100644 --- a/src/EditorFeatures/TestUtilities/Workspaces/EditorTestWorkspace.cs +++ b/src/EditorFeatures/TestUtilities/Workspaces/EditorTestWorkspace.cs @@ -243,7 +243,6 @@ public void OpenSourceGeneratedDocument(DocumentId documentId) /// preserved. The markup may also contain the caret indicator. /// The set of documents from which the projection buffer /// document will be composed. - /// public EditorTestHostDocument CreateProjectionBufferDocument( string markup, IList baseDocuments, diff --git a/src/Features/CSharp/Portable/ConvertLinq/CSharpConvertLinqQueryToForEachProvider.cs b/src/Features/CSharp/Portable/ConvertLinq/CSharpConvertLinqQueryToForEachProvider.cs index efdf299bba080..0babb22fbf6a6 100644 --- a/src/Features/CSharp/Portable/ConvertLinq/CSharpConvertLinqQueryToForEachProvider.cs +++ b/src/Features/CSharp/Portable/ConvertLinq/CSharpConvertLinqQueryToForEachProvider.cs @@ -243,7 +243,6 @@ private bool TryConvertInternal(QueryExpressionProcessingInfo queryExpressionPro /// Checks if the location of the query expression allows to convert it at least to a local function. /// It still does not guarantees that the conversion can be performed. There can be bail outs of later stages. /// - /// private bool CanTryConvertToLocalFunction() { SyntaxNode currentNode = _source; diff --git a/src/Features/CSharp/Portable/ConvertLinq/ConvertForEachToLinqQuery/AbstractConverter.cs b/src/Features/CSharp/Portable/ConvertLinq/ConvertForEachToLinqQuery/AbstractConverter.cs index f8582891b488c..f1326e36a4109 100644 --- a/src/Features/CSharp/Portable/ConvertLinq/ConvertForEachToLinqQuery/AbstractConverter.cs +++ b/src/Features/CSharp/Portable/ConvertLinq/ConvertForEachToLinqQuery/AbstractConverter.cs @@ -34,7 +34,6 @@ internal abstract class AbstractConverter(ForEachInfoextra leading tokens to be added to the select clause /// extra trailing tokens to be added to the select clause /// Flag indicating if a query expression should be generated - /// protected ExpressionSyntax CreateQueryExpressionOrLinqInvocation( ExpressionSyntax selectExpression, IEnumerable leadingTokensForSelect, @@ -52,7 +51,6 @@ protected ExpressionSyntax CreateQueryExpressionOrLinqInvocation( /// expression to be used into the last 'select ...' in the query expression /// extra leading tokens to be added to the select clause /// extra trailing tokens to be added to the select clause - /// private QueryExpressionSyntax CreateQueryExpression( ExpressionSyntax selectExpression, IEnumerable leadingTokensForSelect, @@ -121,7 +119,6 @@ private static FromClauseSyntax CreateFromClause( /// expression to be used in the last 'Select' invocation /// extra leading tokens to be added to the select clause /// extra trailing tokens to be added to the select clause - /// private ExpressionSyntax CreateLinqInvocationOrSimpleExpression( ExpressionSyntax selectExpression, IEnumerable leadingTokensForSelect, diff --git a/src/Features/CSharp/Portable/Copilot/CSharpCopilotCodeFixProvider.cs b/src/Features/CSharp/Portable/Copilot/CSharpCopilotCodeFixProvider.cs index f885a57f9cfe4..f02096c8bd6bb 100644 --- a/src/Features/CSharp/Portable/Copilot/CSharpCopilotCodeFixProvider.cs +++ b/src/Features/CSharp/Portable/Copilot/CSharpCopilotCodeFixProvider.cs @@ -40,13 +40,11 @@ internal sealed partial class CSharpCopilotCodeFixProvider() : CodeFixProvider /// /// Ensure that fixes for Copilot diagnostics are always towards the bottom of the lightbulb. /// - /// protected sealed override CodeActionRequestPriority ComputeRequestPriority() => CodeActionRequestPriority.Low; /// /// We do not support a FixAll operation for Copilot suggestions. /// - /// public sealed override FixAllProvider? GetFixAllProvider() => null; public sealed override ImmutableArray FixableDiagnosticIds => [CopilotDiagnosticId]; diff --git a/src/Features/Core/Portable/ChangeSignature/IChangeSignatureOptionsService.cs b/src/Features/Core/Portable/ChangeSignature/IChangeSignatureOptionsService.cs index 696e3ee5537c4..7013850b6c8ee 100644 --- a/src/Features/Core/Portable/ChangeSignature/IChangeSignatureOptionsService.cs +++ b/src/Features/Core/Portable/ChangeSignature/IChangeSignatureOptionsService.cs @@ -17,7 +17,6 @@ internal interface IChangeSignatureOptionsService : IWorkspaceService /// parameters) /// the symbol for changing the signature /// existing parameters of the symbol - /// ChangeSignatureOptionsResult? GetChangeSignatureOptions( SemanticDocument document, int positionForTypeBinding, diff --git a/src/Features/Core/Portable/Completion/CharacterSetModificationRule.cs b/src/Features/Core/Portable/Completion/CharacterSetModificationRule.cs index b05169033da52..f0a5da70aaa79 100644 --- a/src/Features/Core/Portable/Completion/CharacterSetModificationRule.cs +++ b/src/Features/Core/Portable/Completion/CharacterSetModificationRule.cs @@ -32,7 +32,6 @@ private CharacterSetModificationRule(CharacterSetModificationKind kind, Immutabl /// /// The kind of rule. /// One or more characters. These are typically punctuation characters. - /// public static CharacterSetModificationRule Create(CharacterSetModificationKind kind, ImmutableArray characters) => new(kind, characters); @@ -41,7 +40,6 @@ public static CharacterSetModificationRule Create(CharacterSetModificationKind k /// /// The kind of rule. /// One or more characters. These are typically punctuation characters. - /// public static CharacterSetModificationRule Create(CharacterSetModificationKind kind, params char[] characters) => new(kind, [.. characters]); } diff --git a/src/Features/Core/Portable/Completion/CompletionItem.cs b/src/Features/Core/Portable/Completion/CompletionItem.cs index 2faaf0bf476ad..4b46cd8aeeeea 100644 --- a/src/Features/Core/Portable/Completion/CompletionItem.cs +++ b/src/Features/Core/Portable/Completion/CompletionItem.cs @@ -311,7 +311,6 @@ internal static CompletionItem CreateInternal( /// Additional information. /// Descriptive tags that may influence how the item is displayed. /// The rules that declare how this item should behave. - /// [Obsolete("Use the Create overload that does not take a span", error: true)] [EditorBrowsable(EditorBrowsableState.Never)] public static CompletionItem Create( diff --git a/src/Features/Core/Portable/Completion/CompletionItemRules.cs b/src/Features/Core/Portable/Completion/CompletionItemRules.cs index 4bbe80dec65e2..43dea14382879 100644 --- a/src/Features/Core/Portable/Completion/CompletionItemRules.cs +++ b/src/Features/Core/Portable/Completion/CompletionItemRules.cs @@ -100,7 +100,6 @@ private CompletionItemRules( /// Rule about whether the enter key is passed through to the editor after the selected item has been committed. /// True if the modified text should be formatted automatically. /// True if the related completion item should be initially selected. - /// public static CompletionItemRules Create( ImmutableArray filterCharacterRules, ImmutableArray commitCharacterRules, @@ -123,7 +122,6 @@ public static CompletionItemRules Create( /// True if the modified text should be formatted automatically. /// True if the related completion item should be initially selected. /// How this item should be selected if no text has been typed after the completion list is brought up. - /// public static CompletionItemRules Create( ImmutableArray filterCharacterRules = default, ImmutableArray commitCharacterRules = default, @@ -157,7 +155,6 @@ public static CompletionItemRules Create( /// Rule about whether the enter key is passed through to the editor after the selected item has been committed. /// True if the modified text should be formatted automatically. /// True if the related completion item should be initially selected. - /// internal static CompletionItemRules Create( ImmutableArray filterCharacterRules, ImmutableArray commitCharacterRules, diff --git a/src/Features/Core/Portable/Completion/CompletionList.cs b/src/Features/Core/Portable/Completion/CompletionList.cs index d1f98f7c7c3e5..25e468a4cae63 100644 --- a/src/Features/Core/Portable/Completion/CompletionList.cs +++ b/src/Features/Core/Portable/Completion/CompletionList.cs @@ -99,7 +99,6 @@ private CompletionList( /// The completion items to present to the user. /// The rules used to control behavior of the completion list shown to the user during typing. /// An optional that appears selected in the list presented to the user during suggestion mode. - /// public static CompletionList Create( TextSpan defaultSpan, ImmutableArray items, diff --git a/src/Features/Core/Portable/Copilot/ICopilotOptionsService.cs b/src/Features/Core/Portable/Copilot/ICopilotOptionsService.cs index 62752f0adc64f..234c3ccbc748c 100644 --- a/src/Features/Core/Portable/Copilot/ICopilotOptionsService.cs +++ b/src/Features/Core/Portable/Copilot/ICopilotOptionsService.cs @@ -27,6 +27,5 @@ internal interface ICopilotOptionsService : ILanguageService /// /// Returns true if Copilot on-the-fly docs feature is enabled. /// - /// Task IsOnTheFlyDocsOptionEnabledAsync(); } diff --git a/src/Features/Core/Portable/EmbeddedLanguages/StackFrame/StackFrameLexer.cs b/src/Features/Core/Portable/EmbeddedLanguages/StackFrame/StackFrameLexer.cs index 3577bf1060937..eea827a263e87 100644 --- a/src/Features/Core/Portable/EmbeddedLanguages/StackFrame/StackFrameLexer.cs +++ b/src/Features/Core/Portable/EmbeddedLanguages/StackFrame/StackFrameLexer.cs @@ -286,7 +286,6 @@ public Result TryScanPath() /// Returns a number token with the and remaining /// attached to it. /// - /// public StackFrameToken? TryScanRequiredLineNumber() { var lineTrivia = TryScanLineTrivia(); diff --git a/src/Features/Core/Portable/ExtractMethod/MethodExtractor.Analyzer.cs b/src/Features/Core/Portable/ExtractMethod/MethodExtractor.Analyzer.cs index 1b72ce1b06d49..101a8008e611a 100644 --- a/src/Features/Core/Portable/ExtractMethod/MethodExtractor.Analyzer.cs +++ b/src/Features/Core/Portable/ExtractMethod/MethodExtractor.Analyzer.cs @@ -65,7 +65,6 @@ protected virtual bool IsReadOutside(ISymbol symbol, HashSet readOutsid /// /// check whether the selection is at the placed where read-only field is allowed to be extracted out /// - /// protected abstract bool ReadOnlyFieldAllowed(); public AnalyzerResult Analyze() diff --git a/src/Features/Core/Portable/ExtractMethod/SelectionResult.cs b/src/Features/Core/Portable/ExtractMethod/SelectionResult.cs index 5b54d405c9669..45a1781597991 100644 --- a/src/Features/Core/Portable/ExtractMethod/SelectionResult.cs +++ b/src/Features/Core/Portable/ExtractMethod/SelectionResult.cs @@ -124,7 +124,6 @@ public TExecutableStatementSyntax GetLastStatement() /// name="predicate"/>. Will not descend into local functions or lambdas. /// /// - /// private bool CheckNodesInSelection(Func predicate) { var firstToken = this.GetFirstTokenInSelection(); diff --git a/src/Features/Core/Portable/GenerateFromMembers/GenerateFromMembersHelpers.cs b/src/Features/Core/Portable/GenerateFromMembers/GenerateFromMembersHelpers.cs index 0c2e4449ad2f6..2b3d574956fac 100644 --- a/src/Features/Core/Portable/GenerateFromMembers/GenerateFromMembersHelpers.cs +++ b/src/Features/Core/Portable/GenerateFromMembers/GenerateFromMembersHelpers.cs @@ -81,7 +81,6 @@ private static bool IsViableProperty(IPropertySymbol property) /// /// /// - /// public static ImmutableArray DetermineParameters( ImmutableArray selectedMembers, ImmutableArray rules) { diff --git a/src/Features/Core/Portable/QuickInfo/IndentationHelper.cs b/src/Features/Core/Portable/QuickInfo/IndentationHelper.cs index eecfabd1f574d..c3ae1d1aa73e7 100644 --- a/src/Features/Core/Portable/QuickInfo/IndentationHelper.cs +++ b/src/Features/Core/Portable/QuickInfo/IndentationHelper.cs @@ -23,7 +23,6 @@ internal static class IndentationHelper /// /// The initial set of spans to align. /// The number of spaces to - /// public static ImmutableArray GetSpansWithAlignedIndentation( SourceText text, ImmutableArray classifiedSpans, diff --git a/src/LanguageServer/ExternalAccess/VisualDiagnostics/Contracts/IVisualDiagnosticsLanguageService.cs b/src/LanguageServer/ExternalAccess/VisualDiagnostics/Contracts/IVisualDiagnosticsLanguageService.cs index fa0bfc3b929b4..405a7d91bd7ba 100644 --- a/src/LanguageServer/ExternalAccess/VisualDiagnostics/Contracts/IVisualDiagnosticsLanguageService.cs +++ b/src/LanguageServer/ExternalAccess/VisualDiagnostics/Contracts/IVisualDiagnosticsLanguageService.cs @@ -20,7 +20,6 @@ internal interface IVisualDiagnosticsLanguageService : IWorkspaceService, IDispo /// /// Service broker /// Cancellation token - /// Task InitializeAsync(IServiceBroker serviceBroker, CancellationToken token); } } diff --git a/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/BrokeredServices/Services/BrokeredServiceBridgeManifest/IBrokeredServiceBridgeManifest.cs b/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/BrokeredServices/Services/BrokeredServiceBridgeManifest/IBrokeredServiceBridgeManifest.cs index 8b4df206e01d9..bc546acff94dd 100644 --- a/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/BrokeredServices/Services/BrokeredServiceBridgeManifest/IBrokeredServiceBridgeManifest.cs +++ b/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/BrokeredServices/Services/BrokeredServiceBridgeManifest/IBrokeredServiceBridgeManifest.cs @@ -19,7 +19,6 @@ internal interface IBrokeredServiceBridgeManifest /// Returns services that the container wishes to expose across the bridge. /// /// - /// ValueTask> GetAvailableServicesAsync(CancellationToken cancellationToken); } diff --git a/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/DotnetCliHelper.cs b/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/DotnetCliHelper.cs index 028ab77f67b59..c2d0c09b8c067 100644 --- a/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/DotnetCliHelper.cs +++ b/src/LanguageServer/Microsoft.CodeAnalysis.LanguageServer/DotnetCliHelper.cs @@ -122,7 +122,6 @@ public async Task GetVsTestConsolePathAsync(string projectOutputDirector /// Based on https://github.com/dotnet/msbuild/blob/main/src/Utilities/ToolTask.cs#L1259 /// We also do not include DOTNET_ROOT here, see https://github.com/dotnet/runtime/issues/88754 /// - /// internal string GetDotNetPathOrDefault() { var (fileName, sep) = PlatformInformation.IsWindows diff --git a/src/LanguageServer/Protocol/Extensions/Extensions.cs b/src/LanguageServer/Protocol/Extensions/Extensions.cs index adc965d671f6f..8b49f8c0e80fb 100644 --- a/src/LanguageServer/Protocol/Extensions/Extensions.cs +++ b/src/LanguageServer/Protocol/Extensions/Extensions.cs @@ -325,7 +325,6 @@ public static ContainerElement ToLSPElement(this QuickInfoContainerElement eleme /// in the Visual Studio client. /// /// - /// /// public static (Guid guid, int id) GetVsImageData(this Glyph glyph) { diff --git a/src/Scripting/Core/ScriptOptions.cs b/src/Scripting/Core/ScriptOptions.cs index d39405b24ba86..79a571d06077d 100644 --- a/src/Scripting/Core/ScriptOptions.cs +++ b/src/Scripting/Core/ScriptOptions.cs @@ -366,7 +366,6 @@ public ScriptOptions WithFileEncoding(Encoding encoding) /// /// Create a new with the specified . /// - /// public ScriptOptions WithOptimizationLevel(OptimizationLevel optimizationLevel) => optimizationLevel == OptimizationLevel ? this : new ScriptOptions(this) { OptimizationLevel = optimizationLevel }; diff --git a/src/Test/Perf/Utilities/PerfTestBase.cs b/src/Test/Perf/Utilities/PerfTestBase.cs index b8f6ac36d46cc..51daaa8a58666 100644 --- a/src/Test/Perf/Utilities/PerfTestBase.cs +++ b/src/Test/Perf/Utilities/PerfTestBase.cs @@ -56,7 +56,6 @@ public PerfTest([CallerFilePath] string workingFile = "") : base(workingFile) { /// /// A list of scenarios. /// - /// public abstract string[] GetScenarios(); public virtual ITraceManager GetTraceManager() diff --git a/src/Test/Perf/Utilities/RelativeDirectory.cs b/src/Test/Perf/Utilities/RelativeDirectory.cs index 56eb456587728..21fc198116138 100644 --- a/src/Test/Perf/Utilities/RelativeDirectory.cs +++ b/src/Test/Perf/Utilities/RelativeDirectory.cs @@ -68,7 +68,6 @@ public string TempDirectory /// Returns the directory that contains built roslyn binaries. Usually this will be /// Binaries/Debug or Binaries/Release. /// - /// public string MyBinaries() { ThrowIfNotSetup(); diff --git a/src/Tools/BuildBoss/CompilerNuGetCheckerUtil.cs b/src/Tools/BuildBoss/CompilerNuGetCheckerUtil.cs index f2d6bf97a59b2..684318a2ba787 100644 --- a/src/Tools/BuildBoss/CompilerNuGetCheckerUtil.cs +++ b/src/Tools/BuildBoss/CompilerNuGetCheckerUtil.cs @@ -352,7 +352,6 @@ private static IEnumerable GetPartsInFolder(Package package, string /// properly update this package. /// /// - /// private bool CheckExternalApis(TextWriter textWriter) { var packageFilePath = FindNuGetPackage(Path.Combine(ArtifactsDirectory, "VSSetup", Configuration, "DevDivPackages"), "VS.ExternalAPIs.Roslyn"); diff --git a/src/VisualStudio/Core/Def/CodeMarkers/ManagedCodeMarkers.cs b/src/VisualStudio/Core/Def/CodeMarkers/ManagedCodeMarkers.cs index abecfd1ee7a5a..c9c6e24e1d617 100644 --- a/src/VisualStudio/Core/Def/CodeMarkers/ManagedCodeMarkers.cs +++ b/src/VisualStudio/Core/Def/CodeMarkers/ManagedCodeMarkers.cs @@ -303,7 +303,6 @@ public bool CodeMarkerEx(int nTimerID, string stringData) /// Converts a string into a byte buffer including a zero terminator (needed for proper ETW message formatting) /// /// String to be converted to bytes - /// internal static byte[] StringToBytesZeroTerminated(string stringData) { var encoding = System.Text.Encoding.Unicode; diff --git a/src/VisualStudio/Core/Def/ProjectSystem/VisualStudioWorkspaceImpl.cs b/src/VisualStudio/Core/Def/ProjectSystem/VisualStudioWorkspaceImpl.cs index 7ed72d7b19417..fe498f5cf0ee2 100644 --- a/src/VisualStudio/Core/Def/ProjectSystem/VisualStudioWorkspaceImpl.cs +++ b/src/VisualStudio/Core/Def/ProjectSystem/VisualStudioWorkspaceImpl.cs @@ -1114,7 +1114,6 @@ internal void OpenDocumentFromPath(string? filePath, ProjectId projectId, bool a /// the file path of the file to open. /// used to retrieve the IVsHierarchy to ensure the file is opened in a matching context. /// the window frame. - /// private bool TryGetFrame(string? filePath, ProjectId projectId, [NotNullWhen(returnValue: true)] out IVsWindowFrame? frame) { frame = null; diff --git a/src/VisualStudio/IntegrationTest/New.IntegrationTests/InProcess/AutomationElementExtensions.cs b/src/VisualStudio/IntegrationTest/New.IntegrationTests/InProcess/AutomationElementExtensions.cs index 69246b8e0544b..496fb1647b59b 100644 --- a/src/VisualStudio/IntegrationTest/New.IntegrationTests/InProcess/AutomationElementExtensions.cs +++ b/src/VisualStudio/IntegrationTest/New.IntegrationTests/InProcess/AutomationElementExtensions.cs @@ -109,7 +109,6 @@ public static IUIAutomationElement FindDescendantByClass(this IUIAutomationEleme /// Given an , returns all descendants with the given . /// If none are found, the resulting collection will be empty. /// - /// public static IUIAutomationElementArray FindDescendantsByClass(this IUIAutomationElement parent, string className) { if (parent == null) @@ -310,7 +309,6 @@ private static void ThrowUnableToFindChildException(string path, IUIAutomationEl /// support the . /// /// - /// public static bool IsToggledOn(this IUIAutomationElement element) { var togglePattern = element.GetCurrentPattern(UIA_PatternIds.UIA_TogglePatternId); diff --git a/src/VisualStudio/VisualBasic/Impl/Venus/ContainedLanguageStaticEventBinding.vb b/src/VisualStudio/VisualBasic/Impl/Venus/ContainedLanguageStaticEventBinding.vb index 3f7840928a733..94bcc88675708 100644 --- a/src/VisualStudio/VisualBasic/Impl/Venus/ContainedLanguageStaticEventBinding.vb +++ b/src/VisualStudio/VisualBasic/Impl/Venus/ContainedLanguageStaticEventBinding.vb @@ -19,7 +19,6 @@ Namespace Microsoft.VisualStudio.LanguageServices.VisualBasic.Venus ''' ''' Find all the methods that handle events (though "Handles" clauses). ''' - ''' Public Function GetStaticEventBindings(document As Document, className As String, objectName As String, diff --git a/src/VisualStudio/Xaml/Impl/Features/InlineRename/IXamlRenameInfo.cs b/src/VisualStudio/Xaml/Impl/Features/InlineRename/IXamlRenameInfo.cs index f3bdde0e821cb..cd1437186f716 100644 --- a/src/VisualStudio/Xaml/Impl/Features/InlineRename/IXamlRenameInfo.cs +++ b/src/VisualStudio/Xaml/Impl/Features/InlineRename/IXamlRenameInfo.cs @@ -56,7 +56,6 @@ internal interface IXamlRenameInfo /// /// Checks if the new replacement text is valid for this rename operation. /// - /// bool IsReplacementTextValid(string replacementText); } } diff --git a/src/Workspaces/CSharp/Portable/Rename/CSharpRenameRewriterLanguageService.cs b/src/Workspaces/CSharp/Portable/Rename/CSharpRenameRewriterLanguageService.cs index bd829b0af7481..3dce526ea273b 100644 --- a/src/Workspaces/CSharp/Portable/Rename/CSharpRenameRewriterLanguageService.cs +++ b/src/Workspaces/CSharp/Portable/Rename/CSharpRenameRewriterLanguageService.cs @@ -1142,7 +1142,6 @@ public override void TryAddPossibleNameConflicts(ISymbol symbol, string replacem /// statement of this lambda. /// /// The token to get the complexification target for. - /// public override SyntaxNode? GetExpansionTargetForLocation(SyntaxToken token) => GetExpansionTarget(token); diff --git a/src/Workspaces/CSharp/Portable/Simplification/Simplifiers/NameSimplifier.cs b/src/Workspaces/CSharp/Portable/Simplification/Simplifiers/NameSimplifier.cs index 38174d848956b..0f51cb4f068c5 100644 --- a/src/Workspaces/CSharp/Portable/Simplification/Simplifiers/NameSimplifier.cs +++ b/src/Workspaces/CSharp/Portable/Simplification/Simplifiers/NameSimplifier.cs @@ -548,7 +548,6 @@ private static bool TryReduceAttributeSuffix( /// property. /// /// - /// private static bool IsPartOfNamespaceDeclarationName(SyntaxNode node) { var parent = node; diff --git a/src/Workspaces/Core/MSBuild/MSBuild/ProjectMap.cs b/src/Workspaces/Core/MSBuild/MSBuild/ProjectMap.cs index 74d80f1ea8d2a..f36db3ac66f0c 100644 --- a/src/Workspaces/Core/MSBuild/MSBuild/ProjectMap.cs +++ b/src/Workspaces/Core/MSBuild/MSBuild/ProjectMap.cs @@ -56,7 +56,6 @@ private ProjectMap() /// Create a populated with the given . /// /// The to populate the new with. - /// public static ProjectMap Create(Solution solution) { var projectMap = new ProjectMap(); diff --git a/src/Workspaces/Core/Portable/Editing/SymbolEditor.cs b/src/Workspaces/Core/Portable/Editing/SymbolEditor.cs index 8a54d42c8e009..3fc98f00b8f0b 100644 --- a/src/Workspaces/Core/Portable/Editing/SymbolEditor.cs +++ b/src/Workspaces/Core/Portable/Editing/SymbolEditor.cs @@ -211,7 +211,6 @@ private bool TryGetBestDeclarationForSingleEdit(ISymbol symbol, out SyntaxNode d /// /// The to apply edits to. /// The declaration to edit. - /// public delegate void DeclarationEditAction(DocumentEditor editor, SyntaxNode declaration); /// @@ -220,7 +219,6 @@ private bool TryGetBestDeclarationForSingleEdit(ISymbol symbol, out SyntaxNode d /// The to apply edits to. /// The declaration to edit. /// A cancellation token. - /// public delegate Task AsyncDeclarationEditAction(DocumentEditor editor, SyntaxNode declaration, CancellationToken cancellationToken); /// diff --git a/src/Workspaces/Core/Portable/Editing/SyntaxGenerator.cs b/src/Workspaces/Core/Portable/Editing/SyntaxGenerator.cs index a6c528cdbcc8a..7f1f3d09ca93d 100644 --- a/src/Workspaces/Core/Portable/Editing/SyntaxGenerator.cs +++ b/src/Workspaces/Core/Portable/Editing/SyntaxGenerator.cs @@ -1830,7 +1830,6 @@ public SyntaxNode NullLiteralExpression() /// Creates an expression that denotes a simple identifier name. /// /// - /// public SyntaxNode IdentifierName(string identifier) => this.SyntaxGeneratorInternal.IdentifierName(identifier); @@ -1924,7 +1923,6 @@ public SyntaxNode DottedName(string dottedName) /// Creates a name that denotes a type or namespace. /// /// The symbol to create a name for. - /// public abstract SyntaxNode NameExpression(INamespaceOrTypeSymbol namespaceOrTypeSymbol); /// diff --git a/src/Workspaces/Core/Portable/FindSymbols/SymbolFinder.cs b/src/Workspaces/Core/Portable/FindSymbols/SymbolFinder.cs index f30ad6bfdcc0e..7bff87642a8b2 100644 --- a/src/Workspaces/Core/Portable/FindSymbols/SymbolFinder.cs +++ b/src/Workspaces/Core/Portable/FindSymbols/SymbolFinder.cs @@ -154,7 +154,6 @@ public static async Task FindSymbolAtPositionAsync( /// The symbol to find corresponding matches for. /// A compilation to find the corresponding symbol within. The compilation may or may not be the origin of the symbol. /// A CancellationToken. - /// public static IEnumerable FindSimilarSymbols(TSymbol symbol, Compilation compilation, CancellationToken cancellationToken = default) where TSymbol : ISymbol { diff --git a/src/Workspaces/Core/Portable/Recommendations/AbstractRecommendationServiceRunner.cs b/src/Workspaces/Core/Portable/Recommendations/AbstractRecommendationServiceRunner.cs index 46e89766482bf..4e4f78b91c711 100644 --- a/src/Workspaces/Core/Portable/Recommendations/AbstractRecommendationServiceRunner.cs +++ b/src/Workspaces/Core/Portable/Recommendations/AbstractRecommendationServiceRunner.cs @@ -181,7 +181,6 @@ private ImmutableArray SubstituteTypeParameters(ImmutableArray /// ordinal of the arguments of function: (a,b) or (a,b,c) in the example above /// ordinal of the lambda parameters, e.g. a, b or c. - /// private ImmutableArray GetTypeSymbols( ImmutableArray candidateSymbols, string argumentName, diff --git a/src/Workspaces/Core/Portable/Rename/IRenameRewriterLanguageService.cs b/src/Workspaces/Core/Portable/Rename/IRenameRewriterLanguageService.cs index 519f49b5d141b..55df270cbc0d2 100644 --- a/src/Workspaces/Core/Portable/Rename/IRenameRewriterLanguageService.cs +++ b/src/Workspaces/Core/Portable/Rename/IRenameRewriterLanguageService.cs @@ -105,7 +105,6 @@ bool LocalVariableConflict( /// /// /// - /// bool IsIdentifierValid( string replacementText, ISyntaxFactsService syntaxFactsService); @@ -116,7 +115,6 @@ bool IsIdentifierValid( /// statement of this lambda. /// /// The token to get the complexification target for. - /// SyntaxNode? GetExpansionTargetForLocation(SyntaxToken token); } diff --git a/src/Workspaces/Core/Portable/Serialization/SerializableSourceText.cs b/src/Workspaces/Core/Portable/Serialization/SerializableSourceText.cs index b3f854ae1a892..2a46a92588341 100644 --- a/src/Workspaces/Core/Portable/Serialization/SerializableSourceText.cs +++ b/src/Workspaces/Core/Portable/Serialization/SerializableSourceText.cs @@ -89,7 +89,6 @@ private SerializableSourceText(TemporaryStorageTextHandle? storageHandle, Source /// Returns the strongly referenced SourceText if we have it, or tries to retrieve it from the weak reference if /// it's still being held there. /// - /// private SourceText? TryGetText() => _text ?? _computedText.GetTarget(); diff --git a/src/Workspaces/Core/Portable/Utilities/Documentation/XmlDocumentationProvider.cs b/src/Workspaces/Core/Portable/Utilities/Documentation/XmlDocumentationProvider.cs index f0d7b0f914862..ff7c8b58ebc16 100644 --- a/src/Workspaces/Core/Portable/Utilities/Documentation/XmlDocumentationProvider.cs +++ b/src/Workspaces/Core/Portable/Utilities/Documentation/XmlDocumentationProvider.cs @@ -28,7 +28,6 @@ public abstract class XmlDocumentationProvider : DocumentationProvider /// Gets the source stream for the XML document. /// /// The cancellation token. - /// protected abstract Stream GetSourceStream(CancellationToken cancellationToken); /// diff --git a/src/Workspaces/Core/Portable/Workspace/Host/HostWorkspaceServices.cs b/src/Workspaces/Core/Portable/Workspace/Host/HostWorkspaceServices.cs index 979c1a96abed4..86669133f241b 100644 --- a/src/Workspaces/Core/Portable/Workspace/Host/HostWorkspaceServices.cs +++ b/src/Workspaces/Core/Portable/Workspace/Host/HostWorkspaceServices.cs @@ -47,7 +47,6 @@ public abstract class HostWorkspaceServices : IDisposable /// /// The host services this workspace services originated from. /// - /// public abstract HostServices HostServices { get; } /// diff --git a/src/Workspaces/Core/Portable/Workspace/Solution/AssetPath.cs b/src/Workspaces/Core/Portable/Workspace/Solution/AssetPath.cs index 541b4b7a7278a..9da410e3548ac 100644 --- a/src/Workspaces/Core/Portable/Workspace/Solution/AssetPath.cs +++ b/src/Workspaces/Core/Portable/Workspace/Solution/AssetPath.cs @@ -95,7 +95,6 @@ public AssetPath(AssetPathKind kind, DocumentId? documentId) /// Searches the requested project, and all documents underneath it. Used only in tests. /// /// - /// public static AssetPath SolutionAndProjectForTesting(ProjectId projectId) => new(AssetPathKind.SolutionCompilationState | AssetPathKind.SolutionState | AssetPathKind.Projects, projectId); @@ -103,7 +102,6 @@ public static AssetPath SolutionAndProjectForTesting(ProjectId projectId) /// Searches all documents within the specified project. /// /// - /// public static AssetPath DocumentsInProject(ProjectId projectId) => new(AssetPathKind.Documents, projectId); } diff --git a/src/Workspaces/Core/Portable/Workspace/Solution/TextAndVersion.cs b/src/Workspaces/Core/Portable/Workspace/Solution/TextAndVersion.cs index ed0b6b1c4114e..a284eda91cffd 100644 --- a/src/Workspaces/Core/Portable/Workspace/Solution/TextAndVersion.cs +++ b/src/Workspaces/Core/Portable/Workspace/Solution/TextAndVersion.cs @@ -50,7 +50,6 @@ private TextAndVersion(SourceText text, VersionStamp version, string? filePath, /// The text /// The version /// Obsolete. - /// public static TextAndVersion Create(SourceText text, VersionStamp version, string? filePath = null) => new(text ?? throw new ArgumentNullException(nameof(text)), version, filePath, loadDiagnostic: null); @@ -60,7 +59,6 @@ public static TextAndVersion Create(SourceText text, VersionStamp version, strin /// The text /// The version /// Diagnostic describing failure to load the source text. - /// internal static TextAndVersion Create(SourceText text, VersionStamp version, Diagnostic? loadDiagnostic) => new(text, version, filePath: null, loadDiagnostic); } diff --git a/src/Workspaces/Core/Portable/Workspace/Solution/TextDocumentStates.cs b/src/Workspaces/Core/Portable/Workspace/Solution/TextDocumentStates.cs index 45b1e2c254d8d..7d3a3c7c6f9b4 100644 --- a/src/Workspaces/Core/Portable/Workspace/Solution/TextDocumentStates.cs +++ b/src/Workspaces/Core/Portable/Workspace/Solution/TextDocumentStates.cs @@ -117,7 +117,6 @@ public TState GetRequiredState(DocumentId documentId) /// /// Get states ordered in compilation order. /// - /// public ImmutableArray GetStatesInCompilationOrder() { if (_statesInCompilationOrder.IsDefault) diff --git a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/CSharp/Extensions/SyntaxTokenExtensions.cs b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/CSharp/Extensions/SyntaxTokenExtensions.cs index c9435d5cf131e..56096d6d270ac 100644 --- a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/CSharp/Extensions/SyntaxTokenExtensions.cs +++ b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/CSharp/Extensions/SyntaxTokenExtensions.cs @@ -54,7 +54,6 @@ public static bool IsOpenBraceOfAccessorList(this SyntaxToken token) /// actual keywords, contextual keywords, and even 'var' and 'dynamic' /// /// - /// public static bool CouldBeKeyword(this SyntaxToken token) { if (token.IsKeyword()) diff --git a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/FlowAnalysis/SymbolUsageAnalysis/SymbolUsageAnalysis.DataFlowAnalyzer.FlowGraphAnalysisData.cs b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/FlowAnalysis/SymbolUsageAnalysis/SymbolUsageAnalysis.DataFlowAnalyzer.FlowGraphAnalysisData.cs index 8b0b6e1b3b97a..97ab1349215bc 100644 --- a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/FlowAnalysis/SymbolUsageAnalysis/SymbolUsageAnalysis.DataFlowAnalyzer.FlowGraphAnalysisData.cs +++ b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/FlowAnalysis/SymbolUsageAnalysis/SymbolUsageAnalysis.DataFlowAnalyzer.FlowGraphAnalysisData.cs @@ -183,7 +183,6 @@ private static PooledDictionary CreateAnalys /// /// /// - /// private static ImmutableHashSet GetCapturedLocals(ControlFlowGraph cfg) { using var _ = PooledHashSet.GetInstance(out var builder); diff --git a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/AbstractSpeculationAnalyzer.cs b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/AbstractSpeculationAnalyzer.cs index e255b20b7af62..55d220256f472 100644 --- a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/AbstractSpeculationAnalyzer.cs +++ b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/AbstractSpeculationAnalyzer.cs @@ -556,7 +556,6 @@ protected bool ReplacementChangesSemantics(SyntaxNode currentOriginalNode, Synta /// /// Checks whether the semantic symbols for the and are non-null and compatible. /// - /// public bool SymbolsForOriginalAndReplacedNodesAreCompatible() { if (this.SpeculativeSemanticModel == null) diff --git a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/SpecializedTasks.cs b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/SpecializedTasks.cs index 7f850fd522b94..f9400d9c0065e 100644 --- a/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/SpecializedTasks.cs +++ b/src/Workspaces/SharedUtilitiesAndExtensions/Compiler/Core/Utilities/SpecializedTasks.cs @@ -119,7 +119,6 @@ public static async ValueTask> WhenAll(this IRe /// The synchronous transformation to apply to the result of . /// The state to pass to and . /// The that the operation will observe. - /// public static ValueTask TransformWithoutIntermediateCancellationExceptionAsync( Func> func, Func transform, diff --git a/src/Workspaces/SharedUtilitiesAndExtensions/Workspace/Core/Extensions/SyntaxGeneratorExtensions.cs b/src/Workspaces/SharedUtilitiesAndExtensions/Workspace/Core/Extensions/SyntaxGeneratorExtensions.cs index 98e638b4b02b6..2f943663ea358 100644 --- a/src/Workspaces/SharedUtilitiesAndExtensions/Workspace/Core/Extensions/SyntaxGeneratorExtensions.cs +++ b/src/Workspaces/SharedUtilitiesAndExtensions/Workspace/Core/Extensions/SyntaxGeneratorExtensions.cs @@ -98,7 +98,6 @@ public static SyntaxNode IsPatternExpression(this SyntaxGeneratorInternal genera /// /// Generates a call to a method *through* an existing field or property symbol. /// - /// public static SyntaxNode GenerateDelegateThroughMemberStatement( this SyntaxGenerator generator, IMethodSymbol method, ISymbol throughMember) {