May 2007 - Wine-commits - list.winehq.org

James Hawkins : msi: Create the destination directory if it doesn' t exist when duplicating files.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: 1aa0082558050daed4852ad29cc0a12c531a35e6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1aa0082558050daed4852ad29… Author: James Hawkins <truiken(a)gmail.com> Date: Tue May 29 10:09:44 2007 -0700 msi: Create the destination directory if it doesn't exist when duplicating files. --- dlls/msi/files.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/msi/files.c b/dlls/msi/files.c index 9717537..f807069 100644 --- a/dlls/msi/files.c +++ b/dlls/msi/files.c @@ -899,6 +899,8 @@ static UINT ITERATE_DuplicateFiles(MSIRECORD *row, LPVOID param) TRACE("Duplicating file %s to %s\n",debugstr_w(file->TargetPath), debugstr_w(dest)); + CreateDirectoryW(dest_path, NULL); + if (strcmpW(file->TargetPath,dest)) rc = !CopyFileW(file->TargetPath,dest,TRUE); else

1 0

Misha Koshelev : shell32/tests: Remove duplicate code.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: c7aa087504654cf2d3538bc970f3783bbab3a034 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c7aa087504654cf2d3538bc97… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Sun May 27 20:01:13 2007 -0500 shell32/tests: Remove duplicate code. --- dlls/shell32/tests/shlexec.c | 44 ++++++----------------------------------- 1 files changed, 7 insertions(+), 37 deletions(-) diff --git a/dlls/shell32/tests/shlexec.c b/dlls/shell32/tests/shlexec.c index 2b19a76..8894364 100644 --- a/dlls/shell32/tests/shlexec.c +++ b/dlls/shell32/tests/shlexec.c @@ -233,43 +233,6 @@ static void delete_test_association(const char* extension) SHDeleteKey(HKEY_CLASSES_ROOT, extension); } -static void create_test_verb(const char* extension, const char* verb, - int rawcmd, const char* cmdtail) -{ - HKEY hkey_shell, hkey_verb, hkey_cmd; - char shell[MAX_PATH]; - char* cmd; - LONG rc; - - sprintf(shell, "shlexec%s\\shell", extension); - rc=RegOpenKeyEx(HKEY_CLASSES_ROOT, shell, 0, - KEY_CREATE_SUB_KEY, &hkey_shell); - assert(rc==ERROR_SUCCESS); - rc=RegCreateKeyEx(hkey_shell, verb, 0, NULL, 0, KEY_CREATE_SUB_KEY, - NULL, &hkey_verb, NULL); - assert(rc==ERROR_SUCCESS); - rc=RegCreateKeyEx(hkey_verb, "command", 0, NULL, 0, KEY_SET_VALUE, - NULL, &hkey_cmd, NULL); - assert(rc==ERROR_SUCCESS); - - if (rawcmd) - { - rc=RegSetValueEx(hkey_cmd, NULL, 0, REG_SZ, (LPBYTE)cmdtail, strlen(cmdtail)+1); - } - else - { - cmd=malloc(strlen(argv0)+10+strlen(child_file)+2+strlen(cmdtail)+1); - sprintf(cmd,"%s shlexec \"%s\" %s", argv0, child_file, cmdtail); - rc=RegSetValueEx(hkey_cmd, NULL, 0, REG_SZ, (LPBYTE)cmd, strlen(cmd)+1); - assert(rc==ERROR_SUCCESS); - free(cmd); - } - - CloseHandle(hkey_shell); - CloseHandle(hkey_verb); - CloseHandle(hkey_cmd); -} - static void create_test_verb_dde(const char* extension, const char* verb, int rawcmd, const char* cmdtail, const char *ddeexec, const char *application, const char *topic, @@ -352,6 +315,13 @@ static void create_test_verb_dde(const char* extension, const char* verb, CloseHandle(hkey_cmd); } +static void create_test_verb(const char* extension, const char* verb, + int rawcmd, const char* cmdtail) +{ + create_test_verb_dde(extension, verb, rawcmd, cmdtail, NULL, NULL, + NULL, NULL); +} + /*** * * Functions to check that the child process was started just right

1 0

Misha Koshelev : shell32/tests: Add tests for default DDE application name.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: c8829f5645af7647397a113e957f8a8f40466b7a URL: http://source.winehq.org/git/wine.git/?a=commit;h=c8829f5645af7647397a113e9… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Sun May 27 20:01:06 2007 -0500 shell32/tests: Add tests for default DDE application name. --- dlls/shell32/tests/shlexec.c | 164 +++++++++++++++++++++++++++++++++++++++--- 1 files changed, 154 insertions(+), 10 deletions(-) diff --git a/dlls/shell32/tests/shlexec.c b/dlls/shell32/tests/shlexec.c index eabd25c..37f736d 100644 --- a/dlls/shell32/tests/shlexec.c +++ b/dlls/shell32/tests/shlexec.c @@ -567,6 +567,8 @@ static const char* testfiles[]= "%s\\masked file.shlexec", "%s\\masked", "%s\\test file.sde", + "%s\\test file.exe", + "%s\\test2.exe", NULL }; @@ -1102,8 +1104,9 @@ static dde_tests_t dde_tests[] = {NULL, NULL, NULL, NULL, 0x0, 0} }; +static DWORD ddeInst; static HSZ hszTopic; -static char ddeExec[MAX_PATH]; +static char ddeExec[MAX_PATH], ddeApplication[MAX_PATH]; static BOOL denyNextConnection; static HDDEDATA CALLBACK ddeCb(UINT uType, UINT uFmt, HCONV hConv, @@ -1124,7 +1127,11 @@ static HDDEDATA CALLBACK ddeCb(UINT uType, UINT uFmt, HCONV hConv, if (denyNextConnection) denyNextConnection = FALSE; else + { + size = DdeQueryString(ddeInst, hsz2, ddeApplication, MAX_PATH, CP_WINANSI); + assert(size < MAX_PATH); return (HDDEDATA)TRUE; + } } return (HDDEDATA)FALSE; @@ -1167,12 +1174,12 @@ static void test_dde(void) dde_thread_info_t info = { filename, GetCurrentThreadId() }; const dde_tests_t* test; char params[1024]; - DWORD inst = 0; HANDLE hThread; MSG msg; int rc; - rc = DdeInitializeA(&inst, ddeCb, CBF_SKIP_ALLNOTIFICATIONS | CBF_FAIL_ADVISES | + ddeInst = 0; + rc = DdeInitializeA(&ddeInst, ddeCb, CBF_SKIP_ALLNOTIFICATIONS | CBF_FAIL_ADVISES | CBF_FAIL_POKES | CBF_FAIL_REQUESTS, 0L); assert(rc == DMLERR_NO_ERROR); @@ -1188,12 +1195,12 @@ static void test_dde(void) create_test_association(".sde"); create_test_verb_dde(".sde", "Open", 0, test->command, test->ddeexec, test->application, test->topic, test->ifexec); - hszApplication = DdeCreateStringHandleA(inst, test->application ? + hszApplication = DdeCreateStringHandleA(ddeInst, test->application ? test->application : defApplication, CP_WINANSI); - hszTopic = DdeCreateStringHandleA(inst, test->topic ? test->topic : SZDDESYS_TOPIC, + hszTopic = DdeCreateStringHandleA(ddeInst, test->topic ? test->topic : SZDDESYS_TOPIC, CP_WINANSI); assert(hszApplication && hszTopic); - assert(DdeNameService(inst, hszApplication, 0L, DNS_REGISTER)); + assert(DdeNameService(ddeInst, hszApplication, 0L, DNS_REGISTER)); denyNextConnection = TRUE; ddeExec[0] = 0; @@ -1256,14 +1263,150 @@ static void test_dde(void) } } - assert(DdeNameService(inst, hszApplication, 0L, DNS_UNREGISTER)); - assert(DdeFreeStringHandle(inst, hszTopic)); - assert(DdeFreeStringHandle(inst, hszApplication)); + assert(DdeNameService(ddeInst, hszApplication, 0L, DNS_UNREGISTER)); + assert(DdeFreeStringHandle(ddeInst, hszTopic)); + assert(DdeFreeStringHandle(ddeInst, hszApplication)); + delete_test_association(".sde"); + test++; + } + + assert(DdeUninitialize(ddeInst)); +} + +typedef struct +{ + const char* command; + const char* expectedDdeApplication; + int todo; + int rc; +} dde_default_app_tests_t; + +static dde_default_app_tests_t dde_default_app_tests[] = +{ + /* Test unquoted existing filename with a space */ + {"%s\\test file.exe", "test file", 0x1, 33}, + {"%s\\test file.exe param", "test file", 0x1, 33}, + + /* Test quoted existing filename with a space */ + {"\"%s\\test file.exe\"", "test file", 0x1, 33}, + {"\"%s\\test file.exe\" param", "test file", 0x1, 33}, + + /* Test unquoted filename with a space that doesn't exist, but + * test2.exe does */ + {"%s\\test2 file.exe", "test2", 0x1, 33}, + {"%s\\test2 file.exe param", "test2", 0x1, 33}, + + /* Test quoted filename with a space that does not exist */ + {"\"%s\\test2 file.exe\"", "", 0x1, 5}, + {"\"%s\\test2 file.exe\" param", "", 0x1, 5}, + + /* Test filename supplied without the extension */ + {"%s\\test2", "test2", 0x1, 33}, + {"%s\\test2 param", "test2", 0x1, 33}, + + /* Test an unquoted non-existent filename */ + {"%s\\notexist.exe", "", 0x1, 5}, + {"%s\\notexist.exe param", "", 0x1, 5}, + + /* Test an application that will be found on the path */ + {"cmd", "cmd", 0x1, 33}, + {"cmd param", "cmd", 0x1, 33}, + + /* Test an application that will not be found on the path */ + {"xyzwxyzwxyz", "", 0x1, 5}, + {"xyzwxyzwxyz param", "", 0x1, 5}, + + {NULL, NULL, 0, 0} +}; + +static void test_dde_default_app(void) +{ + char filename[MAX_PATH]; + HSZ hszApplication; + dde_thread_info_t info = { filename, GetCurrentThreadId() }; + const dde_default_app_tests_t* test; + char params[1024]; + HANDLE hThread; + MSG msg; + int rc; + + ddeInst = 0; + rc = DdeInitializeA(&ddeInst, ddeCb, CBF_SKIP_ALLNOTIFICATIONS | CBF_FAIL_ADVISES | + CBF_FAIL_POKES | CBF_FAIL_REQUESTS, 0L); + assert(rc == DMLERR_NO_ERROR); + + sprintf(filename, "%s\\test file.sde", tmpdir); + + /* It is strictly not necessary to register an application name here, but wine's + * DdeNameService implementation complains if 0L is passed instead of + * hszApplication with DNS_FILTEROFF */ + hszApplication = DdeCreateStringHandleA(ddeInst, "shlexec", CP_WINANSI); + hszTopic = DdeCreateStringHandleA(ddeInst, "shlexec", CP_WINANSI); + assert(hszApplication && hszTopic); + assert(DdeNameService(ddeInst, hszApplication, 0L, DNS_REGISTER | DNS_FILTEROFF)); + + test = dde_default_app_tests; + while (test->command) + { + create_test_association(".sde"); + sprintf(params, test->command, tmpdir); + create_test_verb_dde(".sde", "Open", 1, params, "[test]", NULL, + "shlexec", NULL); + denyNextConnection = FALSE; + ddeApplication[0] = 0; + + /* No application will be run as we will respond to the first DDE event, + * so don't wait for it */ + SetEvent(hEvent); + + hThread = CreateThread(NULL, 0, ddeThread, (LPVOID)&info, 0, NULL); + assert(hThread); + while (GetMessage(&msg, NULL, 0, 0)) + { + /* Need a message loop for DDE server */ + TranslateMessage(&msg); + DispatchMessage(&msg); + } + rc = WaitForSingleObject(hThread, 5000); + assert(rc == WAIT_OBJECT_0); + + GetExitCodeThread(hThread, (DWORD *)&rc); + if (rc > 32) + rc=33; + if ((test->todo & 0x1)==0) + { + ok(rc==test->rc, "%s failed: rc=%d err=%d\n", shell_call, + rc, GetLastError()); + } + else todo_wine + { + ok(rc==test->rc, "%s failed: rc=%d err=%d\n", shell_call, + rc, GetLastError()); + } + if (rc == 33) + { + if ((test->todo & 0x2)==0) + { + ok(!strcmp(ddeApplication, test->expectedDdeApplication), + "Expected application '%s', got '%s'\n", + test->expectedDdeApplication, ddeApplication); + } + else todo_wine + { + ok(!strcmp(ddeApplication, test->expectedDdeApplication), + "Expected application '%s', got '%s'\n", + test->expectedDdeApplication, ddeApplication); + } + } + delete_test_association(".sde"); test++; } - assert(DdeUninitialize(inst)); + assert(DdeNameService(ddeInst, hszApplication, 0L, DNS_UNREGISTER)); + assert(DdeFreeStringHandle(ddeInst, hszTopic)); + assert(DdeFreeStringHandle(ddeInst, hszApplication)); + assert(DdeUninitialize(ddeInst)); } static void init_test(void) @@ -1409,6 +1552,7 @@ START_TEST(shlexec) test_exes(); test_exes_long(); test_dde(); + test_dde_default_app(); cleanup_test(); }

1 0

Misha Koshelev : shell32: Handle default DDE application names.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: efb92561e69569991dfdbd8c119d57a74d77e219 URL: http://source.winehq.org/git/wine.git/?a=commit;h=efb92561e69569991dfdbd8c1… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Sun May 27 20:01:10 2007 -0500 shell32: Handle default DDE application names. --- dlls/shell32/shlexec.c | 47 ++++++++++++++++++++++++++++++++++++++++- dlls/shell32/tests/shlexec.c | 34 +++++++++++++++--------------- 2 files changed, 62 insertions(+), 19 deletions(-) diff --git a/dlls/shell32/shlexec.c b/dlls/shell32/shlexec.c index 0d2b364..edb65ea 100644 --- a/dlls/shell32/shlexec.c +++ b/dlls/shell32/shlexec.c @@ -780,8 +780,51 @@ static unsigned dde_connect(WCHAR* key, const WCHAR* start, WCHAR* ddeexec, applen = sizeof(app); if (RegQueryValueW(HKEY_CLASSES_ROOT, key, app, &applen) != ERROR_SUCCESS) { - FIXME("default app name NIY %s\n", debugstr_w(key)); - return 2; + WCHAR command[1024], fullpath[MAX_PATH]; + static const WCHAR wSo[] = { '.','s','o',0 }; + int sizeSo = sizeof(wSo)/sizeof(WCHAR); + LPWSTR ptr = NULL; + DWORD ret = 0; + + /* Get application command from start string and find filename of application */ + if (*start == '"') + { + strcpyW(command, start+1); + if ((ptr = strchrW(command, '"'))) + *ptr = 0; + ret = SearchPathW(NULL, command, wszExe, sizeof(fullpath)/sizeof(WCHAR), fullpath, &ptr); + } + else + { + LPWSTR p,space; + for (p=(LPWSTR)start; (space=strchrW(p, ' ')); p=space+1) + { + int idx = space-start; + memcpy(command, start, idx*sizeof(WCHAR)); + command[idx] = '\0'; + if ((ret = SearchPathW(NULL, command, wszExe, sizeof(fullpath)/sizeof(WCHAR), fullpath, &ptr))) + break; + } + if (!ret) + ret = SearchPathW(NULL, start, wszExe, sizeof(fullpath)/sizeof(WCHAR), fullpath, &ptr); + } + + if (!ret) + { + ERR("Unable to find application path for command %s\n", debugstr_w(start)); + return ERROR_ACCESS_DENIED; + } + strcpyW(app, ptr); + + /* Remove extensions (including .so) */ + ptr = app + strlenW(app) - (sizeSo-1); + if (strlenW(app) >= sizeSo && + !strcmpW(ptr, wSo)) + *ptr = 0; + + ptr = strrchrW(app, '.'); + assert(ptr); + *ptr = 0; } strcpyW(endkey, wTopic); diff --git a/dlls/shell32/tests/shlexec.c b/dlls/shell32/tests/shlexec.c index 37f736d..2b19a76 100644 --- a/dlls/shell32/tests/shlexec.c +++ b/dlls/shell32/tests/shlexec.c @@ -1099,7 +1099,7 @@ static dde_tests_t dde_tests[] = {"", "[open(\"%1\")]", "shlexec", NULL, NULL, FALSE, "[open(\"%s\")]", 0x0, 33}, /* Test default DDE application */ - {"", "[open(\"%1\")]", NULL, "dde", NULL, FALSE, "[open(\"%s\")]", 0x1, 33}, + {"", "[open(\"%1\")]", NULL, "dde", NULL, FALSE, "[open(\"%s\")]", 0x0, 33}, {NULL, NULL, NULL, NULL, 0x0, 0} }; @@ -1284,37 +1284,37 @@ typedef struct static dde_default_app_tests_t dde_default_app_tests[] = { /* Test unquoted existing filename with a space */ - {"%s\\test file.exe", "test file", 0x1, 33}, - {"%s\\test file.exe param", "test file", 0x1, 33}, + {"%s\\test file.exe", "test file", 0x0, 33}, + {"%s\\test file.exe param", "test file", 0x0, 33}, /* Test quoted existing filename with a space */ - {"\"%s\\test file.exe\"", "test file", 0x1, 33}, - {"\"%s\\test file.exe\" param", "test file", 0x1, 33}, + {"\"%s\\test file.exe\"", "test file", 0x0, 33}, + {"\"%s\\test file.exe\" param", "test file", 0x0, 33}, /* Test unquoted filename with a space that doesn't exist, but * test2.exe does */ - {"%s\\test2 file.exe", "test2", 0x1, 33}, - {"%s\\test2 file.exe param", "test2", 0x1, 33}, + {"%s\\test2 file.exe", "test2", 0x0, 33}, + {"%s\\test2 file.exe param", "test2", 0x0, 33}, /* Test quoted filename with a space that does not exist */ - {"\"%s\\test2 file.exe\"", "", 0x1, 5}, - {"\"%s\\test2 file.exe\" param", "", 0x1, 5}, + {"\"%s\\test2 file.exe\"", "", 0x0, 5}, + {"\"%s\\test2 file.exe\" param", "", 0x0, 5}, /* Test filename supplied without the extension */ - {"%s\\test2", "test2", 0x1, 33}, - {"%s\\test2 param", "test2", 0x1, 33}, + {"%s\\test2", "test2", 0x0, 33}, + {"%s\\test2 param", "test2", 0x0, 33}, /* Test an unquoted non-existent filename */ - {"%s\\notexist.exe", "", 0x1, 5}, - {"%s\\notexist.exe param", "", 0x1, 5}, + {"%s\\notexist.exe", "", 0x0, 5}, + {"%s\\notexist.exe param", "", 0x0, 5}, /* Test an application that will be found on the path */ - {"cmd", "cmd", 0x1, 33}, - {"cmd param", "cmd", 0x1, 33}, + {"cmd", "cmd", 0x0, 33}, + {"cmd param", "cmd", 0x0, 33}, /* Test an application that will not be found on the path */ - {"xyzwxyzwxyz", "", 0x1, 5}, - {"xyzwxyzwxyz param", "", 0x1, 5}, + {"xyzwxyzwxyz", "", 0x0, 5}, + {"xyzwxyzwxyz param", "", 0x0, 5}, {NULL, NULL, 0, 0} };

1 0

Misha Koshelev : shell32: Fix execute_from_key to conform to native behavior.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: cfa87660a7cbc3158bfccacb4563f853ca0f2d97 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cfa87660a7cbc3158bfccacb4… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Sun May 27 20:01:01 2007 -0500 shell32: Fix execute_from_key to conform to native behavior. --- dlls/shell32/shlexec.c | 63 ++++++++++++++++-------------------------- dlls/shell32/tests/shlexec.c | 12 ++++---- 2 files changed, 30 insertions(+), 45 deletions(-) diff --git a/dlls/shell32/shlexec.c b/dlls/shell32/shlexec.c index c8a4550..0d2b364 100644 --- a/dlls/shell32/shlexec.c +++ b/dlls/shell32/shlexec.c @@ -876,67 +876,52 @@ static UINT_PTR execute_from_key(LPWSTR key, LPCWSTR lpFile, WCHAR *env, LPCWSTR SHELL_ExecuteW32 execfunc, LPSHELLEXECUTEINFOW psei, LPSHELLEXECUTEINFOW psei_out) { - WCHAR cmd[256]; - LONG cmdlen = sizeof(cmd); + static const WCHAR wCommand[] = {'c','o','m','m','a','n','d',0}; + static const WCHAR wDdeexec[] = {'d','d','e','e','x','e','c',0}; + WCHAR cmd[256], param[1024], ddeexec[256]; + LONG cmdlen = sizeof(cmd), ddeexeclen = sizeof(ddeexec); UINT_PTR retval = SE_ERR_NOASSOC; + DWORD resultLen; + LPWSTR tmp; TRACE("%s %s %s %s %s\n", debugstr_w(key), debugstr_w(lpFile), debugstr_w(env), debugstr_w(szCommandline), debugstr_w(executable_name)); cmd[0] = '\0'; + param[0] = '\0'; /* Get the application from the registry */ if (RegQueryValueW(HKEY_CLASSES_ROOT, key, cmd, &cmdlen) == ERROR_SUCCESS) { - WCHAR param[1024]; - DWORD resultLen; - TRACE("got cmd: %s\n", debugstr_w(cmd)); - param[0] = '\0'; - /* Is there a replace() function anywhere? */ cmdlen /= sizeof(WCHAR); cmd[cmdlen] = '\0'; - if (!SHELL_ArgifyW(param, sizeof(param)/sizeof(WCHAR), cmd, lpFile, psei->lpIDList, szCommandline, &resultLen)) - { - /* looks like there is no %1 param in the cmd, add one */ - static const WCHAR oneW[] = { ' ','\"','%','1','\"',0 }; - strcatW(cmd, oneW); - SHELL_ArgifyW(param, sizeof(param)/sizeof(WCHAR), cmd, lpFile, psei->lpIDList, szCommandline, &resultLen); - } + SHELL_ArgifyW(param, sizeof(param)/sizeof(WCHAR), cmd, lpFile, psei->lpIDList, szCommandline, &resultLen); if (resultLen > sizeof(param)/sizeof(WCHAR)) ERR("Argify buffer not large enough, truncating\n"); + } + /* Get the parameters needed by the application + from the associated ddeexec key */ + tmp = strstrW(key, wCommand); + assert(tmp); + strcpyW(tmp, wDdeexec); + + if (RegQueryValueW(HKEY_CLASSES_ROOT, key, ddeexec, &ddeexeclen) == ERROR_SUCCESS) + { + TRACE("Got ddeexec %s => %s\n", debugstr_w(key), debugstr_w(ddeexec)); + if (!param[0]) strcpyW(param, executable_name); + retval = dde_connect(key, param, ddeexec, lpFile, env, szCommandline, psei->lpIDList, execfunc, psei, psei_out); + } + else if (param[0]) + { TRACE("executing: %s\n", debugstr_w(param)); retval = execfunc(param, env, FALSE, psei, psei_out); } else - { - static const WCHAR wCommand[] = {'c','o','m','m','a','n','d',0}; - static const WCHAR wDdeexec[] = {'d','d','e','e','x','e','c',0}; - LPWSTR tmp; - WCHAR param[256]; - LONG paramlen = sizeof(param); - - param[0] = '\0'; - - /* Get the parameters needed by the application - from the associated ddeexec key */ - tmp = strstrW(key, wCommand); - assert(tmp); - strcpyW(tmp, wDdeexec); - - TRACE("trying ddeexec cmd: %s\n", debugstr_w(key)); - - if (RegQueryValueW(HKEY_CLASSES_ROOT, key, param, &paramlen) == ERROR_SUCCESS) - { - TRACE("Got ddeexec %s => %s\n", debugstr_w(key), debugstr_w(param)); - retval = dde_connect(key, executable_name, param, lpFile, env, szCommandline, psei->lpIDList, execfunc, psei, psei_out); - } - else - WARN("Nothing appropriate found for %s\n", debugstr_w(key)); - } + WARN("Nothing appropriate found for %s\n", debugstr_w(key)); return retval; } diff --git a/dlls/shell32/tests/shlexec.c b/dlls/shell32/tests/shlexec.c index ee03ed8..eabd25c 100644 --- a/dlls/shell32/tests/shlexec.c +++ b/dlls/shell32/tests/shlexec.c @@ -1082,22 +1082,22 @@ static dde_tests_t dde_tests[] = { /* Test passing and not passing command-line * argument, no DDE */ - {"", NULL, NULL, NULL, NULL, FALSE, "", 0x2, 33}, + {"", NULL, NULL, NULL, NULL, FALSE, "", 0x0, 33}, {"\"%1\"", NULL, NULL, NULL, NULL, TRUE, "", 0x0, 33}, /* Test passing and not passing command-line * argument, with DDE */ - {"", "[open(\"%1\")]", "shlexec", "dde", NULL, FALSE, "[open(\"%s\")]", 0xa, 33}, - {"\"%1\"", "[open(\"%1\")]", "shlexec", "dde", NULL, TRUE, "[open(\"%s\")]", 0x8, 33}, + {"", "[open(\"%1\")]", "shlexec", "dde", NULL, FALSE, "[open(\"%s\")]", 0x0, 33}, + {"\"%1\"", "[open(\"%1\")]", "shlexec", "dde", NULL, TRUE, "[open(\"%s\")]", 0x0, 33}, /* Test ifexec precedence over ddeexec */ - {"", "[open(\"%1\")]", "shlexec", "dde", "[ifexec(\"%1\")]", FALSE, "[ifexec(\"%s\")]", 0xa, 33}, + {"", "[open(\"%1\")]", "shlexec", "dde", "[ifexec(\"%1\")]", FALSE, "[ifexec(\"%s\")]", 0x0, 33}, /* Test default DDE topic */ - {"", "[open(\"%1\")]", "shlexec", NULL, NULL, FALSE, "[open(\"%s\")]", 0xa, 33}, + {"", "[open(\"%1\")]", "shlexec", NULL, NULL, FALSE, "[open(\"%s\")]", 0x0, 33}, /* Test default DDE application */ - {"", "[open(\"%1\")]", NULL, "dde", NULL, FALSE, "[open(\"%s\")]", 0xa, 33}, + {"", "[open(\"%1\")]", NULL, "dde", NULL, FALSE, "[open(\"%s\")]", 0x1, 33}, {NULL, NULL, NULL, NULL, 0x0, 0} };

1 0

Alexandre Julliard : kernel32: Remove the fixme about MB_USEGLYPHCHARS.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: 2835d78d8d5ea7592bc04905db4c85c0abab728c URL: http://source.winehq.org/git/wine.git/?a=commit;h=2835d78d8d5ea7592bc04905d… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue May 29 23:31:35 2007 +0200 kernel32: Remove the fixme about MB_USEGLYPHCHARS. --- dlls/kernel32/locale.c | 7 ------- 1 files changed, 0 insertions(+), 7 deletions(-) diff --git a/dlls/kernel32/locale.c b/dlls/kernel32/locale.c index 74593e5..859f30a 100644 --- a/dlls/kernel32/locale.c +++ b/dlls/kernel32/locale.c @@ -1764,7 +1764,6 @@ INT WINAPI MultiByteToWideChar( UINT page, DWORD flags, LPCSTR src, INT srclen, { const union cptable *table; int ret; - static int once; if (!src || (!dst && dstlen)) { @@ -1774,12 +1773,6 @@ INT WINAPI MultiByteToWideChar( UINT page, DWORD flags, LPCSTR src, INT srclen, if (srclen < 0) srclen = strlen(src) + 1; - if (!once && (flags & MB_USEGLYPHCHARS)) - { - once = 1; - FIXME("MB_USEGLYPHCHARS not supported\n"); - } - switch(page) { case CP_SYMBOL:

