search

intel / buxton

8 stars 9 forks source link

BuxtonHashmap #134

Closed ikeydoherty closed 10 years ago

ikeydoherty commented 10 years ago

Pushing this out for initial review..

You'll notice that towards the last few commits I got distracted by various leaks. Thankfully using BuxtonHashmap now makes it easier to spot leaks, as we don't have pages of the old hashmap errors, just the ones in protocol.

I'll work on protocol separately, but the majority of legwork is done in this pull request. I'd like to get feedback mainly on the test suite, and see if we need to expand it at all to ensure there are no errors at all :)

Once this work is totally completed I suggest we completely remove the old hashmap from the tree.

ikeydoherty commented 10 years ago

Rebasing now but @jobol's changes seem to have broken my patchset - investigating.

ikeydoherty commented 10 years ago

Nope, glitch in the matrix. :p

ikeydoherty commented 10 years ago

Closing as I have a clean rebase to push