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.

Sign up for GitHub

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

Jump to bottom

ds-help: avoid unnecessary allocs when posssible and make use of RawKey #3407

Merged
merged 1 commit into from
Nov 30, 2016
Merged

ds-help: avoid unnecessary allocs when posssible and make use of RawKey #3407

merged 1 commit into from
Nov 30, 2016

Conversation

kevina
Copy link
Contributor

@kevina kevina commented Nov 22, 2016 •
edited
Loading

No description provided.

@kevina kevina added status/blocked Unable to be worked further until needs are met status/in-progress In progress labels Nov 22, 2016
@whyrusleeping whyrusleeping added status/ready Ready to be worked and removed status/in-progress In progress labels Nov 28, 2016
@kevina kevina removed the status/blocked Unable to be worked further until needs are met label Nov 29, 2016
@kevina
ds-help: avoid unnecessary allocs when posssible and make use of RawKey
1c7e2b2 
License: MIT
Signed-off-by: Kevin Atkinson <k@kevina.org>
@kevina kevina added the need/review Needs a review label Nov 29, 2016
whyrusleeping
whyrusleeping approved these changes Nov 30, 2016
View reviewed changes
Copy link
Member

@whyrusleeping whyrusleeping left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This LGTM, lets get review from @Kubuxu as well

@whyrusleeping whyrusleeping merged commit f8635ba into master Nov 30, 2016
@whyrusleeping whyrusleeping deleted the kevina/dskey-minalloc branch November 30, 2016 22:48
@whyrusleeping whyrusleeping removed the status/ready Ready to be worked label Nov 30, 2016
@kevina kevina removed the need/review Needs a review label Nov 30, 2016
@ghost ghost mentioned this pull request Dec 23, 2016
Closed
@ajnavarro ajnavarro mentioned this pull request Aug 24, 2022
Closed
72 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kubuxu Kubuxu Kubuxu approved these changes

@whyrusleeping whyrusleeping whyrusleeping approved these changes

Assignees

@whyrusleeping whyrusleeping

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@kevina @whyrusleeping @Kubuxu