languages/dart-sdk
languages/dart-sdk
1
0
Fork 0
mirror of https://github.com/dart-lang/sdk synced 2024-10-14 12:14:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove unnecessary LSP hover handling comment/code

Browse source 
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>
This commit is contained in:
Danny Tuppeny 2019-06-12 13:25:27 +00:00 committed by commit-bot@chromium.org
parent 686df33d99
commit cbb1816177
1 changed files with 0 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
pkg/analysis_server/lib/src/lsp/handlers/handler_hover.dart
View file

@ -63,11 +63,6 @@ class HoverHandler extends MessageHandler<TextDocumentPositionParams, Hover> {
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.