Skip to content

util: use on-stack buffer for Utf8Value - #670

Closed
indutny wants to merge 1 commit into
nodejs:v1.xfrom
indutny:fix/utf8value-performance
Closed

util: use on-stack buffer for Utf8Value#670
indutny wants to merge 1 commit into
nodejs:v1.xfrom
indutny:fix/utf8value-performance

Conversation

@indutny

Copy link
Copy Markdown
Member

Improves crypto.createHash().update().digest() performance by 10%.

cc @piscisaureus@bnoordhuis

Improves `crypto.createHash().update().digest()` performance by 10%.
@indutnyindutny mentioned this pull request Jan 30, 2015
@bnoordhuis

Copy link
Copy Markdown
Member

LGTM

indutny added a commit that referenced this pull request Jan 30, 2015
Improves `crypto.createHash().update().digest()` performance by 10%.
PR-URL: #670
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
@indutny

Copy link
Copy Markdown
MemberAuthor

Yay! Landed in e17e6fb, thank you!

@indutnyindutny closed this Jan 30, 2015
@indutny
indutny deleted the fix/utf8value-performance branch January 30, 2015 19:09
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@indutny@bnoordhuis