X10
  1. X10
  2. XTENLANG-379

Outline view presents constructor for class Foo as Foo() instead of this().

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: X10 2.0.1
    • Fix Version/s: X10 2.0.4
    • Component/s: X10DT
    • Labels:
      None
    • Environment:
      All
    • Number of attachments :
      1

      Description

      In the outline view, constructor for a class Foo are displayed as Foo() instead of as this() (this is the new X10 v1.7 syntax for constructors).

        Activity

        Hide
        Nate Nystrom added a comment -

        The name is stored in the node because of Java. The following should be an error:

        class A {
            B() { }
        }
        

        The parser will not examine the name to make sure it matches the declared class name. Instead, the name in the constructor node is compared against the class name during type checking. So, we can't remove the name from the node that easily.

        But, yes the name can be made protected.

        Show
        Nate Nystrom added a comment - The name is stored in the node because of Java. The following should be an error: class A { B() { } } The parser will not examine the name to make sure it matches the declared class name. Instead, the name in the constructor node is compared against the class name during type checking. So, we can't remove the name from the node that easily. But, yes the name can be made protected.
        Hide
        David Grove added a comment -

        Moving all open items for X10DT 1.7 to X10DT 2.0 since 1.7 has been released.

        Show
        David Grove added a comment - Moving all open items for X10DT 1.7 to X10DT 2.0 since 1.7 has been released.
        Hide
        David Grove added a comment -

        moving open issues in X10 DT 2.0.1.1 to X10 DT 2.0.2.2 as discussed with Bob.

        Show
        David Grove added a comment - moving open issues in X10 DT 2.0.1.1 to X10 DT 2.0.2.2 as discussed with Bob.
        Hide
        Robert M. Fuhrer added a comment -

        This was the result of the front-end's replacing "this" with the containing class name. Instituted a work-around in the X10LabelProvider.

        Show
        Robert M. Fuhrer added a comment - This was the result of the front-end's replacing "this" with the containing class name. Instituted a work-around in the X10LabelProvider.
        Hide
        David Grove added a comment -

        bulk close of all resolved issues as part of closing 2.0.4 items.

        Show
        David Grove added a comment - bulk close of all resolved issues as part of closing 2.0.4 items.

          People

          • Assignee:
            Robert M. Fuhrer
            Reporter:
            Vijay Saraswat
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 1 day
              1d
              Remaining:
              Remaining Estimate - 1 day
              1d
              Logged:
              Time Spent - Not Specified
              Not Specified