Skip to content
This repository has been archived by the owner on May 14, 2020. It is now read-only.

Performance update: & is faster than modulus, NaN check. #13

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

kspalaiologos
Copy link

No description provided.

@ptrcnull
Copy link

Duplicate of #11.

@kspalaiologos
Copy link
Author

#12 too.

but I just wanted to contribute to the most useful javascript library :/

@kspalaiologos kspalaiologos changed the title Performance update: & is faster than modulus. Performance update: & is faster than modulus, NaN check. Feb 12, 2019
@i-voted-for-trump i-voted-for-trump locked as spam and limited conversation to collaborators Feb 14, 2019
@i-voted-for-trump i-voted-for-trump unlocked this conversation Feb 14, 2019
@jonschlinkert
Copy link
Collaborator

but I just wanted to contribute to the most useful javascript library :/

To be clear, is this a real PR, or are you trolling and violating GitHub's code of conduct?

@selfisekai
Copy link

this NaN check actually won't work
https://stackoverflow.com/a/19955919/8222484

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants