Bind iterator only if needed #79

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
Contributor

victor-homyakov commented Oct 23, 2012

No description provided.

Collaborator

savetheclocktower commented Nov 2, 2012

I'd rather not — if we do this sort of thing here, there'd be no reason not to do it in 20 other places. One of the reasons we fall back to Prototype.K on all of these is so that we can assume an iterator is present and not care at all about what the iterator does.

If you can show me that it's worth it performance-wise, I might change my mind.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment