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

Nikolay Sivov : comctl32/tests: Tests for LVM_GETCOLUMNORDERARRAY: it actually uses HDM_GETORDERARRAY.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 3845ed795f65a4fdaf179c7a93bcd09f9fcb741d URL: http://source.winehq.org/git/wine.git/?a=commit;h=3845ed795f65a4fdaf179c7a9… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Fri Apr 24 14:51:43 2009 +0400 comctl32/tests: Tests for LVM_GETCOLUMNORDERARRAY: it actually uses HDM_GETORDERARRAY. --- dlls/comctl32/tests/listview.c | 34 +++++++++++++++++++++++++++++++++- 1 files changed, 33 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/tests/listview.c b/dlls/comctl32/tests/listview.c index 8fcf0f4..1612171 100644 --- a/dlls/comctl32/tests/listview.c +++ b/dlls/comctl32/tests/listview.c @@ -152,6 +152,12 @@ static const struct message listview_ownerdata_switchto_seq[] = { { 0 } }; +static const struct message listview_getorderarray_seq[] = { + { LVM_GETCOLUMNORDERARRAY, sent|id|wparam, 2, 0, LISTVIEW_ID }, + { HDM_GETORDERARRAY, sent|id|wparam, 2, 0, HEADER_ID }, + { 0 } +}; + struct subclass_info { WNDPROC oldproc; @@ -800,9 +806,10 @@ static void test_items(void) static void test_columns(void) { - HWND hwnd; + HWND hwnd, hwndheader; LVCOLUMN column; DWORD rc; + INT order[2]; hwnd = CreateWindowEx(0, "SysListView32", "foo", LVS_REPORT, 10, 10, 100, 200, hwndparent, NULL, NULL, NULL); @@ -821,6 +828,31 @@ static void test_columns(void) "Inserting column with no mask failed to set width to 10 with %d\n", rc); DestroyWindow(hwnd); + + /* LVM_GETCOLUMNORDERARRAY */ + hwnd = create_listview_control(0); + hwndheader = subclass_header(hwnd); + + memset(&column, 0, sizeof(column)); + column.mask = LVCF_WIDTH; + column.cx = 100; + rc = ListView_InsertColumn(hwnd, 0, &column); + ok(rc == 0, "Inserting column failed with %d\n", rc); + + column.cx = 200; + rc = ListView_InsertColumn(hwnd, 1, &column); + ok(rc == 1, "Inserting column failed with %d\n", rc); + + flush_sequences(sequences, NUM_MSG_SEQUENCES); + + rc = SendMessage(hwnd, LVM_GETCOLUMNORDERARRAY, 2, (LPARAM)&order); + ok(rc != 0, "Expected LVM_GETCOLUMNORDERARRAY to succeed\n"); + ok(order[0] == 0, "Expected order 0, got %d\n", order[0]); + ok(order[1] == 1, "Expected order 1, got %d\n", order[1]); + + ok_sequence(sequences, LISTVIEW_SEQ_INDEX, listview_getorderarray_seq, "get order array", TRUE); + + DestroyWindow(hwnd); } /* test setting imagelist between WM_NCCREATE and WM_CREATE */ static WNDPROC listviewWndProc;

1 0

Nikolay Sivov : comctl32/listview: Fix LVM_GETCOLUMNORDERARRAY passing message to header.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 8b26cce8f8fb906aeaf2be09b59764a6299ac338 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8b26cce8f8fb906aeaf2be09b… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Fri Apr 24 14:59:35 2009 +0400 comctl32/listview: Fix LVM_GETCOLUMNORDERARRAY passing message to header. --- dlls/comctl32/listview.c | 8 ++------ dlls/comctl32/tests/listview.c | 2 +- 2 files changed, 3 insertions(+), 7 deletions(-) diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index c6a06c7..2aed87e 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -5375,16 +5375,12 @@ static BOOL LISTVIEW_GetColumnT(const LISTVIEW_INFO *infoPtr, INT nColumn, LPLVC static BOOL LISTVIEW_GetColumnOrderArray(const LISTVIEW_INFO *infoPtr, INT iCount, LPINT lpiArray) { - INT i; + TRACE("iCount=%d, lpiArray=%p\n", iCount, lpiArray); if (!lpiArray) return FALSE; - /* FIXME: little hack */ - for (i = 0; i < iCount; i++) - lpiArray[i] = i; - - return TRUE; + return SendMessageW(infoPtr->hwndHeader, HDM_GETORDERARRAY, iCount, (LPARAM)lpiArray); } /*** diff --git a/dlls/comctl32/tests/listview.c b/dlls/comctl32/tests/listview.c index 1612171..5d40546 100644 --- a/dlls/comctl32/tests/listview.c +++ b/dlls/comctl32/tests/listview.c @@ -850,7 +850,7 @@ static void test_columns(void) ok(order[0] == 0, "Expected order 0, got %d\n", order[0]); ok(order[1] == 1, "Expected order 1, got %d\n", order[1]); - ok_sequence(sequences, LISTVIEW_SEQ_INDEX, listview_getorderarray_seq, "get order array", TRUE); + ok_sequence(sequences, LISTVIEW_SEQ_INDEX, listview_getorderarray_seq, "get order array", FALSE); DestroyWindow(hwnd); }

1 0

André Hentschel : winemaker: Print version in Makefile header.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: f711704cc81a3477131485e271bd3cb96e685c18 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f711704cc81a3477131485e27… Author: André Hentschel <nerv(a)dawncrow.de> Date: Thu Apr 23 19:07:42 2009 +0200 winemaker: Print version in Makefile header. --- tools/winemaker | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/winemaker b/tools/winemaker index ed9c5e5..0847b84 100755 --- a/tools/winemaker +++ b/tools/winemaker @@ -2230,7 +2230,7 @@ sub generate_project_files($) return; } - print FILEO "### Generated by Winemaker\n"; + print FILEO "### Generated by Winemaker $version\n"; print FILEO "\n\n"; generate_list("SRCDIR",1,[ "." ]);

1 0

André Hentschel : winemaker: Print all copyrights in banner.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 0c53d2ef0ddbd874dce527c96b2d9dec71419d8f URL: http://source.winehq.org/git/wine.git/?a=commit;h=0c53d2ef0ddbd874dce527c96… Author: André Hentschel <nerv(a)dawncrow.de> Date: Thu Apr 23 19:07:54 2009 +0200 winemaker: Print all copyrights in banner. --- tools/winemaker | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/tools/winemaker b/tools/winemaker index 0847b84..f3a31ce 100755 --- a/tools/winemaker +++ b/tools/winemaker @@ -2465,6 +2465,8 @@ sub print_banner() { print "Winemaker $version\n"; print "Copyright 2000 Francois Gouget <fgouget\(a)codeweavers.com> for CodeWeavers\n"; + print "Copyright 2004 Dimitrie O. Paun\n"; + print "Copyright 2009 André Hentschel\n"; } sub usage()

1 0

Alexandre Julliard : include: Fix the SYSTEM_CACHE_INFORMATION structure for Win64.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 1f8876b1decd892f7456828b8325ceb307326350 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1f8876b1decd892f7456828b8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 24 12:31:03 2009 +0200 include: Fix the SYSTEM_CACHE_INFORMATION structure for Win64. --- include/winternl.h | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/include/winternl.h b/include/winternl.h index 48fa8d0..5ebbca5 100644 --- a/include/winternl.h +++ b/include/winternl.h @@ -1302,9 +1302,11 @@ typedef struct _SYSTEM_CACHE_INFORMATION { ULONG CurrentSize; ULONG PeakSize; ULONG PageFaultCount; +#ifndef _WIN64 ULONG MinimumWorkingSet; ULONG MaximumWorkingSet; ULONG unused[4]; +#endif } SYSTEM_CACHE_INFORMATION, *PSYSTEM_CACHE_INFORMATION; /* System Information Class 0x17 */

1 0

Alexandre Julliard : include: Fix the DDE structures for 64-bit.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 7bac0f52a674fc4ef271af1e77728b8c488b3e18 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7bac0f52a674fc4ef271af1e7… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 24 13:40:39 2009 +0200 include: Fix the DDE structures for 64-bit. --- include/ddeml.h | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/include/ddeml.h b/include/ddeml.h index 91ec17f..6ec016f 100644 --- a/include/ddeml.h +++ b/include/ddeml.h @@ -314,14 +314,15 @@ typedef struct tagCONVCONTEXT UINT wFlags; UINT wCountryID; INT iCodePage; - DWORD dwLangID; - DWORD dwSecurity; + DWORD dwLangID; + DWORD dwSecurity; + SECURITY_QUALITY_OF_SERVICE qos; } CONVCONTEXT, *PCONVCONTEXT; typedef struct tagCONVINFO { DWORD cb; - DWORD hUser; + DWORD_PTR hUser; HCONV hConvPartner; HSZ hszSvcPartner; HSZ hszServiceReq;

1 0

Alexandre Julliard : user32: Fix the DDE service formatting for 64-bit.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: b5346a8ce3c43a3cd5f570e3676b24a2fd717f34 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b5346a8ce3c43a3cd5f570e36… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 24 12:48:44 2009 +0200 user32: Fix the DDE service formatting for 64-bit. --- dlls/user32/dde_misc.c | 4 ++-- dlls/user32/tests/dde.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/user32/dde_misc.c b/dlls/user32/dde_misc.c index d0243c7..fda8255 100644 --- a/dlls/user32/dde_misc.c +++ b/dlls/user32/dde_misc.c @@ -1647,7 +1647,7 @@ HGLOBAL WDML_DataHandle2Global(HDDEDATA hDdeData, BOOL fResponse, BOOL fRelease, */ WDML_SERVER* WDML_AddServer(WDML_INSTANCE* pInstance, HSZ hszService, HSZ hszTopic) { - static const WCHAR fmtW[] = {'%','s','(','0','x','%','0','8','l','x',')',0}; + static const WCHAR fmtW[] = {'%','s','(','0','x','%','*','x',')',0}; WDML_SERVER* pServer; WCHAR buf1[256]; WCHAR buf2[256]; @@ -1659,7 +1659,7 @@ WDML_SERVER* WDML_AddServer(WDML_INSTANCE* pInstance, HSZ hszService, HSZ hszTop WDML_IncHSZ(pInstance, hszService); DdeQueryStringW(pInstance->instanceID, hszService, buf1, 256, CP_WINUNICODE); - snprintfW(buf2, 256, fmtW, buf1, GetCurrentProcessId()); + snprintfW(buf2, 256, fmtW, buf1, 2*sizeof(ULONG_PTR), GetCurrentProcessId()); pServer->hszServiceSpec = DdeCreateStringHandleW(pInstance->instanceID, buf2, CP_WINUNICODE); pServer->atomService = WDML_MakeAtomFromHsz(pServer->hszService); diff --git a/dlls/user32/tests/dde.c b/dlls/user32/tests/dde.c index e7f09f1..3fa9a4f 100644 --- a/dlls/user32/tests/dde.c +++ b/dlls/user32/tests/dde.c @@ -585,7 +585,7 @@ static HDDEDATA CALLBACK server_ddeml_callback(UINT uType, UINT uFmt, HCONV hcon size = DdeQueryStringA(server_pid, hsz2, str, MAX_PATH, CP_WINANSI); ok(!strncmp(str, "TestDDEServer(", 14), "Expected TestDDEServer(, got %s\n", str); ok(str[size - 1] == ')', "Expected ')', got %c\n", str[size - 1]); - ok(size == 25, "Expected 25, got %d\n", size); + ok(size == 17 + 2*sizeof(ULONG_PTR), "Got size %d for %s\n", size, str); return (HDDEDATA)TRUE; }

1 0

Alexandre Julliard : user32/tests: Don' t try to unpack or free invalid handles.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 5515b022ba59a25bac9b68ecd98daf303d1333e7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5515b022ba59a25bac9b68ecd… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Apr 24 13:38:58 2009 +0200 user32/tests: Don't try to unpack or free invalid handles. --- dlls/user32/tests/dde.c | 103 +---------------------------------------------- 1 files changed, 1 insertions(+), 102 deletions(-) diff --git a/dlls/user32/tests/dde.c b/dlls/user32/tests/dde.c index 3fa9a4f..840406e 100644 --- a/dlls/user32/tests/dde.c +++ b/dlls/user32/tests/dde.c @@ -1188,7 +1188,7 @@ static LRESULT WINAPI dde_server_wndprocW(HWND hwnd, UINT msg, WPARAM wparam, LP ok(!IsWindowUnicode((HWND)wparam), "client should be an ANSI window\n"); old_dde_client_wndproc = (WNDPROC)SetWindowLongPtrA((HWND)wparam, GWLP_WNDPROC, (ULONG_PTR)hook_dde_client_wndproc); trace("server: sending WM_DDE_ACK to %p\n", (HWND)wparam); - SendMessageW((HWND)wparam, WM_DDE_ACK, (WPARAM)hwnd, MAKELPARAM(aService, 0)); + SendMessageW((HWND)wparam, WM_DDE_ACK, (WPARAM)hwnd, PackDDElParam(WM_DDE_ACK, aService, 0)); } else GlobalDeleteAtom(aService); @@ -1802,10 +1802,6 @@ static void test_FreeDDElParam(void) hglobal = GlobalAlloc(GMEM_DDESHARE, 100); ret = FreeDDElParam(WM_DDE_ADVISE, (LPARAM)hglobal); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - val = GlobalFree(hglobal); - ok(val == hglobal, "Expected hglobal, got %p\n", val); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); hglobal = GlobalAlloc(GMEM_DDESHARE, 100); ret = FreeDDElParam(WM_DDE_UNADVISE, (LPARAM)hglobal); @@ -1816,18 +1812,10 @@ static void test_FreeDDElParam(void) hglobal = GlobalAlloc(GMEM_DDESHARE, 100); ret = FreeDDElParam(WM_DDE_ACK, (LPARAM)hglobal); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - val = GlobalFree(hglobal); - ok(val == hglobal, "Expected hglobal, got %p\n", val); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); hglobal = GlobalAlloc(GMEM_DDESHARE, 100); ret = FreeDDElParam(WM_DDE_DATA, (LPARAM)hglobal); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - val = GlobalFree(hglobal); - ok(val == hglobal, "Expected hglobal, got %p\n", val); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); hglobal = GlobalAlloc(GMEM_DDESHARE, 100); ret = FreeDDElParam(WM_DDE_REQUEST, (LPARAM)hglobal); @@ -1838,10 +1826,6 @@ static void test_FreeDDElParam(void) hglobal = GlobalAlloc(GMEM_DDESHARE, 100); ret = FreeDDElParam(WM_DDE_POKE, (LPARAM)hglobal); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - val = GlobalFree(hglobal); - ok(val == hglobal, "Expected hglobal, got %p\n", val); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); hglobal = GlobalAlloc(GMEM_DDESHARE, 100); ret = FreeDDElParam(WM_DDE_EXECUTE, (LPARAM)hglobal); @@ -1853,17 +1837,12 @@ static void test_FreeDDElParam(void) static void test_PackDDElParam(void) { UINT_PTR lo, hi, *ptr; - HGLOBAL hglobal; LPARAM lparam; BOOL ret; lparam = PackDDElParam(WM_DDE_INITIATE, 0xcafe, 0xbeef); /* value gets sign-extended despite being an LPARAM */ ok(lparam == (int)0xbeefcafe, "Expected 0xbeefcafe, got %08lx\n", lparam); - ok(GlobalLock((HGLOBAL)lparam) == NULL, - "Expected NULL, got %p\n", GlobalLock((HGLOBAL)lparam)); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); lo = hi = 0; ret = UnpackDDElParam(WM_DDE_INITIATE, lparam, &lo, &hi); @@ -1876,10 +1855,6 @@ static void test_PackDDElParam(void) lparam = PackDDElParam(WM_DDE_TERMINATE, 0xcafe, 0xbeef); ok(lparam == (int)0xbeefcafe, "Expected 0xbeefcafe, got %08lx\n", lparam); - ok(GlobalLock((HGLOBAL)lparam) == NULL, - "Expected NULL, got %p\n", GlobalLock((HGLOBAL)lparam)); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); lo = hi = 0; ret = UnpackDDElParam(WM_DDE_TERMINATE, lparam, &lo, &hi); @@ -1913,17 +1888,8 @@ static void test_PackDDElParam(void) ret = FreeDDElParam(WM_DDE_ADVISE, lparam); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - hglobal = GlobalFree((HGLOBAL)lparam); - ok(hglobal == (HGLOBAL)lparam, "Expected lparam, got %d\n", ret); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); - lparam = PackDDElParam(WM_DDE_UNADVISE, 0xcafe, 0xbeef); ok(lparam == (int)0xbeefcafe, "Expected 0xbeefcafe, got %08lx\n", lparam); - ok(GlobalLock((HGLOBAL)lparam) == NULL, - "Expected NULL, got %p\n", GlobalLock((HGLOBAL)lparam)); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); lo = hi = 0; ret = UnpackDDElParam(WM_DDE_UNADVISE, lparam, &lo, &hi); @@ -1953,11 +1919,6 @@ static void test_PackDDElParam(void) ret = FreeDDElParam(WM_DDE_ACK, lparam); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - - hglobal = GlobalFree((HGLOBAL)lparam); - ok(hglobal == (HGLOBAL)lparam, "Expected lparam, got %d\n", ret); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); } else win_skip("got lparam 0x%lx for WM_DDE_ACK\n", lparam); @@ -1985,17 +1946,8 @@ static void test_PackDDElParam(void) ret = FreeDDElParam(WM_DDE_DATA, lparam); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - hglobal = GlobalFree((HGLOBAL)lparam); - ok(hglobal == (HGLOBAL)lparam, "Expected lparam, got %d\n", ret); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); - lparam = PackDDElParam(WM_DDE_REQUEST, 0xcafe, 0xbeef); ok(lparam == (int)0xbeefcafe, "Expected 0xbeefcafe, got %08lx\n", lparam); - ok(GlobalLock((HGLOBAL)lparam) == NULL, - "Expected NULL, got %p\n", GlobalLock((HGLOBAL)lparam)); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); lo = hi = 0; ret = UnpackDDElParam(WM_DDE_REQUEST, lparam, &lo, &hi); @@ -2029,17 +1981,8 @@ static void test_PackDDElParam(void) ret = FreeDDElParam(WM_DDE_POKE, lparam); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); - hglobal = GlobalFree((HGLOBAL)lparam); - ok(hglobal == (HGLOBAL)lparam, "Expected lparam, got %d\n", ret); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); - lparam = PackDDElParam(WM_DDE_EXECUTE, 0xcafe, 0xbeef); ok(lparam == 0xbeef, "Expected 0xbeef, got %08lx\n", lparam); - ok(GlobalLock((HGLOBAL)lparam) == NULL, - "Expected NULL, got %p\n", GlobalLock((HGLOBAL)lparam)); - ok(GetLastError() == ERROR_INVALID_HANDLE, - "Expected ERROR_INVALID_HANDLE, got %d\n", GetLastError()); lo = hi = 0; ret = UnpackDDElParam(WM_DDE_EXECUTE, lparam, &lo, &hi); @@ -2106,17 +2049,6 @@ static void test_UnpackDDElParam(void) broken(hi == 0xbeef), /* win2k */ "Expected 0, got %08lx\n", hi); - lo = 0xdead; - hi = 0xbeef; - ret = UnpackDDElParam(WM_DDE_ADVISE, 0xcafebabe, &lo, &hi); - ok(ret == FALSE, "Expected FALSE, got %d\n", ret); - ok(lo == 0 || - broken(lo == 0xdead), /* win2k */ - "Expected 0, got %08lx\n", lo); - ok(hi == 0 || - broken(hi == 0xbeef), /* win2k */ - "Expected 0, got %08lx\n", hi); - hglobal = GlobalAlloc(GMEM_DDESHARE, 2); ptr = GlobalLock(hglobal); ptr[0] = 0xcafebabe; @@ -2139,17 +2071,6 @@ static void test_UnpackDDElParam(void) lo = 0xdead; hi = 0xbeef; - ret = UnpackDDElParam(WM_DDE_ACK, 0xcafebabe, &lo, &hi); - ok(ret == FALSE, "Expected FALSE, got %d\n", ret); - ok(lo == 0 || - broken(lo == 0xdead), /* win2k */ - "Expected 0, got %08lx\n", lo); - ok(hi == 0 || - broken(hi == 0xbeef), /* win2k */ - "Expected 0, got %08lx\n", hi); - - lo = 0xdead; - hi = 0xbeef; ret = UnpackDDElParam(WM_DDE_ACK, (LPARAM)hglobal, &lo, &hi); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); ok(lo == 0xcafebabe, "Expected 0xcafebabe, got %08lx\n", lo); @@ -2157,17 +2078,6 @@ static void test_UnpackDDElParam(void) lo = 0xdead; hi = 0xbeef; - ret = UnpackDDElParam(WM_DDE_DATA, 0xcafebabe, &lo, &hi); - ok(ret == FALSE, "Expected FALSE, got %d\n", ret); - ok(lo == 0 || - broken(lo == 0xdead), /* win2k */ - "Expected 0, got %08lx\n", lo); - ok(hi == 0 || - broken(hi == 0xbeef), /* win2k */ - "Expected 0, got %08lx\n", hi); - - lo = 0xdead; - hi = 0xbeef; ret = UnpackDDElParam(WM_DDE_DATA, (LPARAM)hglobal, &lo, &hi); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); ok(lo == 0xcafebabe, "Expected 0xcafebabe, got %08lx\n", lo); @@ -2182,17 +2092,6 @@ static void test_UnpackDDElParam(void) lo = 0xdead; hi = 0xbeef; - ret = UnpackDDElParam(WM_DDE_POKE, 0xcafebabe, &lo, &hi); - ok(ret == FALSE, "Expected FALSE, got %d\n", ret); - ok(lo == 0 || - broken(lo == 0xdead), /* win2k */ - "Expected 0, got %08lx\n", lo); - ok(hi == 0 || - broken(hi == 0xbeef), /* win2k */ - "Expected 0, got %08lx\n", hi); - - lo = 0xdead; - hi = 0xbeef; ret = UnpackDDElParam(WM_DDE_POKE, (LPARAM)hglobal, &lo, &hi); ok(ret == TRUE, "Expected TRUE, got %d\n", ret); ok(lo == 0xcafebabe, "Expected 0xcafebabe, got %08lx\n", lo);

