remove arguments option in go

[jlongster]

I have a good proposal for error handling I think, but one thing that it does is gives you the ability to configure a go block to automatically propagate. I don't think we should add tons of options to go functions, but this is useful enough to merit it. It should look like:

go(function*() {
  // ....
}, { propagate: true });

Right now you'd always have to pass an empty array as the second argument. I've always found that argument redundant though since we have spawn. Either just use those variables directly in the go block, or use spawn to call the generator with the args, like spawn(gen(arg1, arg2)). In the rare case that you need to use this with a new function, just do spawn((function*(x, y, z) { ... })(x, y, z)).

Removing this gives us the ability to add the above option (which I will go into more details in the errors PR), without removing hardly any functionality. What do you think?

[nmn]

I like the way tj's co handles arguments:

go(function*(...args){
  //...
})(...args);

It's a little extra to type for simple go blocks, but it is actually very useful in many cases. It makes it easy to pass things around, etc.

As for composition, I think it makes a ton of sense that a go block should return a channel with the generator's final return value.

[jlongster]

What you're talking about I think is a different kind of function that doesn't immediately start the process, but returns a function that takes arguments and starts it with them. We can add something like that.

[nmn]

@jlongster Consider making that the default. As that would solve more cases that the current function.

Also, then we don't have to sacrifice arguments for the propagate option:

go(function*(...args) {
  // ....
}, { propagate: true })(...args);

[jlongster]

See my comment here: #15 (comment) Unfortunately I do not have time to contribute here.