April 2010 - Wine-commits - list.winehq.org

Joel Holdsworth : winhlp32: Replaced progam icon with a Tango compliant icon.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: 1aed5d33c9ce65895deff07cfb0a1ec76089fb41 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1aed5d33c9ce65895deff07cf… Author: Joel Holdsworth <joel(a)airwebreathe.org.uk> Date: Sun Apr 18 16:28:36 2010 +0100 winhlp32: Replaced progam icon with a Tango compliant icon. --- programs/winhlp32/winhelp.ico | Bin 15086 -> 25214 bytes programs/winhlp32/winhelp.svg | 546 +++++++++++++++++------------------------ 2 files changed, 220 insertions(+), 326 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=1aed5d33c9ce65895deff…

1 0

Hwang YunSong(황윤성) : wineboot: Update Korean translation.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: ebfedceaeb83db80d6d56622227d58e571b21096 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ebfedceaeb83db80d6d566222… Author: Hwang YunSong(황윤성) <hys545(a)dreamwiz.com> Date: Mon Apr 19 10:28:48 2010 +0900 wineboot: Update Korean translation. --- programs/wineboot/wineboot_Ko.rc | 16 +++++++++++++--- 1 files changed, 13 insertions(+), 3 deletions(-) diff --git a/programs/wineboot/wineboot_Ko.rc b/programs/wineboot/wineboot_Ko.rc index 28bcd4b..ebd295f 100644 --- a/programs/wineboot/wineboot_Ko.rc +++ b/programs/wineboot/wineboot_Ko.rc @@ -32,8 +32,18 @@ FONT 9, "MS Shell Dlg" BEGIN DEFPUSHBUTTON "��",IDCANCEL,129,49,50,15 PUSHBUTTON "��μ�� ",IDOK,51,49,71,15 - LTEXT " �� μ�� α� �� Ḧ �õ��, �� α׷�� ƹ�� ϴ�.", - IDC_STATIC,7,7,172,19 + LTEXT "�� μ�� α� �� Ḧ �õ��, �� α׷�� ƹ�� ϴ�.", + IDC_STATIC,7,5,172,26 LTEXT "�� μ�� ٸ� �� ͸� �Ҿ�� Դϴ�.", - IDC_STATIC,7,28,172,15 + IDC_STATIC,7,28,172,21 +END + +IDD_WAITDLG DIALOG 0, 0, 200, 50 +STYLE DS_MODALFRAME | DS_NOIDLEMSG | DS_CENTER | WS_POPUP | WS_DISABLED +CAPTION "Wine" +FONT 8, "MS Shell Dlg" +BEGIN + ICON "", IDC_WAITICON, 3, 3, 10, 10 + LTEXT "Wine �� %s �� Ʈ�Ǵ� ��Դϴ�, ��ø� ��ٷ��ֽñ� �ٶ��ϴ�...", + IDC_WAITTEXT, 40, 5, 150, 40, SS_NOPREFIX END

1 0

Andrew Nguyen : kernel32: Consistify allocation behavior with FormatMessageA/W.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: fcdd0d30400ee791ed4098bcf360eb928b01309b URL: http://source.winehq.org/git/wine.git/?a=commit;h=fcdd0d30400ee791ed4098bcf… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 18 09:09:21 2010 -0500 kernel32: Consistify allocation behavior with FormatMessageA/W. --- dlls/kernel32/format_msg.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/kernel32/format_msg.c b/dlls/kernel32/format_msg.c index 8c37434..db97300 100644 --- a/dlls/kernel32/format_msg.c +++ b/dlls/kernel32/format_msg.c @@ -752,8 +752,7 @@ DWORD WINAPI FormatMessageW( TRACE("-- %s\n",debugstr_w(target)); if (dwFlags & FORMAT_MESSAGE_ALLOCATE_BUFFER) { /* nSize is the MINIMUM size */ - DWORD len = strlenW(target) + 1; - *((LPVOID*)lpBuffer) = LocalAlloc(LMEM_ZEROINIT,len*sizeof(WCHAR)); + *((LPVOID*)lpBuffer) = LocalAlloc(LMEM_ZEROINIT, max(nSize, talloced)*sizeof(WCHAR)); strcpyW(*(LPWSTR*)lpBuffer, target); } else

1 0

Andrew Nguyen : kernel32: Separate format string parsing in FormatMessageW into a new function.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: df83469258e25d2ca8e8d4a5f04d3c2817c7785c URL: http://source.winehq.org/git/wine.git/?a=commit;h=df83469258e25d2ca8e8d4a5f… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 18 09:09:28 2010 -0500 kernel32: Separate format string parsing in FormatMessageW into a new function. --- dlls/kernel32/format_msg.c | 209 +++++++++++++++++++++++--------------------- 1 files changed, 111 insertions(+), 98 deletions(-) diff --git a/dlls/kernel32/format_msg.c b/dlls/kernel32/format_msg.c index db97300..0ba0229 100644 --- a/dlls/kernel32/format_msg.c +++ b/dlls/kernel32/format_msg.c @@ -404,6 +404,115 @@ static LPCWSTR format_insertW( int insert, LPCWSTR format, DWORD flags, return format; } +/********************************************************************** + * format_messageW (internal) + */ +static LPWSTR format_messageW( DWORD dwFlags, LPCWSTR fmtstr, struct format_args *format_args ) +{ + LPWSTR target,t; + DWORD talloced; + LPCWSTR f; + DWORD width = dwFlags & FORMAT_MESSAGE_MAX_WIDTH_MASK; + BOOL eos = FALSE; + WCHAR ch; + + target = t = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, 100 * sizeof(WCHAR) ); + talloced = 100; + +#define ADD_TO_T(c) do {\ + *t++=c;\ + if ((DWORD)(t-target) == talloced) {\ + target = HeapReAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,target,talloced*2*sizeof(WCHAR));\ + t = target+talloced;\ + talloced*=2;\ + } \ +} while (0) + + f = fmtstr; + if (dwFlags & FORMAT_MESSAGE_IGNORE_INSERTS) { + while (*f && !eos) + ADD_TO_T(*f++); + } + else { + while (*f && !eos) { + if (*f=='%') { + int insertnr; + WCHAR *str,*x; + + f++; + if (!*f) { + ADD_TO_T('%'); + continue; + } + + switch (*f) { + case '1':case '2':case '3':case '4':case '5': + case '6':case '7':case '8':case '9': + insertnr = *f-'0'; + switch (f[1]) { + case '0':case '1':case '2':case '3': + case '4':case '5':case '6':case '7': + case '8':case '9': + f++; + insertnr = insertnr*10 + *f-'0'; + f++; + break; + default: + f++; + break; + } + f = format_insertW( insertnr, f, dwFlags, format_args, &str ); + for (x = str; *x; x++) ADD_TO_T(*x); + HeapFree( GetProcessHeap(), 0, str ); + break; + case 'n': + ADD_TO_T('\r'); + ADD_TO_T('\n'); + f++; + break; + case '0': + eos = TRUE; + f++; + break; + default: + ADD_TO_T(*f++); + break; + } + } else { + ch = *f; + f++; + if (ch == '\r') { + if (*f == '\n') + f++; + if(width) + ADD_TO_T(' '); + else + { + ADD_TO_T('\r'); + ADD_TO_T('\n'); + } + } else { + if (ch == '\n') + { + if(width) + ADD_TO_T(' '); + else + { + ADD_TO_T('\r'); + ADD_TO_T('\n'); + } + } + else + ADD_TO_T(ch); + } + } + } + } + *t = '\0'; + + return target; +} +#undef ADD_TO_T /*********************************************************************** * FormatMessageA (KERNEL32.@) @@ -603,13 +712,10 @@ DWORD WINAPI FormatMessageW( { struct format_args format_args; DWORD ret = 0; - LPWSTR target,t; + LPWSTR target; DWORD talloced; LPWSTR from; - LPCWSTR f; DWORD width = dwFlags & FORMAT_MESSAGE_MAX_WIDTH_MASK; - BOOL eos = FALSE; - WCHAR ch; TRACE("(0x%x,%p,%d,0x%x,%p,%d,%p)\n", dwFlags,lpSource,dwMessageId,dwLanguageId,lpBuffer,nSize,args); @@ -652,102 +758,9 @@ DWORD WINAPI FormatMessageW( from = load_messageW( kernel32_handle, dwMessageId, dwLanguageId ); if (!from) return 0; } - target = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, 100 * sizeof(WCHAR) ); - t = target; - talloced= 100; - -#define ADD_TO_T(c) do {\ - *t++=c;\ - if ((DWORD)(t-target) == talloced) {\ - target = HeapReAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,target,talloced*2*sizeof(WCHAR));\ - t = target+talloced;\ - talloced*=2;\ - } \ -} while (0) - - if (from) { - f=from; - if (dwFlags & FORMAT_MESSAGE_IGNORE_INSERTS) { - while (*f && !eos) - ADD_TO_T(*f++); - } - else { - while (*f && !eos) { - if (*f=='%') { - int insertnr; - WCHAR *str,*x; - f++; - if (!*f) { - ADD_TO_T('%'); - continue; - } + target = format_messageW( dwFlags, from, &format_args ); - switch (*f) { - case '1':case '2':case '3':case '4':case '5': - case '6':case '7':case '8':case '9': - insertnr=*f-'0'; - switch (f[1]) { - case '0':case '1':case '2':case '3': - case '4':case '5':case '6':case '7': - case '8':case '9': - f++; - insertnr=insertnr*10+*f-'0'; - f++; - break; - default: - f++; - break; - } - f = format_insertW( insertnr, f, dwFlags, &format_args, &str ); - for (x = str; *x; x++) ADD_TO_T(*x); - HeapFree( GetProcessHeap(), 0, str ); - break; - case 'n': - ADD_TO_T('\r'); - ADD_TO_T('\n'); - f++; - break; - case '0': - eos = TRUE; - f++; - break; - default: - ADD_TO_T(*f++); - break; - } - } else { - ch = *f; - f++; - if (ch == '\r') { - if (*f == '\n') - f++; - if(width) - ADD_TO_T(' '); - else - { - ADD_TO_T('\r'); - ADD_TO_T('\n'); - } - } else { - if (ch == '\n') - { - if(width) - ADD_TO_T(' '); - else - { - ADD_TO_T('\r'); - ADD_TO_T('\n'); - } - } - else - ADD_TO_T(ch); - } - } - } - } - *t='\0'; - } talloced = strlenW(target)+1; TRACE("-- %s\n",debugstr_w(target)); if (dwFlags & FORMAT_MESSAGE_ALLOCATE_BUFFER) {

1 0

Andrew Nguyen : kernel32: Separate format string parsing in FormatMessageA into a new function.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: f199a83d476d27250f559e8dbb2d245181f8df93 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f199a83d476d27250f559e8db… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 18 09:09:45 2010 -0500 kernel32: Separate format string parsing in FormatMessageA into a new function. --- dlls/kernel32/format_msg.c | 208 +++++++++++++++++++++++-------------------- 1 files changed, 111 insertions(+), 97 deletions(-) diff --git a/dlls/kernel32/format_msg.c b/dlls/kernel32/format_msg.c index 0ba0229..1ee5bd4 100644 --- a/dlls/kernel32/format_msg.c +++ b/dlls/kernel32/format_msg.c @@ -405,6 +405,115 @@ static LPCWSTR format_insertW( int insert, LPCWSTR format, DWORD flags, } /********************************************************************** + * format_messageA (internal) + */ +static LPSTR format_messageA( DWORD dwFlags, LPCSTR fmtstr, struct format_args *format_args ) +{ + LPSTR target,t; + DWORD talloced; + LPCSTR f; + DWORD width = dwFlags & FORMAT_MESSAGE_MAX_WIDTH_MASK; + BOOL eos = FALSE; + CHAR ch; + + target = t = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, 100); + talloced = 100; + +#define ADD_TO_T(c) do { \ + *t++=c;\ + if ((DWORD)(t-target) == talloced) {\ + target = HeapReAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,target,talloced*2);\ + t = target+talloced;\ + talloced*=2;\ + }\ +} while (0) + + f = fmtstr; + if (dwFlags & FORMAT_MESSAGE_IGNORE_INSERTS) { + while (*f && !eos) + ADD_TO_T(*f++); + } + else { + while (*f && !eos) { + if (*f=='%') { + int insertnr; + char *str,*x; + + f++; + if (!*f) { + ADD_TO_T('%'); + continue; + } + switch (*f) { + case '1':case '2':case '3':case '4':case '5': + case '6':case '7':case '8':case '9': + insertnr = *f-'0'; + switch (f[1]) { + case '0':case '1':case '2':case '3': + case '4':case '5':case '6':case '7': + case '8':case '9': + f++; + insertnr = insertnr*10 + *f-'0'; + f++; + break; + default: + f++; + break; + } + f = format_insertA( insertnr, f, dwFlags, format_args, &str ); + for (x = str; *x; x++) ADD_TO_T(*x); + HeapFree( GetProcessHeap(), 0, str ); + break; + case 'n': + ADD_TO_T('\r'); + ADD_TO_T('\n'); + f++; + break; + case '0': + eos = TRUE; + f++; + break; + default: + ADD_TO_T(*f++); + break; + } + } else { + ch = *f; + f++; + if (ch == '\r') { + if (*f == '\n') + f++; + if(width) + ADD_TO_T(' '); + else + { + ADD_TO_T('\r'); + ADD_TO_T('\n'); + } + } else { + if (ch == '\n') + { + if(width) + ADD_TO_T(' '); + else + { + ADD_TO_T('\r'); + ADD_TO_T('\n'); + } + } + else + ADD_TO_T(ch); + } + } + } + } + *t = '\0'; + + return target; +} +#undef ADD_TO_T + +/********************************************************************** * format_messageW (internal) */ static LPWSTR format_messageW( DWORD dwFlags, LPCWSTR fmtstr, struct format_args *format_args ) @@ -529,13 +638,10 @@ DWORD WINAPI FormatMessageA( { struct format_args format_args; DWORD ret = 0; - LPSTR target,t; + LPSTR target; DWORD talloced; LPSTR from; - LPCSTR f; DWORD width = dwFlags & FORMAT_MESSAGE_MAX_WIDTH_MASK; - BOOL eos = FALSE; - CHAR ch; TRACE("(0x%x,%p,%d,0x%x,%p,%d,%p)\n", dwFlags,lpSource,dwMessageId,dwLanguageId,lpBuffer,nSize,args); @@ -578,101 +684,9 @@ DWORD WINAPI FormatMessageA( from = load_messageA( kernel32_handle, dwMessageId, dwLanguageId ); if (!from) return 0; } - target = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, 100); - t = target; - talloced= 100; -#define ADD_TO_T(c) do { \ - *t++=c;\ - if ((DWORD)(t-target) == talloced) {\ - target = HeapReAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,target,talloced*2);\ - t = target+talloced;\ - talloced*=2;\ - }\ -} while (0) - - if (from) { - f=from; - if (dwFlags & FORMAT_MESSAGE_IGNORE_INSERTS) { - while (*f && !eos) - ADD_TO_T(*f++); - } - else { - while (*f && !eos) { - if (*f=='%') { - int insertnr; - char *str,*x; + target = format_messageA( dwFlags, from, &format_args ); - f++; - if (!*f) { - ADD_TO_T('%'); - continue; - } - switch (*f) { - case '1':case '2':case '3':case '4':case '5': - case '6':case '7':case '8':case '9': - insertnr=*f-'0'; - switch (f[1]) { - case '0':case '1':case '2':case '3': - case '4':case '5':case '6':case '7': - case '8':case '9': - f++; - insertnr=insertnr*10+*f-'0'; - f++; - break; - default: - f++; - break; - } - f = format_insertA( insertnr, f, dwFlags, &format_args, &str ); - for (x = str; *x; x++) ADD_TO_T(*x); - HeapFree( GetProcessHeap(), 0, str ); - break; - case 'n': - ADD_TO_T('\r'); - ADD_TO_T('\n'); - f++; - break; - case '0': - eos = TRUE; - f++; - break; - default: - ADD_TO_T(*f++); - break; - } - } else { - ch = *f; - f++; - if (ch == '\r') { - if (*f == '\n') - f++; - if(width) - ADD_TO_T(' '); - else - { - ADD_TO_T('\r'); - ADD_TO_T('\n'); - } - } else { - if (ch == '\n') - { - if(width) - ADD_TO_T(' '); - else - { - ADD_TO_T('\r'); - ADD_TO_T('\n'); - } - } - else - ADD_TO_T(ch); - } - } - } - } - *t='\0'; - } talloced = strlen(target)+1; TRACE("-- %s\n",debugstr_a(target)); if (dwFlags & FORMAT_MESSAGE_ALLOCATE_BUFFER) {

1 0

Andrew Nguyen : kernel32: Transition FormatMessageA to the internal use of Unicode strings.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: 053d4385c9f929d44bcbb00090cd2d8ae475735c URL: http://source.winehq.org/git/wine.git/?a=commit;h=053d4385c9f929d44bcbb0009… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 18 09:09:53 2010 -0500 kernel32: Transition FormatMessageA to the internal use of Unicode strings. --- dlls/kernel32/format_msg.c | 134 +++++++++++++++++--------------------------- 1 files changed, 51 insertions(+), 83 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=053d4385c9f929d44bcbb…

1 0

Andrew Nguyen : kernel32: Unify the format_insert ANSI and Unicode helpers.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: 4c657b870f053c042d30be3f93c7c2fc510a3e55 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4c657b870f053c042d30be3f9… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 18 09:09:59 2010 -0500 kernel32: Unify the format_insert ANSI and Unicode helpers. --- dlls/kernel32/format_msg.c | 159 +++++++------------------------------------- 1 files changed, 24 insertions(+), 135 deletions(-) diff --git a/dlls/kernel32/format_msg.c b/dlls/kernel32/format_msg.c index 72cc55e..adb24f2 100644 --- a/dlls/kernel32/format_msg.c +++ b/dlls/kernel32/format_msg.c @@ -121,12 +121,12 @@ static ULONG_PTR get_arg( int nr, DWORD flags, struct format_args *args ) return args->args[nr - 1]; } - /********************************************************************** - * format_insertA (internal) + * format_insert (internal) */ -static LPCWSTR format_insertA( int insert, LPCWSTR format, DWORD flags, - struct format_args *args, LPWSTR *result ) +static LPCWSTR format_insert( BOOL unicode_caller, int insert, LPCWSTR format, + DWORD flags, struct format_args *args, + LPWSTR *result ) { static const WCHAR fmt_lu[] = {'%','l','u',0}; WCHAR *wstring = NULL, *p, fmt[256]; @@ -135,135 +135,20 @@ static LPCWSTR format_insertA( int insert, LPCWSTR format, DWORD flags, if (*format != '!') /* simple string */ { - char *str = (char *)get_arg( insert, flags, args ); - DWORD length = MultiByteToWideChar(CP_ACP, 0, str, -1, NULL, 0); - *result = HeapAlloc( GetProcessHeap(), 0, length * sizeof(WCHAR) ); - MultiByteToWideChar(CP_ACP, 0, str, -1, *result, length); - return format; - } - - format++; - p = fmt; - *p++ = '%'; - - while (*format == '0' || - *format == '+' || - *format == '-' || - *format == ' ' || - *format == '*' || - *format == '#') - { - if (*format == '*') + arg = get_arg( insert, flags, args ); + if (unicode_caller) { - p += sprintfW( p, fmt_lu, get_arg( insert, flags, args )); - insert = -1; - format++; - } - else *p++ = *format++; - } - while (isdigit(*format)) *p++ = *format++; - - if (*format == '.') - { - *p++ = *format++; - if (*format == '*') - { - p += sprintfW( p, fmt_lu, get_arg( insert, flags, args )); - insert = -1; - format++; - } - else - while (isdigit(*format)) *p++ = *format++; - } - - /* replicate MS bug: drop an argument when using va_list with width/precision */ - if (insert == -1 && args->list) args->last--; - arg = get_arg( insert, flags, args ); - - /* check for wide string format */ - if ((format[0] == 'l' && format[1] == 's') || - (format[0] == 'l' && format[1] == 'S') || - (format[0] == 'w' && format[1] == 's') || - (format[0] == 'S')) - { - *p++ = 's'; - } - /* check for wide character format */ - else if ((format[0] == 'l' && format[1] == 'c') || - (format[0] == 'l' && format[1] == 'C') || - (format[0] == 'w' && format[1] == 'c') || - (format[0] == 'C')) - { - *p++ = 'c'; - } - /* check for ascii string format */ - else if ((format[0] == 'h' && format[1] == 's') || - (format[0] == 'h' && format[1] == 'S') || - (format[0] == 's')) - { - DWORD len = MultiByteToWideChar( CP_ACP, 0, (char *)arg, -1, NULL, 0 ); - wstring = HeapAlloc( GetProcessHeap(), 0, len * sizeof(WCHAR) ); - MultiByteToWideChar( CP_ACP, 0, (char *)arg, -1, wstring, len ); - arg = (ULONG_PTR)wstring; - *p++ = 's'; - } - /* check for ascii character format */ - else if ((format[0] == 'h' && format[1] == 'c') || - (format[0] == 'h' && format[1] == 'C') || - (format[0] == 'c')) - { - char ch = arg; - wstring = HeapAlloc( GetProcessHeap(), 0, 2 * sizeof(WCHAR) ); - MultiByteToWideChar( CP_ACP, 0, &ch, 1, wstring, 1 ); - wstring[1] = 0; - arg = (ULONG_PTR)wstring; - *p++ = 's'; - } - /* FIXME: handle I64 etc. */ - else while (*format && *format != '!') *p++ = *format++; - - *p = 0; - size = 256; - for (;;) - { - WCHAR *ret = HeapAlloc( GetProcessHeap(), 0, size * sizeof(WCHAR)); - int needed = snprintfW( ret, size, fmt, arg ); - if (needed == -1 || needed >= size) - { - HeapFree( GetProcessHeap(), 0, ret ); - size = max( needed + 1, size * 2 ); + WCHAR *str = (WCHAR *)arg; + *result = HeapAlloc( GetProcessHeap(), 0, (strlenW(str) + 1) * sizeof(WCHAR) ); + strcpyW( *result, str ); } else { - *result = ret; - break; + char *str = (char *)get_arg( insert, flags, args ); + DWORD length = MultiByteToWideChar( CP_ACP, 0, str, -1, NULL, 0 ); + *result = HeapAlloc( GetProcessHeap(), 0, length * sizeof(WCHAR) ); + MultiByteToWideChar( CP_ACP, 0, str, -1, *result, length ); } - } - - while (*format && *format != '!') format++; - if (*format == '!') format++; - - HeapFree( GetProcessHeap(), 0, wstring ); - return format; -} - - -/********************************************************************** - * format_insertW (internal) - */ -static LPCWSTR format_insertW( int insert, LPCWSTR format, DWORD flags, - struct format_args *args, LPWSTR *result ) -{ - static const WCHAR fmt_lu[] = {'%','l','u',0}; - WCHAR *wstring = NULL, *p, fmt[256]; - ULONG_PTR arg; - int size; - - if (*format != '!') /* simple string */ - { - WCHAR *str = (WCHAR *)get_arg( insert, flags, args ); - *result = HeapAlloc( GetProcessHeap(), 0, (strlenW(str) + 1) * sizeof(WCHAR) ); - strcpyW( *result, str ); return format; } @@ -308,9 +193,10 @@ static LPCWSTR format_insertW( int insert, LPCWSTR format, DWORD flags, /* check for ascii string format */ if ((format[0] == 'h' && format[1] == 's') || (format[0] == 'h' && format[1] == 'S') || - (format[0] == 'S')) + (unicode_caller && format[0] == 'S') || + (!unicode_caller && format[0] == 's')) { - DWORD len = MultiByteToWideChar( CP_ACP, 0, (char *)arg, -1, /*FIXME*/ NULL, 0 ); + DWORD len = MultiByteToWideChar( CP_ACP, 0, (char *)arg, -1, NULL, 0 ); wstring = HeapAlloc( GetProcessHeap(), 0, len * sizeof(WCHAR) ); MultiByteToWideChar( CP_ACP, 0, (char *)arg, -1, wstring, len ); arg = (ULONG_PTR)wstring; @@ -319,7 +205,8 @@ static LPCWSTR format_insertW( int insert, LPCWSTR format, DWORD flags, /* check for ascii character format */ else if ((format[0] == 'h' && format[1] == 'c') || (format[0] == 'h' && format[1] == 'C') || - (format[0] == 'C')) + (unicode_caller && format[0] == 'C') || + (!unicode_caller && format[0] == 'c')) { char ch = arg; wstring = HeapAlloc( GetProcessHeap(), 0, 2 * sizeof(WCHAR) ); @@ -331,14 +218,16 @@ static LPCWSTR format_insertW( int insert, LPCWSTR format, DWORD flags, /* check for wide string format */ else if ((format[0] == 'l' && format[1] == 's') || (format[0] == 'l' && format[1] == 'S') || - (format[0] == 'w' && format[1] == 's')) + (format[0] == 'w' && format[1] == 's') || + (!unicode_caller && format[0] == 'S')) { *p++ = 's'; } /* check for wide character format */ else if ((format[0] == 'l' && format[1] == 'c') || (format[0] == 'l' && format[1] == 'C') || - (format[0] == 'w' && format[1] == 'c')) + (format[0] == 'w' && format[1] == 'c') || + (!unicode_caller && format[0] == 'C')) { *p++ = 'c'; } @@ -426,7 +315,7 @@ static LPWSTR format_messageA( DWORD dwFlags, LPCWSTR fmtstr, struct format_args f++; break; } - f = format_insertA( insertnr, f, dwFlags, format_args, &str ); + f = format_insert( FALSE, insertnr, f, dwFlags, format_args, &str ); for (x = str; *x; x++) ADD_TO_T(*x); HeapFree( GetProcessHeap(), 0, str ); break; @@ -536,7 +425,7 @@ static LPWSTR format_messageW( DWORD dwFlags, LPCWSTR fmtstr, struct format_args f++; break; } - f = format_insertW( insertnr, f, dwFlags, format_args, &str ); + f = format_insert( TRUE, insertnr, f, dwFlags, format_args, &str ); for (x = str; *x; x++) ADD_TO_T(*x); HeapFree( GetProcessHeap(), 0, str ); break;

1 0

Andrew Nguyen : kernel32: Unify the format_message ANSI and Unicode helpers .
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: 43436f64d858eec8b12b3427a95bf534093ef681 URL: http://source.winehq.org/git/wine.git/?a=commit;h=43436f64d858eec8b12b3427a… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 18 09:10:08 2010 -0500 kernel32: Unify the format_message ANSI and Unicode helpers. --- dlls/kernel32/format_msg.c | 120 ++----------------------------------------- 1 files changed, 6 insertions(+), 114 deletions(-) diff --git a/dlls/kernel32/format_msg.c b/dlls/kernel32/format_msg.c index adb24f2..a5426ae 100644 --- a/dlls/kernel32/format_msg.c +++ b/dlls/kernel32/format_msg.c @@ -260,118 +260,10 @@ static LPCWSTR format_insert( BOOL unicode_caller, int insert, LPCWSTR format, } /********************************************************************** - * format_messageA (internal) + * format_message (internal) */ -static LPWSTR format_messageA( DWORD dwFlags, LPCWSTR fmtstr, struct format_args *format_args ) -{ - LPWSTR target,t; - DWORD talloced; - LPCWSTR f; - DWORD width = dwFlags & FORMAT_MESSAGE_MAX_WIDTH_MASK; - BOOL eos = FALSE; - WCHAR ch; - - target = t = HeapAlloc( GetProcessHeap(), HEAP_ZERO_MEMORY, 100 * sizeof(WCHAR)); - talloced = 100; - -#define ADD_TO_T(c) do { \ - *t++=c;\ - if ((DWORD)(t-target) == talloced) {\ - target = HeapReAlloc(GetProcessHeap(),HEAP_ZERO_MEMORY,target,talloced*2*sizeof(WCHAR));\ - t = target+talloced;\ - talloced*=2;\ - }\ -} while (0) - - f = fmtstr; - if (dwFlags & FORMAT_MESSAGE_IGNORE_INSERTS) { - while (*f && !eos) - ADD_TO_T(*f++); - } - else { - while (*f && !eos) { - if (*f=='%') { - int insertnr; - WCHAR *str,*x; - - f++; - if (!*f) { - ADD_TO_T('%'); - continue; - } - switch (*f) { - case '1':case '2':case '3':case '4':case '5': - case '6':case '7':case '8':case '9': - insertnr = *f-'0'; - switch (f[1]) { - case '0':case '1':case '2':case '3': - case '4':case '5':case '6':case '7': - case '8':case '9': - f++; - insertnr = insertnr*10 + *f-'0'; - f++; - break; - default: - f++; - break; - } - f = format_insert( FALSE, insertnr, f, dwFlags, format_args, &str ); - for (x = str; *x; x++) ADD_TO_T(*x); - HeapFree( GetProcessHeap(), 0, str ); - break; - case 'n': - ADD_TO_T('\r'); - ADD_TO_T('\n'); - f++; - break; - case '0': - eos = TRUE; - f++; - break; - default: - ADD_TO_T(*f++); - break; - } - } else { - ch = *f; - f++; - if (ch == '\r') { - if (*f == '\n') - f++; - if(width) - ADD_TO_T(' '); - else - { - ADD_TO_T('\r'); - ADD_TO_T('\n'); - } - } else { - if (ch == '\n') - { - if(width) - ADD_TO_T(' '); - else - { - ADD_TO_T('\r'); - ADD_TO_T('\n'); - } - } - else - ADD_TO_T(ch); - } - } - } - } - *t = '\0'; - - return target; -} -#undef ADD_TO_T - -/********************************************************************** - * format_messageW (internal) - */ -static LPWSTR format_messageW( DWORD dwFlags, LPCWSTR fmtstr, struct format_args *format_args ) +static LPWSTR format_message( BOOL unicode_caller, DWORD dwFlags, LPCWSTR fmtstr, + struct format_args *format_args ) { LPWSTR target,t; DWORD talloced; @@ -425,7 +317,7 @@ static LPWSTR format_messageW( DWORD dwFlags, LPCWSTR fmtstr, struct format_args f++; break; } - f = format_insert( TRUE, insertnr, f, dwFlags, format_args, &str ); + f = format_insert( unicode_caller, insertnr, f, dwFlags, format_args, &str ); for (x = str; *x; x++) ADD_TO_T(*x); HeapFree( GetProcessHeap(), 0, str ); break; @@ -540,7 +432,7 @@ DWORD WINAPI FormatMessageA( if (!from) return 0; } - target = format_messageA( dwFlags, from, &format_args ); + target = format_message( FALSE, dwFlags, from, &format_args ); TRACE("-- %s\n", debugstr_w(target)); destlength = WideCharToMultiByte(CP_ACP, 0, target, -1, NULL, 0, NULL, NULL); @@ -630,7 +522,7 @@ DWORD WINAPI FormatMessageW( if (!from) return 0; } - target = format_messageW( dwFlags, from, &format_args ); + target = format_message( TRUE, dwFlags, from, &format_args ); talloced = strlenW(target)+1; TRACE("-- %s\n",debugstr_w(target));

1 0

Andrew Nguyen : kernel32: Rename the load_message helper to reflect its sole handling of Unicode strings .
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: 12efa6135cf60e4739935697bf4791d0377b3d3d URL: http://source.winehq.org/git/wine.git/?a=commit;h=12efa6135cf60e4739935697b… Author: Andrew Nguyen <anguyen(a)codeweavers.com> Date: Sun Apr 18 09:10:19 2010 -0500 kernel32: Rename the load_message helper to reflect its sole handling of Unicode strings. --- dlls/kernel32/format_msg.c | 13 ++++++------- 1 files changed, 6 insertions(+), 7 deletions(-) diff --git a/dlls/kernel32/format_msg.c b/dlls/kernel32/format_msg.c index a5426ae..4dbaf36 100644 --- a/dlls/kernel32/format_msg.c +++ b/dlls/kernel32/format_msg.c @@ -72,9 +72,9 @@ static const WCHAR PCNTFMTWSTR[] = { '%','%','%','s',0 }; static const WCHAR FMTWSTR[] = { '%','s',0 }; /********************************************************************** - * load_messageW (internal) + * load_message (internal) */ -static LPWSTR load_messageW( HMODULE module, UINT id, WORD lang ) +static LPWSTR load_message( HMODULE module, UINT id, WORD lang ) { const MESSAGE_RESOURCE_ENTRY *mre; WCHAR *buffer; @@ -426,9 +426,9 @@ DWORD WINAPI FormatMessageA( } else { if (dwFlags & FORMAT_MESSAGE_FROM_HMODULE) - from = load_messageW( (HMODULE)lpSource, dwMessageId, dwLanguageId ); + from = load_message( (HMODULE)lpSource, dwMessageId, dwLanguageId ); if (!from && (dwFlags & FORMAT_MESSAGE_FROM_SYSTEM)) - from = load_messageW( kernel32_handle, dwMessageId, dwLanguageId ); + from = load_message( kernel32_handle, dwMessageId, dwLanguageId ); if (!from) return 0; } @@ -514,11 +514,10 @@ DWORD WINAPI FormatMessageW( strcpyW( from, lpSource ); } else { - from = NULL; if (dwFlags & FORMAT_MESSAGE_FROM_HMODULE) - from = load_messageW( (HMODULE)lpSource, dwMessageId, dwLanguageId ); + from = load_message( (HMODULE)lpSource, dwMessageId, dwLanguageId ); if (!from && (dwFlags & FORMAT_MESSAGE_FROM_SYSTEM)) - from = load_messageW( kernel32_handle, dwMessageId, dwLanguageId ); + from = load_message( kernel32_handle, dwMessageId, dwLanguageId ); if (!from) return 0; }

1 0

Nikolay Sivov : shell32: Load shell folder column names as A/ W strings depending on 9x/NT selector.
by Alexandre Julliard 19 Apr '10

19 Apr '10

Module: wine Branch: master Commit: 1b31b377b6f4efa9d707541f3c0903e07993fda2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1b31b377b6f4efa9d707541f3… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Sun Apr 18 18:48:58 2010 +0400 shell32: Load shell folder column names as A/W strings depending on 9x/NT selector. --- dlls/shell32/shlfolder.c | 16 ++++++++- dlls/shell32/shlview.c | 60 +++++++++++++++++----------------- dlls/shell32/tests/shfldr_special.c | 14 +++++++- 3 files changed, 57 insertions(+), 33 deletions(-) diff --git a/dlls/shell32/shlfolder.c b/dlls/shell32/shlfolder.c index 99773b3..230d7c6 100644 --- a/dlls/shell32/shlfolder.c +++ b/dlls/shell32/shlfolder.c @@ -548,8 +548,20 @@ HRESULT SHELL32_GetColumnDetails(const shvheader *data, int column, SHELLDETAILS { details->fmt = data[column].fmt; details->cxChar = data[column].cxChar; - details->str.uType = STRRET_CSTR; - LoadStringA (shell32_hInstance, data[column].colnameid, details->str.u.cStr, MAX_PATH); + + if (SHELL_OsIsUnicode()) + { + details->str.u.pOleStr = CoTaskMemAlloc(MAX_PATH * sizeof(WCHAR)); + if (!details->str.u.pOleStr) return E_OUTOFMEMORY; + + details->str.uType = STRRET_WSTR; + LoadStringW(shell32_hInstance, data[column].colnameid, details->str.u.pOleStr, MAX_PATH); + } + else + { + details->str.uType = STRRET_CSTR; + LoadStringA(shell32_hInstance, data[column].colnameid, details->str.u.cStr, MAX_PATH); + } return S_OK; } diff --git a/dlls/shell32/shlview.c b/dlls/shell32/shlview.c index 653e5f3..938ec13 100644 --- a/dlls/shell32/shlview.c +++ b/dlls/shell32/shlview.c @@ -387,42 +387,44 @@ static BOOL ShellView_CreateList (IShellViewImpl * This) * * - adds all needed columns to the shellview */ -static BOOL ShellView_InitList(IShellViewImpl * This) +static void ShellView_InitList(IShellViewImpl *This) { - LVCOLUMNW lvColumn; - SHELLDETAILS sd; - int i; - WCHAR szTemp[50]; + LVCOLUMNW lvColumn; + SHELLDETAILS sd; + WCHAR nameW[50]; - TRACE("%p\n",This); + TRACE("(%p)\n", This); - SendMessageW(This->hWndList, LVM_DELETEALLITEMS, 0, 0); + SendMessageW(This->hWndList, LVM_DELETEALLITEMS, 0, 0); - lvColumn.mask = LVCF_FMT | LVCF_WIDTH | LVCF_TEXT; - lvColumn.pszText = szTemp; + lvColumn.mask = LVCF_FMT | LVCF_WIDTH | LVCF_TEXT; + lvColumn.pszText = nameW; + + if (This->pSF2Parent) + { + HRESULT hr; + INT i; + + for (i = 0; 1; i++) + { + hr = IShellFolder2_GetDetailsOf(This->pSF2Parent, NULL, i, &sd); + if (FAILED(hr)) break; - if (This->pSF2Parent) - { - for (i=0; 1; i++) - { - if (FAILED(IShellFolder2_GetDetailsOf(This->pSF2Parent, NULL, i, &sd))) - break; lvColumn.fmt = sd.fmt; lvColumn.cx = sd.cxChar*8; /* chars->pixel */ - StrRetToStrNW( szTemp, 50, &sd.str, NULL); + StrRetToStrNW(nameW, sizeof(nameW)/sizeof(WCHAR), &sd.str, NULL); SendMessageW(This->hWndList, LVM_INSERTCOLUMNW, i, (LPARAM) &lvColumn); - } - } - else - { - FIXME("no SF2\n"); - } - - SendMessageW(This->hWndList, LVM_SETIMAGELIST, LVSIL_SMALL, (LPARAM)ShellSmallIconList); - SendMessageW(This->hWndList, LVM_SETIMAGELIST, LVSIL_NORMAL, (LPARAM)ShellBigIconList); + } + } + else + { + FIXME("no SF2\n"); + } - return TRUE; + SendMessageW(This->hWndList, LVM_SETIMAGELIST, LVSIL_SMALL, (LPARAM)ShellSmallIconList); + SendMessageW(This->hWndList, LVM_SETIMAGELIST, LVSIL_NORMAL, (LPARAM)ShellBigIconList); } + /********************************************************** * ShellView_CompareItems() * @@ -688,10 +690,8 @@ static LRESULT ShellView_OnCreate(IShellViewImpl *This) if (ShellView_CreateList(This)) { - if (ShellView_InitList(This)) - { - ShellView_FillList(This); - } + ShellView_InitList(This); + ShellView_FillList(This); } hr = IShellView2_QueryInterface(iface, &IID_IDropTarget, (LPVOID*)&pdt); diff --git a/dlls/shell32/tests/shfldr_special.c b/dlls/shell32/tests/shfldr_special.c index 3320130..7be2bdf 100644 --- a/dlls/shell32/tests/shfldr_special.c +++ b/dlls/shell32/tests/shfldr_special.c @@ -33,6 +33,11 @@ #include "wine/test.h" +static inline BOOL SHELL_OsIsUnicode(void) +{ + return !(GetVersion() & 0x80000000); +} + /* Tests for My Network Places */ static void test_parse_for_entire_network(void) { @@ -129,8 +134,8 @@ static void test_printers_folder(void) SHELLDETAILS details; SHCOLSTATEF state; LPITEMIDLIST pidl1, pidl2; - INT i; HRESULT hr; + INT i; CoInitialize( NULL ); @@ -166,6 +171,9 @@ if (0) ok(hr == S_OK, "got 0x%08x\n", hr); /* all columns are left-aligned */ ok(details.fmt == LVCFMT_LEFT, "got 0x%x\n", details.fmt); + /* can't be on w9x at this point, IShellFolder2 unsupported there, + check present for running Wine with w9x setup */ + if (SHELL_OsIsUnicode()) SHFree(details.str.u.pOleStr); hr = IShellFolder2_GetDefaultColumnState(folder, i, &state); ok(hr == S_OK, "got 0x%08x\n", hr); @@ -176,6 +184,10 @@ if (0) ok(state == (SHCOLSTATE_TYPE_STR | SHCOLSTATE_ONBYDEFAULT), "got 0x%x\n", state); } + hr = IShellFolder2_GetDetailsOf(folder, NULL, 0, &details); + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(details.str.uType == STRRET_WSTR, "got %d\n", details.str.uType); + /* default pidl */ hr = IShellFolder2_QueryInterface(folder, &IID_IPersistFolder2, (void**)&pf); ok(hr == S_OK, "got 0x%08x\n", hr);

1 0