Skip to content

Conversation

decioferreira
Copy link

Relates to #107 and elm/bytes#19.

Updated _Bytes_read_string to use array instead of string as accumulator.

When compiling rtfeldman/elm-css, before changes:

elm-css % rm -rf guida-stuff; time ../guida-lang/compiler/bin/index.js make
Dependencies ready!
Success! Compiled 25 modules.
../guida-lang/compiler/bin/index.js make  21.36s user 2.76s system 101% cpu 23.686 total

After changes:

elm-css % rm -rf guida-stuff; time ../guida-lang/compiler/bin/index.js make
Dependencies ready!
Success! Compiled 25 modules.
../guida-lang/compiler/bin/index.js make  13.95s user 2.20s system 107% cpu 14.984 total

@decioferreira
Copy link
Author

Also relates to elm/bytes#25 and elm-janitor/bytes@9a496d7

@decioferreira decioferreira merged commit 85c9c83 into master Aug 23, 2025
1 of 2 checks passed
@decioferreira decioferreira deleted the bytes-read-string branch August 23, 2025 00:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant