Add method reverse() to List

[DartBot]

This issue was originally filed by @pjako

---

It would be nice to have a method on Lists to reverse them...

print([1,2,3].reverse());

out:
3 2 1

[dgrove]

Removed Type-Defect label.
Added Type-Enhancement, Area-Library, Triaged labels.

[sethladd]

Added Library-Collections label.

[alan-knight]

I note that this is polyfilled in internationalization, which should be removed once this is available.

[DartBot]

This comment was originally written by @seaneagan

---

As mentioned in issue #1305, I think this method should return a view (wrapper) on the original List, rather than copying each element into a brand new List. Also, "reverse" has a mutative connotation. I think it would be better as a getter, some options for the name are:

"reversed"
"inReverse"
"backwards"

In the case where one does want to copy the elements into a brand new List, they could do the following:

var reversedCopy = new List.from(list.reversed);

[alan-knight]

I agree that reversed is better for something that returns either a copy or a wrapper.

[munificent]

Issue #5728 has been merged into this issue.

[DartBot]

This comment was originally written by [email protected]

---

It'd be great if we had a unicode-aware reverse method on String as well.

[dgrove]

Removed Library-Collections label.
Added Library-Collection label.

[rakudrama]

Just last week I wanted a mutating reverse of a subrange of a list.

For example, you can exchange two adjacent blocks [start, middle) and [middle,end) like this:

list.reverse(start, end);
list.reverse(start, middle);
list.reverse(middle, end);

[alan-knight]

There is now a reversed() method on List. Should this be closed?

---

cc @floitschG.
cc @lrhn.

[lrhn]

Closing as fixed.
The original request has been fulfilled (even if its not mutating).

If there is a need for a mutating "reverse" method, it should be filed as a separate feature request.

---

Added Fixed label.