Piotr Caban : msvcrt: Improve tolower_l implementation.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 6a63e0e6bc5bcb7ee07e2d92b4b501865b0d2077
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=6a63e0e6bc5bcb7ee07e2d92b4b501865b0d2077

Author: Piotr Caban <piotr at codeweavers.com>
Date:   Thu Mar 13 14:12:41 2014 +0100

msvcrt: Improve tolower_l implementation.

---

 dlls/msvcrt/ctype.c |   39 ++++++++++++++++++++++-----------------
 1 file changed, 22 insertions(+), 17 deletions(-)

diff --git a/dlls/msvcrt/ctype.c b/dlls/msvcrt/ctype.c
index ca8b5fd..c486f8e 100644
--- a/dlls/msvcrt/ctype.c
+++ b/dlls/msvcrt/ctype.c
@@ -394,37 +394,42 @@ int CDECL MSVCRT__toupper(int c)
 int CDECL MSVCRT__tolower_l(int c, MSVCRT__locale_t locale)
 {
     MSVCRT_pthreadlocinfo locinfo;
+    unsigned char str[2], *p = str;
+    WCHAR wide, lower;
 
     if(!locale)
         locinfo = get_locinfo();
     else
         locinfo = locale->locinfo;
 
-    if(c < 256)
-        return locinfo->pclmap[(unsigned char)c];
+    if((unsigned)c < 256)
+        return locinfo->pclmap[c];
 
     if(locinfo->pctype[(c>>8)&255] & MSVCRT__LEADBYTE)
-    {
-        WCHAR wide, upper;
-        char str[2], *p = str;
         *p++ = (c>>8) & 255;
-        *p++ = c & 255;
-
-        if(!MultiByteToWideChar(locinfo->lc_codepage,
-                    MB_ERR_INVALID_CHARS, str, 2, &wide, 1))
-            return c;
+    else {
+        *MSVCRT__errno() = MSVCRT_EILSEQ;
+        str[1] = 0;
+    }
+    *p++ = c & 255;
 
-        upper = tolowerW(wide);
-        if(upper == wide)
-            return c;
+    if(!MultiByteToWideChar(locinfo->lc_codepage,
+                MB_ERR_INVALID_CHARS, (char*)str, p-str, &wide, 1))
+        return c;
 
-        WideCharToMultiByte(locinfo->lc_codepage, 0,
-                &upper, 1, str, 2, NULL, NULL);
+    lower = tolowerW(wide);
+    if(lower == wide)
+        return str[0] + (str[1]<<8);
 
+    switch(WideCharToMultiByte(locinfo->lc_codepage, 0,
+                &lower, 1, (char*)str, 2, NULL, NULL)) {
+    case 0:
+        return c;
+    case 1:
+        return str[0];
+    default:
         return str[0] + (str[1]<<8);
     }
-
-    return c;
 }
 
 /*********************************************************************