From 00b81c7afee7bdc491a42ea9876f4a66a9494e91 Mon Sep 17 00:00:00 2001 From: Yagiz Nizipli Date: Thu, 2 Feb 2023 13:04:53 -0500 Subject: [PATCH] src: replace icu with simdutf for char counts MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR-URL: https://github.com/nodejs/node/pull/46472 Reviewed-By: Luigi Pinca Reviewed-By: Anna Henningsen Reviewed-By: Tobias Nießen Reviewed-By: Minwoo Jung Reviewed-By: James M Snell Reviewed-By: Rafael Gonzaga --- src/inspector/node_string.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/inspector/node_string.cc b/src/inspector/node_string.cc index 4cb8b573cc1312..c020130ae65c52 100644 --- a/src/inspector/node_string.cc +++ b/src/inspector/node_string.cc @@ -1,5 +1,6 @@ #include "node_string.h" #include "node/inspector/protocol/Protocol.h" +#include "simdutf.h" #include @@ -118,9 +119,8 @@ const uint8_t* CharactersUTF8(const String& s) { } size_t CharacterCount(const String& s) { - icu::UnicodeString utf16 = - icu::UnicodeString::fromUTF8(icu::StringPiece(s.data(), s.length())); - return utf16.countChar32(); + // TODO(@anonrig): Test to make sure CharacterCount returns correctly. + return simdutf::utf32_length_from_utf8(s.data(), s.length()); } } // namespace StringUtil