X10

Better error message when having more statements before a call to "this(...)

Details

  • Number of attachments :
    0

Description

class Bla {
	def this() {
		val z = 0;
		this(4);
	} 
	def this(i:String) {}
}

C:\cygwin\home\Yoav\test\Hello.x10:8: No valid method call found for apply(x10.lang.String) in Bla{self==Bla#this}.

The error should be:
Call to "this" must be the first statement in constructor.

(We should copy the error messages wording from Java!)

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Igor Peshansky added a comment -

I thought the proto rules in X10 allowed statements before the this(), super(), and property() calls...

Show
Igor Peshansky added a comment - I thought the proto rules in X10 allowed statements before the this() , super() , and property() calls...
Hide
Permalink
Igor Peshansky added a comment -

Bulk-assigning unscheduled issues to the next release.

Show
Igor Peshansky added a comment - Bulk-assigning unscheduled issues to the next release.
Hide
Permalink
David Grove added a comment -

bulk defer of features/tasks/wishes from 2.1.0 to 2.1.1.

Show
David Grove added a comment - bulk defer of features/tasks/wishes from 2.1.0 to 2.1.1.
Hide
Permalink
David Grove added a comment -

bulk move of all unresolved issues from 2.1.1 to 2.1.2

Show
David Grove added a comment - bulk move of all unresolved issues from 2.1.1 to 2.1.2
Hide
Permalink
David Grove added a comment -

defer all non-critical 2.1.2 issues to 2.2.

Show
David Grove added a comment - defer all non-critical 2.1.2 issues to 2.2.
Hide
Permalink
Igor Peshansky added a comment -

We need to change the parsing strategy and push the disambiguation of whether this() is a constructor call or a closure call into the typechecking phase, rather than assuming that the first this() statement is a constructor call...

Show
Igor Peshansky added a comment - We need to change the parsing strategy and push the disambiguation of whether this() is a constructor call or a closure call into the typechecking phase, rather than assuming that the first this() statement is a constructor call...
Hide
Permalink
Vijay Saraswat added a comment -

Move to 2.2.1 for lack of time. No issue with backwards compatibility.

Show
Vijay Saraswat added a comment - Move to 2.2.1 for lack of time. No issue with backwards compatibility.
Hide
Permalink
David Grove added a comment -

bulk defer of open issues to 2.2.2.

Show
David Grove added a comment - bulk defer of open issues to 2.2.2.
Hide
Permalink
David Grove added a comment -

bulk defer of issues to 2.2.3.

Show
David Grove added a comment - bulk defer of issues to 2.2.3.
Hide
Permalink
David Grove added a comment -

bulk defer of 2.3.0 open issues to 2.3.1.

Show
David Grove added a comment - bulk defer of 2.3.0 open issues to 2.3.1.
Hide
Permalink
David Grove added a comment -

bulk defer to 2.3.2

Show
David Grove added a comment - bulk defer to 2.3.2

People

  • Assignee:
    Unassigned
    Reporter:
    Yoav Zibin
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:

Time Tracking

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