1 0

Huw Davies : ole32: Expose the marshalled data on the clipboard rather than as a window prop so that other processes can actually read it .
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 12edc21a3a8ab38768859f758f452651adcf8eeb URL: http://source.winehq.org/git/wine.git/?a=commit;h=12edc21a3a8ab38768859f758… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Apr 24 12:02:48 2009 +0100 ole32: Expose the marshalled data on the clipboard rather than as a window prop so that other processes can actually read it. --- dlls/ole32/clipboard.c | 49 +++++++++++++++++++++++++++++++++++++++++------ 1 files changed, 42 insertions(+), 7 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index ec11411..4d7a7db 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -208,6 +208,8 @@ UINT object_descriptor_clipboard_format = 0; UINT link_source_descriptor_clipboard_format = 0; UINT ole_private_data_clipboard_format = 0; +static UINT wine_marshal_clipboard_format; + static inline char *dump_fmtetc(FORMATETC *fmt) { static char buf[100]; @@ -905,8 +907,9 @@ static HRESULT get_current_dataobject(IDataObject **data) *data = NULL; if(!wnd) return S_FALSE; - h = GetPropW(wnd, wine_marshal_dataobject); + h = GetClipboardData(wine_marshal_clipboard_format); if(!h) return S_FALSE; + if(GlobalSize(h) == 0) return S_FALSE; ptr = GlobalLock(h); if(!ptr) return S_FALSE; @@ -1379,6 +1382,9 @@ static void register_clipboard_formats(void) 'D','e','s','c','r','i','p','t','o','r',0}; static const WCHAR OlePrivateData[] = {'O','l','e',' ','P','r','i','v','a','t','e',' ','D','a','t','a',0}; + static const WCHAR WineMarshalledDataObject[] = {'W','i','n','e',' ','M','a','r','s','h','a','l','l','e','d',' ', + 'D','a','t','a','O','b','j','e','c','t',0}; + ownerlink_clipboard_format = RegisterClipboardFormatW(OwnerLink); filename_clipboard_format = RegisterClipboardFormatW(FileName); filenameW_clipboard_format = RegisterClipboardFormatW(FileNameW); @@ -1390,6 +1396,8 @@ static void register_clipboard_formats(void) object_descriptor_clipboard_format = RegisterClipboardFormatW(ObjectDescriptor); link_source_descriptor_clipboard_format = RegisterClipboardFormatW(LinkSourceDescriptor); ole_private_data_clipboard_format = RegisterClipboardFormatW(OlePrivateData); + + wine_marshal_clipboard_format = RegisterClipboardFormatW(WineMarshalledDataObject); } /*********************************************************************** @@ -1584,13 +1592,38 @@ static inline void release_marshal_data(IStream *stm) } /*********************************************************************** + * expose_marshalled_dataobject + * + * Sets the marshalled dataobject to the clipboard. In the flushed case + * we set a zero sized HGLOBAL to clear the old marshalled data. + */ +static HRESULT expose_marshalled_dataobject(ole_clipbrd *clipbrd, IDataObject *data) +{ + HGLOBAL h; + + if(data) + { + HGLOBAL h_stm; + GetHGlobalFromStream(clipbrd->marshal_data, &h_stm); + dup_global_mem(h_stm, GMEM_DDESHARE|GMEM_MOVEABLE, &h); + } + else /* flushed */ + h = GlobalAlloc(GMEM_DDESHARE|GMEM_MOVEABLE, 0); + + if(!h) return E_OUTOFMEMORY; + + SetClipboardData(wine_marshal_clipboard_format, h); + return S_OK; +} + +/*********************************************************************** * set_src_dataobject * * Clears and sets the clipboard's src IDataObject. * * To marshal the source dataobject we do something rather different from Windows. - * We set a window prop which contains the marshalled data. - * Windows set two props one of which is an IID, the other is an endpoint number. + * We set a clipboard format which contains the marshalled data. + * Windows sets two window props one of which is an IID, the other is an endpoint number. */ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) { @@ -1601,7 +1634,6 @@ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) if(clipbrd->src_data) { - RemovePropW(wnd, wine_marshal_dataobject); release_marshal_data(clipbrd->marshal_data); IDataObject_Release(clipbrd->src_data); @@ -1612,7 +1644,6 @@ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) if(data) { - HGLOBAL h; IUnknown *unk; IDataObject_AddRef(data); @@ -1623,8 +1654,6 @@ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) MSHCTX_LOCAL, NULL, MSHLFLAGS_TABLESTRONG); IUnknown_Release(unk); /* Don't hold a ref on IUnknown, we have one on IDataObject. */ if(FAILED(hr)) return hr; - GetHGlobalFromStream(clipbrd->marshal_data, &h); - SetPropW(wnd, wine_marshal_dataobject, h); hr = set_clipboard_formats(clipbrd, data); } return hr; @@ -1784,7 +1813,11 @@ HRESULT WINAPI OleSetClipboard(IDataObject* data) if(FAILED(hr)) goto end; if(data) + { + hr = expose_marshalled_dataobject(clipbrd, data); + if(FAILED(hr)) goto end; hr = set_dataobject_format(wnd); + } end: @@ -1792,6 +1825,7 @@ end: if ( FAILED(hr) ) { + expose_marshalled_dataobject(clipbrd, NULL); set_src_dataobject(clipbrd, NULL); } @@ -1865,6 +1899,7 @@ HRESULT WINAPI OleFlushClipboard(void) hr = set_dataobject_format(NULL); + expose_marshalled_dataobject(clipbrd, NULL); set_src_dataobject(clipbrd, NULL); if ( !CloseClipboard() ) hr = CLIPBRD_E_CANT_CLOSE;

