search

basvandijk / case-insensitive

Case insensitive string comparison
Other
25 stars 19 forks source link

Avoid using ByteString.Char8. #6

Closed kazu-yamamoto closed 12 years ago

kazu-yamamoto commented 12 years ago

I would like to remove overhead of conversion (Word8 -> Char then Char -> Word8).

basvandijk commented 12 years ago

Thanks for the patch!

I also lower case capitals in the Latin-1 Supplement so that all capitals that can occur in ByteStrings are lower cased. This ensures the behavior is identical to before the optimization.

If this is OK with you I will make a 0.4.0.3 release.

kazu-yamamoto commented 12 years ago

Ah, yes. Please include the Latin-1 stuff. Thanks.

basvandijk commented 12 years ago

Released as 0.4.0.3.

kazu-yamamoto commented 12 years ago

Thank you!