"bös".chars.reverse is "s̈ob" for decomposed string

The failing test isn't likely something that would be used as-is if this issue is fixed, but is rather meant to illustrate the issue.

Works fine in 2.2, so problem solved.

Oops, no. I forgot that the ticket was about decomposed strings, specifically, and they still fail:

>> dc = "bös".mb_chars.normalize(:kd).reverse.to_s
=> "s̈ob"

I'm not empowered to reopen tickets, I think, but someone else please do.

+1 - This is still a problem in 3.0. If you create an @string variable that equals "bös", then @string.chars.reverse has the desired result ("söb"), but @string.mb_chars.normalize(:kd).reverse.to_s outputs "s̈ob".

+1 indeed verified on master, unsuccessful to reproduce with other UTF8 strings (hebrew)

+1 verified on master, this also happens w/ the string "bés"

I've attached a patch that fixes this issue. It applies cleanly to both master and 2-3-stable.

+1 - applies cleanly to master and 2-3-stable and tests pass

+1 verified.

The patch applies and all tests pass.

+1 verified.

Patch applies to master.

(from [23780850237876cf81038534d8f59fa307af0b31]) Fix chars.reverse for multibyte decomposed strings

[#597 "bös".chars.reverse is "s̈ob" for decomposed string state:committed]

Signed-off-by: Jeremy Kemper jeremy@bitsweat.net
http://github.com/rails/rails/commit/23780850237876cf81038534d8f59f...