GRECLIPSE

Content assist for closure arguments

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 2.5.1Release
  • Fix Version/s: 2.5.2.Release
  • Component/s: Content Assist
  • Labels:
    None
  • Number of attachments :
    0

Description

It would be nice to be able to get content assist for the closure's arguments at the invocation point f(1,2) in the following example:

def f = { int x, int y -> }

f(1,2) // Content assist arguments

Issue Links

is related to

Improvement - An improvement or enhancement to an existing feature or task. GRECLIPSE-1150 Include method-like content assist proposal for local variables that are assigned to a closure

  • Major - Major loss of function.
  • Open - The issue is open and ready for the assignee to start work on it.

Improvement - An improvement or enhancement to an existing feature or task. GRECLIPSE-1170 If a field has an initializer with a known static type, should use that type as the field's type

  • Major - Major loss of function.
  • Resolved - A resolution has been taken, and it is awaiting verification by reporter. From here issues are either reopened, or are closed.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Andrew Eisenberg added a comment -

It is a different thing if the target closure of the content assist is a field or a local variable.

Show
Andrew Eisenberg added a comment - It is a different thing if the target closure of the content assist is a field or a local variable.
Hide
Permalink
Andrew Eisenberg added a comment -

I have a solution that works for field proposals, but not for local variable proposals. I am going to write up some tests, close this issue and open a new one for local variable proposals.

That one will be harder since at the point of proposing, we don't have inferred type information for local variables.

Show
Andrew Eisenberg added a comment - I have a solution that works for field proposals, but not for local variable proposals. I am going to write up some tests, close this issue and open a new one for local variable proposals. That one will be harder since at the point of proposing, we don't have inferred type information for local variables.
Hide
Permalink
Bob Tiernay added a comment -

Awesome! Great work Andrew

Show
Bob Tiernay added a comment - Awesome! Great work Andrew
Hide
Permalink
Andrew Eisenberg added a comment -

This issue is specifically about local variables and closures.

Show
Andrew Eisenberg added a comment - This issue is specifically about local variables and closures.
Hide
Permalink
Bob Tiernay added a comment -

In the following example:

new C().c(i)

class C {
   def c = { int i ->
      println i
   } 
}

The type of c is not inferred, but it has content assist for the argument i. It would be nice if type inferencing could detect the fact that it is a Closure since content assist seems to already know this.

Show
Bob Tiernay added a comment - In the following example: new C().c(i) class C { def c = { int i -> println i } } The type of c is not inferred, but it has content assist for the argument i . It would be nice if type inferencing could detect the fact that it is a Closure since content assist seems to already know this.
Hide
Permalink
Bob Tiernay added a comment -

In the following example:

new C().c(i)

class C {
   def c = { int i ->
      println i
   } 
}

The type of c is not inferred, but it has content assist for the argument i. It would be nice if type inferencing could detect the fact that it is a Closure since content assist seems to already know this.

Show
Bob Tiernay added a comment - In the following example: new C().c(i) class C { def c = { int i -> println i } } The type of c is not inferred, but it has content assist for the argument i . It would be nice if type inferencing could detect the fact that it is a Closure since content assist seems to already know this.
Hide
Permalink
Andrew Eisenberg added a comment -

Created GRECLIPSE-1170 because this is a more general issue and might be solvable in a different way.

Show
Andrew Eisenberg added a comment - Created GRECLIPSE-1170 because this is a more general issue and might be solvable in a different way.

People

  • Assignee:
    Andrew Eisenberg
    Reporter:
    Bob Tiernay
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: