Skip to content
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

Code completion not working after ... (spread) #36733

Closed
jwren opened this issue Apr 24, 2019 · 9 comments
Closed

Code completion not working after ... (spread) #36733

jwren opened this issue Apr 24, 2019 · 9 comments
Assignees
Labels
analyzer-completion Issues with the analysis server's code completion feature area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Milestone

Comments

@jwren
Copy link
Member

jwren commented Apr 24, 2019

No completions from the Analysis Server at the |:

void main() async {
  final list1 = [1, 2, 3];

  final list2 = [0, ...li|];

  print(list2);
}

Issue originally filed at https://youtrack.jetbrains.com/issue/WEB-38571

@jwren jwren added the analyzer-completion Issues with the analysis server's code completion feature label Apr 24, 2019
@devoncarew devoncarew added the area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. label Apr 24, 2019
@scheglov
Copy link
Contributor

@stereotype441 stereotype441 added this to the Dart 2.3 milestone Apr 25, 2019
@stereotype441 stereotype441 added the P1 A high priority bug; for example, a single project is unusable or has many test failures label Apr 25, 2019
@scheglov
Copy link
Contributor

There is a couple of other cases when completion does not work perfectly, even with this CL.

  1. No completion for the loop variable in IfElement, e.g. no name in [for (var name in names) ^].
  2. No else completion in IfElement.

@stereotype441
Copy link
Member

We decided it's not necessary to try to cherry-pick a fix into Dart 2.3.

@stereotype441 stereotype441 modified the milestones: Dart 2.3, D24 Release Apr 25, 2019
@stereotype441 stereotype441 added P2 A bug or feature request we're likely to work on and removed P1 A high priority bug; for example, a single project is unusable or has many test failures labels Apr 25, 2019
dart-bot pushed a commit that referenced this issue Apr 25, 2019
R=brianwilkerson@google.com

Bug: #36733
Change-Id: I019741086495eaaf1f2dc078d801f61a0919cc78
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/100523
Reviewed-by: Paul Berry <paulberry@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>
@aadilmaan
Copy link
Contributor

Did this change land already?

@scheglov
Copy link
Contributor

scheglov commented May 6, 2019

Yes, see the dart-bot message just above.

But #36733 (comment) is not addresses yet.

@stereotype441
Copy link
Member

@scheglov are the remaining issues likely to be addressed in D24? If not let's change the milestone.

@scheglov scheglov modified the milestones: D24 Release, Future May 29, 2019
@scheglov
Copy link
Contributor

I moved it from D24 to Future.

@aadilmaan aadilmaan modified the milestones: Future, D25 Release Jun 4, 2019
@bwilkerson
Copy link
Member

Completion now works after the for loop parts, but still doesn't complete else in the following example

void f(List<String> names) {
  var x = [if (names.isNotEmpty) names[0] el ];
}

@bwilkerson bwilkerson added this to To do in Code completion update 2021 via automation Jan 23, 2021
@srawlins srawlins changed the title Code completion not working after ... Code completion not working after ... (spread) Apr 28, 2021
@srawlins srawlins added the type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) label Nov 16, 2021
@srawlins
Copy link
Member

As spread completions work now, this issue is closed. I've opened a new issue for the unrelated completion that @bwilkerson brings up in #36733 (comment): #48837

Code completion update 2021 automation moved this from To do to Done Apr 19, 2022
copybara-service bot pushed a commit that referenced this issue Apr 19, 2022
Bug: #36733 and #48837
Change-Id: I4c846426c17fa7ebc5dd544b24f1667945a9f1a7
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/241214
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Samuel Rawlins <srawlins@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer-completion Issues with the analysis server's code completion feature area-analyzer Use area-analyzer for Dart analyzer issues, including the analysis server and code completion. P2 A bug or feature request we're likely to work on type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Development

No branches or pull requests

7 participants