1 0

Huw Davies : ole32: Fix a couple of comments.
by Alexandre Julliard 24 Apr '09

24 Apr '09

Module: wine Branch: master Commit: 739ca6deae8c80d47d9f857477f03d4f938e1eb3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=739ca6deae8c80d47d9f85747… Author: Huw Davies <huw(a)codeweavers.com> Date: Fri Apr 24 12:15:47 2009 +0100 ole32: Fix a couple of comments. --- dlls/ole32/moniker.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/ole32/moniker.c b/dlls/ole32/moniker.c index 1cd18aa..f65fe8f 100644 --- a/dlls/ole32/moniker.c +++ b/dlls/ole32/moniker.c @@ -340,7 +340,7 @@ RunningObjectTableImpl_AddRef(IRunningObjectTable* iface) } /*********************************************************************** - * RunningObjectTable_Initialize + * RunningObjectTable_Destroy */ static HRESULT RunningObjectTableImpl_Destroy(void) @@ -414,7 +414,7 @@ RunningObjectTableImpl_Release(IRunningObjectTable* iface) * grfFlags [in] Registration options * punkObject [in] the object being registered * pmkObjectName [in] the moniker of the object being registered - * pdwRegister [in] the value identifying the registration + * pdwRegister [out] the value identifying the registration */ static HRESULT WINAPI RunningObjectTableImpl_Register(IRunningObjectTable* iface, DWORD grfFlags,

1 0