Work on Rail/ValRail API

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Critical Critical
  • Resolution: Fixed
  • Affects Version/s: X10 1.7.7, X10 2.0.0
  • Fix Version/s: None
  • Component/s: Class Libraries
  • Labels:
    None

Description

(FIXED) Why do we make a ValRail through Rail.makeVal?
There is no reason to allow making a Rail from a ValRail when we have a conversion op that does the same thing.

(FIXED)Why do ValRail/Rail have a get() method?
(CHANGED TO INT)Why are we using 'nat' for indexes, is it meant to be an unsigned type?
Is 'nat' deprecated?
Do we still need uninitialised rails? (These are currently unsound in the presence of constraints)

Issue Links

is depended upon by

Task - A task that needs to be done. XTENLANG-962 Array library redesign

  • 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 -

Move bulk of 2.0 performance items to 2.1 target.

Show
David Grove added a comment - Move bulk of 2.0 performance items to 2.1 target.
Hide
Permalink
Igor Peshansky added a comment -

This is getting extremely confusing (I've seen references to Array.makeVal()).

Show
Igor Peshansky added a comment - This is getting extremely confusing (I've seen references to Array.makeVal()).
Hide
Permalink
Dave Cunningham added a comment -

It ought to be possible to do Rail.make(n, other_rail); in order to truncate another rail. We lost this functionality when rail stopped being a function type.

Also, Rail.make(n, default_value); would be good for those times when a zero-ed rail is what you want.

Show
Dave Cunningham added a comment - It ought to be possible to do Rail.make(n, other_rail); in order to truncate another rail. We lost this functionality when rail stopped being a function type. Also, Rail.make(n, default_value); would be good for those times when a zero-ed rail is what you want.
Hide
Permalink
Dave Cunningham added a comment -

I did some work on this. I updated the samples and the test cases. Uninitialised rails are still needed by the Array code.

Show
Dave Cunningham added a comment - I did some work on this. I updated the samples and the test cases. Uninitialised rails are still needed by the Array code.
Hide
Permalink
David Grove added a comment -

Defer all open 2.0.1 issues to 2.0.2.

Show
David Grove added a comment - Defer all open 2.0.1 issues to 2.0.2.
Hide
Permalink
Vijay Saraswat added a comment -

Go ahead and make a proposal for what changes you want and then implement. Should be straight XRX/native code. I dont think any changes to the frontend should be required.

Show
Vijay Saraswat added a comment - Go ahead and make a proposal for what changes you want and then implement. Should be straight XRX/native code. I dont think any changes to the frontend should be required.
Hide
Permalink
David Grove added a comment -

Defer all unresolved issues from 2.0.2 to 2.0.3.

Show
David Grove added a comment - Defer all unresolved issues from 2.0.2 to 2.0.3.
Hide
Permalink
David Grove added a comment -

Original points raised in the issue have all been addressed.

Show
David Grove added a comment - Original points raised in the issue have all been addressed.
Hide
Permalink
David Grove added a comment -

bulk close of resolved issues.

Show
David Grove added a comment - bulk close of resolved issues.

People

  • Assignee:
    David Grove
    Reporter:
    Dave Cunningham
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: