|
|
|
Alissa Sabre made changes - 23/Dec/07 05:47 PM
Alissa Sabre made changes - 24/Dec/07 05:08 AM
lindenrobot made changes - 26/Dec/07 03:58 PM
James Linden made changes - 27/Dec/07 02:44 PM
[
Permlink
| « Hide
]
James Linden added a comment - 27/Dec/07 02:46 PM
Yes, this is a sign extension issue. Thank you so much for finding this and for narrowing the patch down.
Thank you for your help reproducing the problem and narrowing the patch. The LLURI portion of the patch is not quite right, it needs to be static_cast<S32>(static_cast<U8>(c)), otherwise some normal escaping does not work correctly.
I have changed llpaneldirfind.cpp to use LLURI::escape() and added a unit test for escaping the cedilla in Français.
James Linden made changes - 27/Dec/07 04:42 PM
The fix will appear in the next 1.18.6 release candidate.
I tested this issue on 1.19.0 RC1 viewer and found that it is fixed.
Thank you for the fix.
Alissa Sabre made changes - 09/Feb/08 06:02 AM
Sue Linden made changes - 13/Nov/08 11:47 AM
Sue Linden made changes - 13/Nov/08 06:50 PM
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||