[webkit-reviews] review granted: [Bug 32857] RVCT (__ARMCC_VERSION < 400000) does not provide strcasecmp and strncasecmp : [Attachment 45376] Fix the style error.

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Kwang Yul Seo
<kwangyul.seo at gmail.com>'s request for review:
Bug 32857: RVCT (__ARMCC_VERSION < 400000) does not provide strcasecmp and
strncasecmp
https://bugs.webkit.org/show_bug.cgi?id=32857

Attachment 45376: Fix the style error.
https://bugs.webkit.org/attachment.cgi?id=45376&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
This patch is OK, but I'd prefer to see us eliminate strcasecmp and strncasecmp
from the code instead.

These compare non-ASCII characters without any processing, which is better for
our purposes, whereas strcasecmp and strncasecmp may handle non-ASCII
characters as letters based on the current POSIX locale setting.

It'd be good to provide an ASCII version of strcasecmp and strncasecmp and get
rid of the current uses of them.