GRECLIPSE

compareTo operator should support type inferencing

Details

  • Type: Improvement Improvement
  • Status: Resolved Resolved
  • Priority: Major Major
  • 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

The type of the following is not inferred:

(1 <=> 2).intValue()

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Bob Tiernay added a comment -

Related is the following as well, which I'm putting here to prevent a proliferation of jira issues:

(1 < 2).booleanValue()
(1 > 2).booleanValue()
Show
Bob Tiernay added a comment - Related is the following as well, which I'm putting here to prevent a proliferation of jira issues: (1 < 2).booleanValue() (1 > 2).booleanValue()
Hide
Permalink
Andrew Eisenberg added a comment -

Hmmm...currently the types of all of these expressions are inferred to be String. Not right.

Show
Andrew Eisenberg added a comment - Hmmm...currently the types of all of these expressions are inferred to be String. Not right.
Hide
Permalink
Andrew Eisenberg added a comment -

I fixed the three cases mentioned here and I also went through the list of groovy operators to make sure there aren't others that I am missing.

Here: http://groovy.codehaus.org/Operators

I added support for things like '.', '.@', '.&'. Regression tests are included.

Show
Andrew Eisenberg added a comment - I fixed the three cases mentioned here and I also went through the list of groovy operators to make sure there aren't others that I am missing. Here: http://groovy.codehaus.org/Operators I added support for things like ' .', ' .@', '.&'. Regression tests are included.

People

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

Dates

  • Created:
    Updated:
    Resolved: