Add list.removeAt method.

Add list.removeAt method.

Committed: https://code.google.com/p/dart/source/detail?r=12627

[ngeoffray, 2012-09-20 09:16:24]

https://codereview.chromium.org/10970009/diff/1/lib/core/list.dart
File lib/core/list.dart (right):

https://codereview.chromium.org/10970009/diff/1/lib/core/list.dart#newcode99
lib/core/list.dart:99: * Removes the element at position[index] from the list.
FYI: we did not have removeAt initially because it makes it too simple to use,
without showing that this is an expensive operation. So we have removeRange
already that does removeAt, but with the 'Range' in the name, it *feels*
expensive.

[Johnni Winther, 2012-09-20 09:22:50]

lgtm

https://codereview.chromium.org/10970009/diff/1/runtime/lib/array.dart
File runtime/lib/array.dart (right):

https://codereview.chromium.org/10970009/diff/1/runtime/lib/array.dart#newcod...
runtime/lib/array.dart:171: "Cannot modify an immitable array");
immitable => immutable

[Lasse Reichstein Nielsen, 2012-09-20 10:50:29]

https://codereview.chromium.org/10970009/diff/1/lib/core/list.dart
File lib/core/list.dart (right):

https://codereview.chromium.org/10970009/diff/1/lib/core/list.dart#newcode99
lib/core/list.dart:99: * Removes the element at position[index] from the list.
Performance through inconvenience :)
I don't think it's going to work in the long run. I'd rather provide the methods
that people want to call than make it hard to get the job done (even if it's
done slowly).

https://codereview.chromium.org/10970009/diff/1/runtime/lib/array.dart
File runtime/lib/array.dart (right):

https://codereview.chromium.org/10970009/diff/1/runtime/lib/array.dart#newcod...
runtime/lib/array.dart:171: "Cannot modify an immitable array");
On 2012/09/20 09:22:50, Johnni Winther wrote:
> immitable => immutable

Done.