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

kernel: Short-circuit when casting from list to list #7933

Merged
merged 1 commit into from Dec 20, 2023

Conversation

garazdawi
Copy link
Contributor

We can do this because we know that the list will contains valid unicode code points so there is no need to validate it again. This speeds up reading standard_io a lot.

Solves #7924

We can do this because we know that the list will contains valid
unicode code points so there is no need to validate it again. This
speeds up reading standard_io a lot.

Solves erlang#7924
@garazdawi garazdawi added the fix label Dec 4, 2023
@garazdawi garazdawi self-assigned this Dec 4, 2023
Copy link
Contributor

github-actions bot commented Dec 4, 2023

CT Test Results

       2 files       65 suites   1h 0m 31s ⏱️
1 505 tests 1 254 ✔️ 251 💤 0
1 697 runs  1 398 ✔️ 299 💤 0

Results for commit fc14619.

♻️ 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 Dec 4, 2023
@garazdawi garazdawi merged commit db4796b into erlang:maint Dec 20, 2023
17 checks passed
@garazdawi garazdawi deleted the lukas/kernel/fix-slow-read_line branch December 20, 2023 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants