Skip to content

benchmark: add a few querystring benchmarks - #846

Closed
mscdex wants to merge 1 commit into
nodejs:v1.xfrom
mscdex:add-qs-benchmarks
Closed

benchmark: add a few querystring benchmarks#846
mscdex wants to merge 1 commit into
nodejs:v1.xfrom
mscdex:add-qs-benchmarks

Conversation

@mscdex

Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

does having the || []; portion give a benefit here?

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Leftovers from copy and paste from another benchmark. I'll remove it.

@brendanashworth

Copy link
Copy Markdown
Contributor

Thanks @mscdex, merged in 35ed799!

mscdex referenced this pull request Feb 15, 2015
PR-URL: #847
Reviewed-By: Brendan Ashworth <brendan.ashworth@me.com>
@mscdex
mscdex deleted the add-qs-benchmarks branch December 17, 2015 18:24
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

@mscdex@brendanashworth