groovy

RangeExpression providing wrong ClassNode

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.5.6
  • Fix Version/s: 1.5.7, 1.6-beta-2
  • Component/s: parser
  • Labels:
    None
  • Number of attachments :
    0

Description

When simple range is parsed (I attached the whole snippet to be accurate, but problem is not related to list):

(5..8).a
[1,2].a

The name of the ClassNode is "groovy.lang.Range", but its clazz is java.lang.Object and isInterface() method returns false (which I think is incorrect).

Issue Links

is duplicated by

Bug - A problem which impairs or prevents the functions of the product. GROOVY-3052 RangeExpression wrong line, column data

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Petr Hejl added a comment -

Sorry, most likely false alarm. I'll evaluate in more depth.

Show
Petr Hejl added a comment - Sorry, most likely false alarm. I'll evaluate in more depth.
Hide
Permalink
Guillaume Laforge added a comment -

If a false alarm, after further evaluation, don't forget to close the issue.

Show
Guillaume Laforge added a comment - If a false alarm, after further evaluation, don't forget to close the issue.
Hide
Permalink
blackdrag blackdrag added a comment -

I close the bug for now, because such things tend to be forgotten. If it really is an issue, then you can reopen the issue or fill a new issue

Show
blackdrag blackdrag added a comment - I close the bug for now, because such things tend to be forgotten. If it really is an issue, then you can reopen the issue or fill a new issue
Hide
Permalink
Petr Hejl added a comment -

Mea culpa. Not a bug.

Show
Petr Hejl added a comment - Mea culpa. Not a bug.
Hide
Permalink
Petr Hejl added a comment -

I finally found the original use case:

def rangeTest() {
    (1..10).a
}

Line 2 is properly parsed as PropertyExpression and its objectExpression is RangeExpression. However RangeExpression.getType().getName() is "java.lang.Object".

Show
Petr Hejl added a comment - I finally found the original use case: def rangeTest() { (1..10).a } Line 2 is properly parsed as PropertyExpression and its objectExpression is RangeExpression. However RangeExpression.getType().getName() is "java.lang.Object".

People

  • Assignee:
    blackdrag blackdrag
    Reporter:
    Petr Hejl
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: