New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

perf: use optional chaining #850

Merged

Fdawgs merged 1 commit into master from perf/chain

Jan 5, 2025

Member

Fdawgs commented Jan 5, 2025

Teeny tiny bit faster for a single hop: https://github.com/RafaelGSS/nodejs-bench-operations/blob/main/RESULTS-v22.md#optional-chain--vs--operator

Also allows some ternary operators to be changed to or operators.

Checklist

run npm run test and npm run benchmark
commit message and code follows the Developer's Certification of Origin
and the Code of conduct


          perf: use optional chaining

cf692c8

Fdawgs merged commit 79384b1 into master

13 checks passed

Fdawgs deleted the perf/chain branch

January 5, 2025 19:09

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet