<div dir="auto"><div><div class="gmail_quote"><div dir="ltr">On Mon, Apr 30, 2018, 20:26 Ben Hamilton via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">benhamilton added a comment.<br>
<br>
Is it super tricky to at least extract the UTF-8 validating and sanitizing functions into their own header and source files within the `Support` library, if not a separate library?<br></blockquote></div></div><div dir="auto">It's not *technically* hard, but it will encourage wide use for general purposes, attract patches to extend its functionality, and confuse people as that functionality overlaps with ConvertUTF more and more.</div><div dir="auto"><br></div><div dir="auto">I think it's probably a good idea, but if I were reviewing it, I'd want to see that the author has a clear plan (that isn't MiscUnicodeUtils.h or rewriting ICU!)</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D46274" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D46274</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>