Closed jxguan closed 8 years ago
OK, I've added support for the unicode characters. But the issue is that the ToLower would not work for foreign languages, unless we use a ToLowerSpecial, which requires specifying a language. But still I think it's better than nothing. A quick LGTM for the change?
Looks good to me, just one suggestion since there is a unicode.ToLower()
that works on runes.
Looks good to me, thanks!