1 0

Misha Koshelev : shell32/tests: Add tests for ShellExecute DDE behavior.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: b95d0fb9c39dfe480a3025dc2346537e90a930e4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b95d0fb9c39dfe480a3025dc2… Author: Misha Koshelev <mk144210(a)bcm.edu> Date: Sun May 27 20:00:52 2007 -0500 shell32/tests: Add tests for ShellExecute DDE behavior. --- dlls/shell32/tests/shlexec.c | 282 ++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 282 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b95d0fb9c39dfe480a302…

1 0

Alexandre Julliard : wininet: Fixed compilation without SSL.
by Alexandre Julliard 30 May '07

30 May '07

Module: wine Branch: master Commit: 1ada7ba20e791c400390c314804a2541b2f3b307 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1ada7ba20e791c400390c3148… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Tue May 29 23:27:08 2007 +0200 wininet: Fixed compilation without SSL. --- dlls/wininet/netconnection.c | 9 +++++---- 1 files changed, 5 insertions(+), 4 deletions(-) diff --git a/dlls/wininet/netconnection.c b/dlls/wininet/netconnection.c index 125a90b..0a9f7ba 100644 --- a/dlls/wininet/netconnection.c +++ b/dlls/wininet/netconnection.c @@ -582,10 +582,11 @@ BOOL NETCON_query_data_available(WININET_NETCONNECTION *connection, DWORD *avail if (!NETCON_connected(connection)) return FALSE; - if (connection->peek_msg) - *available = connection->peek_len; - else - *available = 0; + *available = 0; +#if defined HAVE_OPENSSL_SSL_H + if (connection->peek_msg) *available = connection->peek_len; +#endif + #ifdef FIONREAD if (!connection->useSSL) {

1 0

Alexandre Julliard : libwine: Add support for glyph chars mapping in OEM codepage tables.
by Alexandre Julliard 30 May '07

30 May '07

1 0

Dmitry Timoshkov : winebuild: Mark builtin DLLs as IMAGE_FILE_LARGE_ADDRESS_AWARE on a 64-bit platform.
by Alexandre Julliard 29 May '07

29 May '07

Module: wine Branch: master Commit: bb15683818abf950987aa3abbe7fff5a10db7247 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bb15683818abf950987aa3abb… Author: Dmitry Timoshkov <dmitry(a)codeweavers.com> Date: Mon May 28 13:44:46 2007 +0900 winebuild: Mark builtin DLLs as IMAGE_FILE_LARGE_ADDRESS_AWARE on a 64-bit platform. --- tools/winebuild/utils.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tools/winebuild/utils.c b/tools/winebuild/utils.c index a251e6f..c1e42ad 100644 --- a/tools/winebuild/utils.c +++ b/tools/winebuild/utils.c @@ -334,6 +334,8 @@ DLLSPEC *alloc_dll_spec(void) spec->nb_names = 0; spec->nb_resources = 0; spec->characteristics = IMAGE_FILE_EXECUTABLE_IMAGE; + if (get_ptr_size() > 4) + spec->characteristics |= IMAGE_FILE_LARGE_ADDRESS_AWARE; spec->dll_characteristics = IMAGE_DLLCHARACTERISTICS_NX_COMPAT; spec->subsystem = 0; spec->subsystem_major = 4;

1 0