homepage

In the documentation for `itertools.islice` I see this line:
 it = iter(xrange(s.start or 0, s.stop or sys.maxint, s.step or 1))
Is it really okay to do `s.stop or sys.maxint`? I'm assuming this was targeting `None`, but what if `s.stop == 0`? And `s.step` could (pathologically) be `0` too, no?

`step` argument for xrange() could not be 0.
But `s.stop or sys.maxint` is really a problem, in the case of `s.stop == 0`.
So the given `Equivalent to` python code in the doc is not precisely equivalent to the c implementation. The doc needs a fix.

I've got from here. Thanks.

The attached patch fixes this

LGTM. However note, that for 2.7 the patch should be modified (maxsize -> maxint, range -> xrange).

ysj: The "equivalent" means "roughly equivalent" not "precisely equivalent". The purpose of the code in the docs is to help communicate what islice() is all about. Practicality beats purity in this regard. I know of no one who has ever been mislead by the islice() docs.
I don't really like the proposed patch -- it improves a bit on technical accuracy but forgoes readability and still falls short on what stop=None does and on which arguments are required. 
That said, I don't really like the current version of the "roughly equivalent code either". It should probably be rewritten completely (no longer using slice() and it should show the actual looping logic. For clarity, it may be helpful to split it into two code-equivalents, one for the finite case (where stop is specified) and one for the infinite case (where stop is None). 
As it stands, the examples are beating the code equivalent when it comes to communicating what islice() does. I don't want to lengthen the code fragment unless it becomes *both* more communicative and more accurate.

Closing for the reasons lists above.