code review 2436041: exp/iterable: delete

exp/iterable: delete

Package iterable has outlived its utility.

It is an interesting demonstration, but it encourages
people to use iteration over channels where simple
iteration over array indices or a linked list would be
cheaper, simpler, and have fewer races.

[rsc, 2010-10-11 15:58:27]

Hello dsymonds, r (cc: golang-dev@googlegroups.com),

I'd like you to review this change.

[dsymonds2, 2010-10-11 20:21:00]

LGTM
On 12 Oct 2010 02:58, <rsc@golang.org> wrote:
> Reviewers: dsymonds, r,
>
> Message:
> Hello dsymonds, r (cc: golang-dev@googlegroups.com),
>
> I'd like you to review this change.
>
>
> Description:
> exp/iterable: delete
>
> Package iterable has outlived its utility.
>
> It is an interesting demonstration, but it encourages
> people to use iteration over channels where simple
> iteration over array indices or a linked list would be
> cheaper, simpler, and have fewer races.
>
> Please review this at http://codereview.appspot.com/2436041/
>
> Affected files:
> M src/pkg/Makefile
> M src/pkg/container/list/list.go
> M src/pkg/container/list/list_test.go
> R src/pkg/exp/iterable/Makefile
> R src/pkg/exp/iterable/array.go
> R src/pkg/exp/iterable/iterable.go
> R src/pkg/exp/iterable/iterable_test.go
>
>

[rsc, 2010-10-12 02:38:45]

*** Submitted as http://code.google.com/p/go/source/detail?r=ed32ab5693e1 ***

exp/iterable: delete

Package iterable has outlived its utility.

It is an interesting demonstration, but it encourages
people to use iteration over channels where simple
iteration over array indices or a linked list would be
cheaper, simpler, and have fewer races.

R=dsymonds, r
CC=golang-dev
http://codereview.appspot.com/2436041