X10

-> considered harmful

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Not A Bug
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: Class Library: Array Library
  • Labels:
    None
  • Number of attachments :
    0

Description

We've got special syntax,

R -> P

for a constant distribution. I don't think that constant distributions are so important as to need special syntax. All the other distributions make do with factory methods Dist.makeBlock(R), and even constant distributions can also be called Dist.makeConstant(R,P), so ... maybe we could get rid of that arrow and trim the syntax and complexity of the language down a touch?

Issue Links

is duplicated by

Improvement - An improvement or enhancement to an existing feature or task. XTENLANG-1671 Consider destroying the -> operation

  • 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
David Grove added a comment -

I think the syntax had some value when all arrays were defined over distributions, so -> was a useful shorthand for defining the distribution one needed for a local array. I agree that I don't see much value in it now that constant distributions are less important (local arrays are defined over regions, not over distributions).

Show
David Grove added a comment - I think the syntax had some value when all arrays were defined over distributions, so -> was a useful shorthand for defining the distribution one needed for a local array. I agree that I don't see much value in it now that constant distributions are less important (local arrays are defined over regions, not over distributions).
Hide
Permalink
David Grove added a comment -

Bulk assign all unscheduled issues with priority major or less to 2.1.0 target.

Show
David Grove added a comment - Bulk assign all unscheduled issues with priority major or less to 2.1.0 target.
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
Vijay Saraswat added a comment -

Agree. Lets remove from language.

Show
Vijay Saraswat added a comment - Agree. Lets remove from language.
Hide
Permalink
Vijay Saraswat added a comment -

Bard, please remove from manual.

Show
Vijay Saraswat added a comment - Bard, please remove from manual.
Hide
Permalink
Igor Peshansky added a comment -

This issue is out of date. -> is no longer special syntax for distributions, but a user-overloadable operator. So the original reason for this JIRA is gone. We may as well keep it.

Show
Igor Peshansky added a comment - This issue is out of date. -> is no longer special syntax for distributions, but a user-overloadable operator. So the original reason for this JIRA is gone. We may as well keep it.
Hide
Permalink
Bard Bloom added a comment -

Vijay, do you still want it gone, given Igor's fact?

Show
Bard Bloom added a comment - Vijay, do you still want it gone, given Igor's fact?
Hide
Permalink
Vijay Saraswat added a comment -

Given Igor's input this is no longer a lang design issue, but an API design issue.. so handing it over to Dave G.

You should still make sure that -> does not occur in the manual other than in the list of defined operators, and in the API section around arrays/region/dist/point.

Show
Vijay Saraswat added a comment - Given Igor's input this is no longer a lang design issue, but an API design issue.. so handing it over to Dave G. You should still make sure that -> does not occur in the manual other than in the list of defined operators, and in the API section around arrays/region/dist/point.
Hide
Permalink
David Grove added a comment -

Keeping -> as an operator on Region is harmless, so no point to break code just to remove it.

Show
David Grove added a comment - Keeping -> as an operator on Region is harmless, so no point to break code just to remove it.
Hide
Permalink
David Grove added a comment -

bulk close of resolved issues with unspecified fix target (svn head, won't fix, duplicate, etc).

Show
David Grove added a comment - bulk close of resolved issues with unspecified fix target (svn head, won't fix, duplicate, etc).

People

  • Assignee:
    David Grove
    Reporter:
    Bard Bloom
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: