Also, why no 'const' correctness: void * swisstable_map_search(void *root, void *key, size_t keysize); vs. void * swisstable_map_search(const void *root, const void *key, size_t keysize); Does the 'search' function write to the key? Does it it write to hashmap?
-
-
-
Damn. I'm willing to do so when I get back from my walk if you want :-). I think this would be a good effort.
1 reply 0 retweets 7 likes -
Replying to @ErrataRob @bitsrot
That would be very welcome! I just spent a couple of hours on this today to see if it would be worth it, I think it will be and plan to keep working on it
2 replies 0 retweets 3 likes -
Bah. Sorry, I'd love to play with it and do some benchmarks, as it's exactly the sort of thing I'd love to play with, but Abseil's dependency wall is just too high to climb.
1 reply 0 retweets 0 likes -
Replying to @ErrataRob @bitsrot
Hmm, it needs c++17 for for string_view (e.g. gcc7 otherwise you have to copy a lot of buffers around to translate between c/c++), otherwise that's it. I could make it optional, then I think any old gcc will do, but it will be much slower..
2 replies 0 retweets 1 like -
The frustration I'm having is trying to get the build system to work. I've never heard of Bazel before.
1 reply 0 retweets 0 likes -
1 reply 0 retweets 1 like -
Replying to @ErrataRob @bitsrot
oh! You can skip that, just use cmake - just these commands should be enough: https://github.com/taviso/swisstable#building …
1 reply 2 retweets 5 likes -
damn, somehow I missed the obvious cmake instructions hunting down this Bazel thing
1 reply 0 retweets 0 likes
I cleaned up the types a bit this afternoon
It's probably not perfect, I need to use it more to get a feel for what's missing, but it's a start!
-
This Tweet is unavailable.
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.