fixup! Add feature to strike out obsolete symbols

dotnet · Mar 26, 2024 · 4aa6979 · 4aa6979
1 parent 9764c54
commit 4aa6979
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/src/Features/Lsif/Generator/Generator.cs b/src/Features/Lsif/Generator/Generator.cs
@@ -125,13 +125,8 @@ public async Task GenerateForProjectAsync(
                 {
                     QuickInfoOptions = options.SymbolDescriptionOptions.QuickInfoOptions with
                     {
-                        IncludeNavigationHintsInQuickInfo = false,
-                    },
-                    ClassificationOptions = options.SymbolDescriptionOptions.ClassificationOptions with
-                    {
-                        // Make sure to include 'deprecated' in the semantic token modifiers where applicable
-                        ClassifyObsoleteSymbols = true,
-                    },
+                        IncludeNavigationHintsInQuickInfo = false
+                    }
                 }
             };
 

diff --git a/src/Workspaces/Core/Portable/Classification/ClassificationOptions.cs b/src/Workspaces/Core/Portable/Classification/ClassificationOptions.cs
@@ -10,7 +10,7 @@ namespace Microsoft.CodeAnalysis.Classification;
 internal readonly record struct ClassificationOptions
 {
     [DataMember] public bool ClassifyReassignedVariables { get; init; } = false;
-    [DataMember] public bool ClassifyObsoleteSymbols { get; init; } = false;
+    [DataMember] public bool ClassifyObsoleteSymbols { get; init; } = true;
     [DataMember] public bool ColorizeRegexPatterns { get; init; } = true;
     [DataMember] public bool ColorizeJsonPatterns { get; init; } = true;
     [DataMember] public bool ForceFrozenPartialSemanticsForCrossProcessOperations { get; init; } = false;