Skip to content

Commit cbb1816

Browse files
DanTupcommit-bot@chromium.org
authored andcommitted
Remove unnecessary LSP hover handling comment/code
Change-Id: I1609dd07dae57b6bac48492c64f7ac1a7fb82d5b Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/105741 Auto-Submit: Danny Tuppeny <[email protected]> Commit-Queue: Brian Wilkerson <[email protected]> Reviewed-by: Brian Wilkerson <[email protected]>
1 parent 686df33 commit cbb1816

1 file changed

Lines changed: 0 additions & 5 deletions

File tree

pkg/analysis_server/lib/src/lsp/handlers/handler_hover.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,11 +63,6 @@ class HoverHandler extends MessageHandler<TextDocumentPositionParams, Hover> {
6363
if (hover.containingLibraryName != null &&
6464
hover.containingLibraryName.isNotEmpty) {
6565
content..writeln('*${hover.containingLibraryName}*')..writeln();
66-
} else if (hover.containingLibraryPath != null) {
67-
// TODO(dantup): Support displaying the package name (probably by adding
68-
// containingPackageName to the main hover?) once the analyzer work to
69-
// support this (inc Bazel/Gn) is done.
70-
// content..writeln('*${hover.containingPackageName}*')..writeln();
7166
}
7267

7368
// Doc comments.

0 commit comments

Comments
 (0)