<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><br></div><div>Thank you for the feedback, Konstantin and Darin.<br></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 1, 2018 at 1:52 AM Darin Adler <<a href="mailto:darin@apple.com">darin@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div><br></div><div>With a few seconds thought I am thinking that maybe UTF8Conversion.h or UTF8Transcoding.h are possible better ideas for new names. Neither is completely accurate. If we were going to add the word “helpers” than I would say UTF8Helpers.h, but I really don’t like those kinds of word in header names (“utilities”, “helpers”, “functions”, “classes”).</div></div></blockquote><div><br></div><div>Sounds good. I take UTF8Conversion.h.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div style="overflow-wrap: break-word;"><div></div><div>A separate issue once we rename: the header is also pretty old and crufty. Eventually we might want to remove or refine the functions in here. Not sure how widely they are used.</div></div></blockquote><div><br></div><div> There are 8 functions in the header.</div><div><br></div><div>Following 2 functions are used only in JavaScriptCore/runtime/JSGlobalObjectFunctions.cpp.</div><div><br></div><div>* UTF8SequenceLength</div><div>* decodeUTF8Sequence</div><div><br></div><div>Following 3 functions are used only in WTF/wtf/text/AtomicStringImpl.cpp</div><div><br></div><div>* calculateStringHashAndLengthFromUTF8MaskingTop8Bits</div><div>* equalUTF16WithUTF8</div><div>* equalLatin1WithUTF8</div><div><br></div><div>Following 3 functions are widely used</div><div><br></div><div>* convertUTF8ToUTF16</div><div>* convertLatin1ToUTF8</div><div>* convertUTF16ToUTF8</div><div><br></div><div>Any suggestions are welcome to improve.</div><div><br></div></div></div></div></div></div>