Skip to content
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

Disable mlock and munlock in places if encryption wasn't requested #563

Merged
merged 2 commits into from Jan 18, 2019

Conversation

Projects
None yet
2 participants
@thomcc
Copy link
Contributor

commented Jan 18, 2019

This is a >6x speedup for me in the autocomplete demo, and makes importing your history from places in it bearable.

I also fixed the fmt issue that the new cargo fmt (which our CI has already) insists on.

thomcc added some commits Jan 18, 2019

@thomcc thomcc requested a review from mhammond Jan 18, 2019

@mhammond
Copy link
Member

left a comment

LGTM

@thomcc thomcc merged commit c0afe50 into master Jan 18, 2019

4 of 5 checks passed

ci/circleci: Sync integration tests Your tests failed on CircleCI
Details
Taskcluster (pull_request) TaskGroup: success
Details
ci/circleci: Check Rust formatting Your tests passed on CircleCI!
Details
ci/circleci: Rust tests - beta Your tests passed on CircleCI!
Details
ci/circleci: Rust tests - stable Your tests passed on CircleCI!
Details

@thomcc thomcc deleted the disable-mlock-munlock branch Jan 18, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.