[cfe-commits] [PATCH] isLegalUTF8() giving the wrong answer

Eli Friedman eli.friedman at gmail.com
Tue Oct 30 15:29:06 PDT 2012


On Mon, Oct 29, 2012 at 11:33 PM, Seth Cantrell <seth.cantrell at gmail.com> wrote:
> for pre-commit review...
>
> I used Howard Hinnant's handy reference to check this modification: http://home.roadrunner.com/~hinnant/utf_summary.html
>
> (Also, according to _ http://www.unicode.org/forum/viewtopic.php?f=9&t=90 _ the original ConvertUTF.c source has known bugs, one of which is probably the one I fix here.)

Looks fine.

-Eli



More information about the cfe-commits mailing list