-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add missing definition of operator() for hashing support #659
Conversation
It makes it easier to see if there are any missing object files.
Codecov Report
@@ Coverage Diff @@
## v0.x.x #659 +/- ##
==========================================
+ Coverage 90.18% 90.19% +<.01%
==========================================
Files 62 63 +1
Lines 4648 4649 +1
==========================================
+ Hits 4192 4193 +1
Misses 456 456
Continue to review full report at Codecov.
|
I guess you want to import the quorum intersection check in C++ ? |
void | ||
initialize() | ||
{ | ||
crypto_shorthash_keygen(sKey); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has to be called so you might want to stick a shared static this
in the bindings
7d20527
to
0395cf9
Compare
Yes. And it's a lot of code (2 Kloc or so) so I opted to bind to it rather than port it to D. Updated to add the initialization call. |
0395cf9
to
123ef1c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
std::hash is used for things like storing structs in std::set / std::map.
Required for the quorum intersection checker.