Remove unnecessary LSP hover handling comment/code
Change-Id: I1609dd07dae57b6bac48492c64f7ac1a7fb82d5b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/105741
Auto-Submit: Danny Tuppeny <dantup@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
diff --git a/pkg/analysis_server/lib/src/lsp/handlers/handler_hover.dart b/pkg/analysis_server/lib/src/lsp/handlers/handler_hover.dart
index b724b1a..2d1bb84 100644
--- a/pkg/analysis_server/lib/src/lsp/handlers/handler_hover.dart
+++ b/pkg/analysis_server/lib/src/lsp/handlers/handler_hover.dart
@@ -63,11 +63,6 @@
if (hover.containingLibraryName != null &&
hover.containingLibraryName.isNotEmpty) {
content..writeln('*${hover.containingLibraryName}*')..writeln();
- } else if (hover.containingLibraryPath != null) {
- // TODO(dantup): Support displaying the package name (probably by adding
- // containingPackageName to the main hover?) once the analyzer work to
- // support this (inc Bazel/Gn) is done.
- // content..writeln('*${hover.containingPackageName}*')..writeln();
}
// Doc comments.
