-
-
Notifications
You must be signed in to change notification settings - Fork 3k
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
miscellaneous fixes #294
miscellaneous fixes #294
Conversation
682d2ba
to
80f598a
Compare
@jbenet CR? |
for i := int64(0); i < bufsize; i++ { | ||
n = randmath.Int63() | ||
for j := 0; j < 8 && i < bufsize; j++ { | ||
var n uint32 |
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 is reverting back to an old version of go-random. i.e. go-random shouldnt be here nevermind
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.
oh nope, this is correct
LGTM! |
|
License: MIT Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
remove 'adapter' concept instead, describe the component as the bitswap network it's still an adapter, but it's just not necessary to describe it as such
vanilla: 21.57 real 45.14 user 8.51 sys short: 14.40 real 31.13 user 5.56 sys License: MIT Signed-off-by: Brian Tiger Chow <brian@perfmode.com>
License: MIT Signed-off-by: Brian Tiger Chow <brian@perfmode.com>
48a5e7f
to
162169a
Compare
Another branch for miscellaneous fixes and updates. Feel free to push any minor changes to this branch. We can merge the batch before the end of the week.