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

EmHttpRequestAdapter - Call detect instead of select..first #1042

Merged

bblimke merged 1 commit into bblimke:master from RicardoTrindade:patch-1

Feb 4, 2024

Contributor

RicardoTrindade commented Oct 27, 2023

Tiny tiny optimization PR

As per https://github.com/fastruby/fast-ruby/blob/main/code/enumerable/select-first-vs-detect.rb
Detect is slightly faster than select{}.first


          Call detect instead of select..first

5c898ed

Detect is slightly faster than select{}.first

RicardoTrindade changed the title ~~Call detect instead of select..first~~ EmHttpRequestAdapter - Call detect instead of select..first

Owner

bblimke commented Feb 4, 2024

Thank you for this optimisation @RicardoTrindade 👍

bblimke merged commit 890d41e into bblimke:master

RicardoTrindade deleted the patch-1 branch

February 4, 2024 23:18

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

Labels

None yet