Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Optimize maps:merge_with/3 for maps of equal size #8430

Merged
merged 1 commit into from
May 30, 2024

Conversation

josevalim
Copy link
Contributor

Previously, we were wrapping the combiner function without an actual need. We also use this opportunity to remove an unused argument.

Previously, we were wrapping the combiner function
without an actual need. We also use this opportunity
to remove an unused argument.
Copy link
Contributor

github-actions bot commented Apr 28, 2024

CT Test Results

    2 files     94 suites   34m 8s ⏱️
2 044 tests 1 996 ✅ 48 💤 0 ❌
2 353 runs  2 303 ✅ 50 💤 0 ❌

Results for commit c6ad0d5.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@rickard-green rickard-green added the team:VM Assigned to OTP team VM label Apr 29, 2024
@bjorng bjorng added enhancement testing currently being tested, tag is used by OTP internal CI labels May 29, 2024
@bjorng bjorng merged commit 844ada4 into erlang:master May 30, 2024
17 checks passed
@bjorng
Copy link
Contributor

bjorng commented May 30, 2024

Thanks for your pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants