-
Notifications
You must be signed in to change notification settings - Fork 4k
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 code for arbitrary diagnostic taggers #72409
Remove code for arbitrary diagnostic taggers #72409
Conversation
// The .NET Foundation licenses this file to you under the MIT license. | ||
// See the LICENSE file in the project root for more information. | ||
|
||
#nullable disable |
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.
no longer used. missed in prior PRs.
@@ -91,9 +91,6 @@ protected sealed override ITaggerEventSource CreateEventSource(ITextView? textVi | |||
private async Task ProduceTagsAsync( | |||
TaggerContext<TTag> context, DocumentSnapshotSpan documentSpanToTag, CancellationToken cancellationToken) | |||
{ | |||
if (!_callback.IsEnabled) |
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.
no concept of 'enabled' or not. inline diagnostics aren't enabled/disabled depending on if we're using lsp diagnostics or not.
@@ -111,4 +111,36 @@ private static ITaggerEventSource CreateEventSourceWorker(ITextBuffer subjectBuf | |||
TaggerEventSources.OnDiagnosticsChanged(subjectBuffer, diagnosticService), | |||
TaggerEventSources.OnTextChanged(subjectBuffer)); | |||
} | |||
|
|||
protected ITagSpan<TTag>? CreateTagSpan(Workspace workspace, SnapshotSpan span, DiagnosticData data) |
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.
moved from intermediary subclass to this type. methods unchanged.
|
||
namespace Microsoft.CodeAnalysis.Editor.InlineDiagnostics | ||
namespace Microsoft.CodeAnalysis.Editor.InlineDiagnostics; |
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.
view with whitespace off.
IEditorFormatMapService editorFormatMapService, | ||
IClassificationFormatMapService classificationFormatMapService, | ||
IClassificationTypeRegistryService classificationTypeRegistryService) | ||
: AbstractDiagnosticsTaggerProvider<InlineDiagnosticsTag>( |
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.
moved to primary constructor. and now derives from AbstractDiagnosticsTaggerProvider. AbstractDiagnosticsAdornmentTaggerProvider was merged into AbstractDiagnosticsTaggerProvider
We only have the inline-diagnostics tagger now. So moved all code related to that to a single location.
Followup to #62735.