Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Uniq optimization #37

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

TheBits commented Dec 12, 2010

This is small speed optimization.

TheBits commented Dec 12, 2010

This function don't use anywhere. :)

Contributor

aaronsw commented Dec 12, 2010

Using set loses the order of the items so this change doesn't actually work.

>>> list(set([7,1,2,3,1,4,5,6]))
[1, 2, 3, 4, 5, 6, 7]
>>> uniq([7,1,2,3,1,4,5,6])
[7, 1, 2, 3, 4, 5, 6]

This issue was closed.

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