search

pytries / hat-trie

HAT-Trie for Python
MIT License
86 stars 21 forks source link

Fix issue where `iterkeys` does not decode keys but `keys` does. #9

Closed b4hand closed 10 years ago

b4hand commented 10 years ago

This fixes a bug where the result of list(trie.iterkeys()) does not equal trie.keys() because iterkeys was not getting overridden in Trie.

kmike commented 10 years ago

Thanks! This makes sense. Travis build failure for Python 3.2 was strange (it failed to install Cython); I've restarted the build, it seems to work now (failing test_iterkeys test).

b4hand commented 10 years ago

Sorry, I'm still on a Python 2.7 environment locally, so I'm still getting used to Python 3.

By the way, I recommend you merge this one before #8, and then I can rebase #8 to include this change as it's a bug that I discovered along the way.

kmike commented 10 years ago

Thanks! Could you please also add yourself to README under "Authors and Contributors" section? I should have asked that earlier..