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.

Sign up for GitHub

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

util: add fast path for Latin1 decoding #55275

Merged
merged 15 commits into from
Dec 3, 2024
Merged

util: add fast path for Latin1 decoding #55275

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
b12ba58
util: add fast path for Latin1 decoding
Oct 5, 2024
3c239bd
handle input correctly in decode functions
Oct 5, 2024
17f317e
repair
Oct 5, 2024
e0a40c6
Update lib/internal/encoding.js
mertcanaltin Oct 5, 2024
b7420b5
add ignoreBOM and fatal support to decodeLatin1
Oct 5, 2024
8bc747e
lint
Oct 5, 2024
fced029
update tests to support ignoreBOM and fatal for decodeLatin1
Oct 5, 2024
af7cca5
lint
Oct 5, 2024
659f1b6
correct BOM check in decodeLatin1 for uint8_t comparison
Oct 5, 2024
b92e571
add BOM test case to cover 0xFF in decodeLatin1
Oct 5, 2024
94109bf
Update encoding_binding.cc
mertcanaltin Oct 6, 2024
6dec37f
delete result.resize
Oct 6, 2024
f62f4c4
Update encoding_binding.cc
mertcanaltin Oct 7, 2024
f96e2ef
Update encoding_binding.cc
mertcanaltin Oct 7, 2024
77bc5ee
update text-decoder to use windows-1252 encoding
Oct 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
update text-decoder to use windows-1252 encoding
  • Loading branch information
Mert Can Altin committed Oct 8, 2024
commit 77bc5eeec288eb305a08269ef9b0f6a68f42e3b8
2 changes: 1 addition & 1 deletion benchmark/util/text-decoder.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
const common = require('../common.js');

const bench = common.createBenchmark(main, {
encoding: ['utf-8', 'latin1', 'iso-8859-3'],
encoding: ['utf-8', 'windows-1252', 'iso-8859-3'],
ignoreBOM: [0, 1],
fatal: [0, 1],
len: [256, 1024 * 16, 1024 * 128],
Expand Down
Loading