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

Huw Davies : ole32: Add support for getting global handle objects from a flushed clipboard.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: f8a9ca2cface5a57088a139ca4e082004a12a15f URL: http://source.winehq.org/git/wine.git/?a=commit;h=f8a9ca2cface5a57088a139ca… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:32:58 2009 +0100 ole32: Add support for getting global handle objects from a flushed clipboard. --- dlls/ole32/clipboard.c | 143 +++++++++++++++++++++++++++++------------------ 1 files changed, 88 insertions(+), 55 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index c1ef5d3..57a48fb 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -904,20 +904,76 @@ end: return hr; } +/*********************************************************** + * get_priv_data + * + * Returns a copy of the Ole Private Data + */ +static HRESULT get_priv_data(ole_priv_data **data) +{ + HGLOBAL handle; + HRESULT hr = S_OK; + + *data = NULL; + + handle = GetClipboardData( ole_priv_data_clipboard_format ); + if(handle) + { + ole_priv_data *src = GlobalLock(handle); + if(src) + { + /* FIXME: sanity check on size */ + *data = HeapAlloc(GetProcessHeap(), 0, src->size); + if(!*data) + { + GlobalUnlock(handle); + return E_OUTOFMEMORY; + } + memcpy(*data, src, src->size); + GlobalUnlock(handle); + } + } + + if(!*data) hr = create_empty_priv_data(data); + + return hr; +} + +/************************************************************************ + * get_stgmed_for_global + * + * Returns a stg medium with a copy of the global handle + */ +static HRESULT get_stgmed_for_global(HGLOBAL h, STGMEDIUM *med) +{ + HRESULT hr; + + med->pUnkForRelease = NULL; + med->tymed = TYMED_NULL; + + hr = dup_global_mem(h, GMEM_MOVEABLE, &med->u.hGlobal); + + if(SUCCEEDED(hr)) med->tymed = TYMED_HGLOBAL; + + return hr; +} + /************************************************************************ * snapshot_GetData */ -static HRESULT WINAPI snapshot_GetData(IDataObject *iface, - FORMATETC *pformatetcIn, - STGMEDIUM *pmedium) +static HRESULT WINAPI snapshot_GetData(IDataObject *iface, FORMATETC *fmt, + STGMEDIUM *med) { snapshot *This = impl_from_IDataObject(iface); - HANDLE h, hData = 0; + HANDLE h; HRESULT hr; + ole_priv_data *enum_data = NULL; + ole_priv_data_entry *entry; + DWORD mask; - TRACE("(%p,%p,%p)\n", iface, pformatetcIn, pmedium); + TRACE("(%p,%p,%p)\n", iface, fmt, med); - if ( !pformatetcIn || !pmedium ) return E_INVALIDARG; + if ( !fmt || !med ) return E_INVALIDARG; if ( !OpenClipboard(NULL)) return CLIPBRD_E_CANT_OPEN; @@ -926,43 +982,39 @@ static HRESULT WINAPI snapshot_GetData(IDataObject *iface, if(This->data) { - hr = IDataObject_GetData(This->data, pformatetcIn, pmedium); + hr = IDataObject_GetData(This->data, fmt, med); CloseClipboard(); return hr; } + h = GetClipboardData(fmt->cfFormat); + if(!h) + { + hr = DV_E_FORMATETC; + goto end; + } - /* - * Otherwise, get the data from the windows clipboard using GetClipboardData - */ - - if ( pformatetcIn->lindex != -1 ) - { - hr = DV_E_FORMATETC; - goto end; - } - - if ( (pformatetcIn->tymed & TYMED_HGLOBAL) != TYMED_HGLOBAL ) - { - hr = DV_E_TYMED; - goto end; - } + hr = get_priv_data(&enum_data); + if(FAILED(hr)) goto end; - h = GetClipboardData(pformatetcIn->cfFormat); - hr = dup_global_mem(h, GMEM_MOVEABLE, &hData); + entry = find_format_in_list(enum_data->entries, enum_data->count, fmt->cfFormat); + if(entry) + mask = fmt->tymed & entry->fmtetc.tymed; + else /* non-Ole format */ + mask = fmt->tymed & TYMED_HGLOBAL; - /* - * Return the clipboard data in the storage medium structure - */ - pmedium->tymed = (hData == 0) ? TYMED_NULL : TYMED_HGLOBAL; - pmedium->u.hGlobal = hData; - pmedium->pUnkForRelease = NULL; + if(mask & TYMED_HGLOBAL) + hr = get_stgmed_for_global(h, med); + else + { + FIXME("Unhandled tymed - emum tymed %x req tymed %x\n", entry->fmtetc.tymed, fmt->tymed); + hr = E_FAIL; + goto end; + } end: - if ( !CloseClipboard() ) return CLIPBRD_E_CANT_CLOSE; - - if(FAILED(hr)) return hr; - return (hData == 0) ? DV_E_FORMATETC : S_OK; + if ( !CloseClipboard() ) hr = CLIPBRD_E_CANT_CLOSE; + return hr; } /************************************************************************ @@ -1029,8 +1081,7 @@ static HRESULT WINAPI snapshot_SetData(IDataObject *iface, FORMATETC *fmt, static HRESULT WINAPI snapshot_EnumFormatEtc(IDataObject *iface, DWORD dir, IEnumFORMATETC **enum_fmt) { - HRESULT hr = S_OK; - HGLOBAL handle; + HRESULT hr; ole_priv_data *data = NULL; TRACE("(%p, %x, %p)\n", iface, dir, enum_fmt); @@ -1040,26 +1091,8 @@ static HRESULT WINAPI snapshot_EnumFormatEtc(IDataObject *iface, DWORD dir, if ( dir != DATADIR_GET ) return E_NOTIMPL; if ( !OpenClipboard(NULL) ) return CLIPBRD_E_CANT_OPEN; - handle = GetClipboardData( ole_priv_data_clipboard_format ); - if(handle) - { - ole_priv_data *src = GlobalLock(handle); - if(src) - { - /* FIXME: sanity check on size */ - data = HeapAlloc(GetProcessHeap(), 0, src->size); - if(!data) - { - GlobalUnlock(handle); - hr = E_OUTOFMEMORY; - goto end; - } - memcpy(data, src, src->size); - GlobalUnlock(handle); - } - } + hr = get_priv_data(&data); - if(!data) hr = create_empty_priv_data(&data); if(FAILED(hr)) goto end; hr = enum_fmtetc_construct( data, 0, enum_fmt );

1 0

Huw Davies : ole32: The consumer data object is not a singleton, but reflects the state of the clipboard when it was acquired.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: df08d463c921a94681d04b9b00952808e792ed34 URL: http://source.winehq.org/git/wine.git/?a=commit;h=df08d463c921a94681d04b9b0… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:14:49 2009 +0100 ole32: The consumer data object is not a singleton, but reflects the state of the clipboard when it was acquired. --- dlls/ole32/clipboard.c | 294 ++++++++++++++++++++---------------------- dlls/ole32/tests/clipboard.c | 4 - 2 files changed, 140 insertions(+), 158 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=df08d463c921a94681d04…

1 0

Huw Davies : ole32/tests: Add a few more reference count tests.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: 718831b77c5d8b6916b7456d8b311d638aca2adf URL: http://source.winehq.org/git/wine.git/?a=commit;h=718831b77c5d8b6916b7456d8… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:17:51 2009 +0100 ole32/tests: Add a few more reference count tests. --- dlls/ole32/tests/clipboard.c | 43 ++++++++++++++++++++++++++++++++++++++++- 1 files changed, 41 insertions(+), 2 deletions(-) diff --git a/dlls/ole32/tests/clipboard.c b/dlls/ole32/tests/clipboard.c index c77ec95..c3db7b2 100644 --- a/dlls/ole32/tests/clipboard.c +++ b/dlls/ole32/tests/clipboard.c @@ -871,7 +871,7 @@ static inline ULONG count_refs(IDataObject *d) static void test_consumer_refs(void) { HRESULT hr; - IDataObject *src, *get1, *get2, *get3; + IDataObject *src, *src2, *get1, *get2, *get3; ULONG refs, old_refs; FORMATETC fmt; STGMEDIUM med; @@ -885,6 +885,8 @@ static void test_consumer_refs(void) hr = DataObjectImpl_CreateText("data1", &src); ok(hr == S_OK, "got %08x\n", hr); + hr = DataObjectImpl_CreateText("data2", &src2); + ok(hr == S_OK, "got %08x\n", hr); hr = OleSetClipboard(src); ok(hr == S_OK, "got %08x\n", hr); @@ -931,7 +933,6 @@ static void test_consumer_refs(void) ok(hr == S_OK, "got %08x\n", hr); old_refs = count_refs(src); - trace("old_refs %d\n", old_refs); hr = OleGetClipboard(&get1); ok(hr == S_OK, "got %08x\n", hr); @@ -963,6 +964,44 @@ static void test_consumer_refs(void) refs = count_refs(src); ok(refs == 1, "%d\n", refs); + /* Now set a second src object before the call to GetData + and show that GetData calls that second src. */ + + hr = OleSetClipboard(src); + ok(hr == S_OK, "got %08x\n", hr); + + old_refs = count_refs(src); + + hr = OleGetClipboard(&get1); + ok(hr == S_OK, "got %08x\n", hr); + + refs = count_refs(src); + ok(refs == old_refs, "%d %d\n", refs, old_refs); + + hr = OleSetClipboard(src2); + ok(hr == S_OK, "got %08x\n", hr); + + old_refs = count_refs(src2); + + DataObjectImpl_GetData_calls = 0; + hr = IDataObject_GetData(get1, &fmt, &med); + ok(hr == S_OK, "got %08x\n", hr); + ok(DataObjectImpl_GetData_calls == 1, "GetData not called\n"); + ReleaseStgMedium(&med); + + refs = count_refs(src); + ok(refs == 1, "%d\n", refs); + refs = count_refs(src2); + ok(refs == old_refs + 1, "%d %d\n", refs, old_refs); + + OleSetClipboard(NULL); + + refs = count_refs(src2); + ok(refs == 2, "%d\n", refs); + + IDataObject_Release(get1); + + IDataObject_Release(src2); IDataObject_Release(src); OleUninitialize();

1 0

Huw Davies : ole32/tests: Tests that show the consumer data object isn' t a singleton.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: f8a1b7e5a5c375af628cb622c0f05c283b24dc89 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f8a1b7e5a5c375af628cb622c… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:14:19 2009 +0100 ole32/tests: Tests that show the consumer data object isn't a singleton. --- dlls/ole32/tests/clipboard.c | 51 ++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 51 insertions(+), 0 deletions(-) diff --git a/dlls/ole32/tests/clipboard.c b/dlls/ole32/tests/clipboard.c index fc55d38..aadc1ec 100644 --- a/dlls/ole32/tests/clipboard.c +++ b/dlls/ole32/tests/clipboard.c @@ -862,8 +862,59 @@ static void test_set_clipboard(void) OleUninitialize(); } +static void test_consumer_refs(void) +{ + HRESULT hr; + IDataObject *src, *get1, *get2, *get3; + LONG refs; + + OleInitialize(NULL); + + hr = DataObjectImpl_CreateText("data1", &src); + ok(hr == S_OK, "got %08x\n", hr); + + hr = OleSetClipboard(src); + ok(hr == S_OK, "got %08x\n", hr); + + hr = OleGetClipboard(&get1); + ok(hr == S_OK, "got %08x\n", hr); + + hr = OleGetClipboard(&get2); + ok(hr == S_OK, "got %08x\n", hr); + + ok(get1 == get2, "data objects differ\n"); + refs = IDataObject_Release(get2); +todo_wine + ok(refs == 1, "got %d\n", refs); + + OleFlushClipboard(); + + hr = OleGetClipboard(&get2); + ok(hr == S_OK, "got %08x\n", hr); + +todo_wine + ok(get1 != get2, "data objects match\n"); + + OleSetClipboard(NULL); + + hr = OleGetClipboard(&get3); + ok(hr == S_OK, "got %08x\n", hr); + +todo_wine { + ok(get1 != get3, "data objects match\n"); + ok(get2 != get3, "data objects match\n"); +} + + IDataObject_Release(get3); + IDataObject_Release(get2); + IDataObject_Release(get1); + IDataObject_Release(src); + + OleUninitialize(); +} START_TEST(clipboard) { test_set_clipboard(); + test_consumer_refs(); }

1 0

Huw Davies : ole32: Add a helper function to set and release the source IDataObject.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: 6ce9922f3d98c15b827a8179e3d132f10aff7218 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6ce9922f3d98c15b827a8179e… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:08:11 2009 +0100 ole32: Add a helper function to set and release the source IDataObject. --- dlls/ole32/clipboard.c | 244 +++++++++++++++++++++++------------------------ 1 files changed, 119 insertions(+), 125 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 2c39e45..0b67476 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -753,68 +753,6 @@ static HRESULT render_format(IDataObject *data, LPFORMATETC fmt) return hr; } -/*********************************************************************** - * clipbrd_wndproc - */ -static LRESULT CALLBACK clipbrd_wndproc(HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam) -{ - ole_clipbrd *clipbrd; - - get_ole_clipbrd(&clipbrd); - - switch (message) - { - case WM_RENDERFORMAT: - { - UINT cf = wparam; - ole_priv_data_entry *entry; - - TRACE("(): WM_RENDERFORMAT(cfFormat=%x)\n", cf); - entry = find_format_in_list(clipbrd->cached_enum->entries, clipbrd->cached_enum->count, cf); - - if(entry) - render_format(clipbrd->pIDataObjectSrc, &entry->fmtetc); - - break; - } - - case WM_RENDERALLFORMATS: - { - DWORD i; - ole_priv_data_entry *entries = clipbrd->cached_enum->entries; - - TRACE("(): WM_RENDERALLFORMATS\n"); - - for(i = 0; i < clipbrd->cached_enum->count; i++) - { - if(entries[i].first_use) - render_format(clipbrd->pIDataObjectSrc, &entries[i].fmtetc); - } - break; - } - - case WM_DESTROYCLIPBOARD: - { - TRACE("(): WM_DESTROYCLIPBOARD\n"); - - if ( clipbrd->pIDataObjectSrc ) - { - IDataObject_Release(clipbrd->pIDataObjectSrc); - clipbrd->pIDataObjectSrc = NULL; - HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); - clipbrd->cached_enum = NULL; - } - break; - } - - default: - return DefWindowProcW(hwnd, message, wparam, lparam); - } - - return 0; -} - - /*---------------------------------------------------------------------* * Implementation of the internal IDataObject interface exposed by * the OLE clipboard. @@ -1249,37 +1187,6 @@ void OLEClipbrd_UnInitialize(void) } } -/*********************************************************************** - * OLEClipbrd_CreateWindow() - * Create the clipboard window - */ -static HWND OLEClipbrd_CreateWindow(void) -{ - WNDCLASSEXW class; - static const WCHAR ole32W[] = {'o','l','e','3','2',0}; - static const WCHAR title[] = {'C','l','i','p','b','o','a','r','d','W','i','n','d','o','w',0}; - HINSTANCE hinst = GetModuleHandleW(ole32W); - - class.cbSize = sizeof(class); - class.style = 0; - class.lpfnWndProc = clipbrd_wndproc; - class.cbClsExtra = 0; - class.cbWndExtra = 0; - class.hInstance = hinst; - class.hIcon = 0; - class.hCursor = 0; - class.hbrBackground = 0; - class.lpszMenuName = NULL; - class.lpszClassName = clipbrd_wndclass; - class.hIconSm = NULL; - - RegisterClassExW(&class); - - return CreateWindowW(clipbrd_wndclass, title, WS_POPUP | WS_CLIPSIBLINGS | WS_OVERLAPPED, - CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, - NULL, NULL, hinst, 0); -} - /********************************************************************* * set_clipboard_formats * @@ -1372,6 +1279,119 @@ static HRESULT set_clipboard_formats(ole_clipbrd *clipbrd, IDataObject *data) return S_OK; } +/*********************************************************************** + * set_src_dataobject + * + * Clears and sets the clipboard's src IDataObject. + */ +static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) +{ + HRESULT hr = S_OK; + + if(clipbrd->pIDataObjectSrc) + { + IDataObject_Release(clipbrd->pIDataObjectSrc); + clipbrd->pIDataObjectSrc = NULL; + HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); + clipbrd->cached_enum = NULL; + } + + if(data) + { + IDataObject_AddRef(data); + clipbrd->pIDataObjectSrc = data; + hr = set_clipboard_formats(clipbrd, data); + } + return hr; +} + +/*********************************************************************** + * clipbrd_wndproc + */ +static LRESULT CALLBACK clipbrd_wndproc(HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam) +{ + ole_clipbrd *clipbrd; + + get_ole_clipbrd(&clipbrd); + + switch (message) + { + case WM_RENDERFORMAT: + { + UINT cf = wparam; + ole_priv_data_entry *entry; + + TRACE("(): WM_RENDERFORMAT(cfFormat=%x)\n", cf); + entry = find_format_in_list(clipbrd->cached_enum->entries, clipbrd->cached_enum->count, cf); + + if(entry) + render_format(clipbrd->pIDataObjectSrc, &entry->fmtetc); + + break; + } + + case WM_RENDERALLFORMATS: + { + DWORD i; + ole_priv_data_entry *entries = clipbrd->cached_enum->entries; + + TRACE("(): WM_RENDERALLFORMATS\n"); + + for(i = 0; i < clipbrd->cached_enum->count; i++) + { + if(entries[i].first_use) + render_format(clipbrd->pIDataObjectSrc, &entries[i].fmtetc); + } + break; + } + + case WM_DESTROYCLIPBOARD: + { + TRACE("(): WM_DESTROYCLIPBOARD\n"); + + set_src_dataobject(clipbrd, NULL); + break; + } + + default: + return DefWindowProcW(hwnd, message, wparam, lparam); + } + + return 0; +} + + +/*********************************************************************** + * OLEClipbrd_CreateWindow() + * Create the clipboard window + */ +static HWND OLEClipbrd_CreateWindow(void) +{ + WNDCLASSEXW class; + static const WCHAR ole32W[] = {'o','l','e','3','2',0}; + static const WCHAR title[] = {'C','l','i','p','b','o','a','r','d','W','i','n','d','o','w',0}; + HINSTANCE hinst = GetModuleHandleW(ole32W); + + class.cbSize = sizeof(class); + class.style = 0; + class.lpfnWndProc = clipbrd_wndproc; + class.cbClsExtra = 0; + class.cbWndExtra = 0; + class.hInstance = hinst; + class.hIcon = 0; + class.hCursor = 0; + class.hbrBackground = 0; + class.lpszMenuName = NULL; + class.lpszClassName = clipbrd_wndclass; + class.hIconSm = NULL; + + RegisterClassExW(&class); + + return CreateWindowW(clipbrd_wndclass, title, WS_POPUP | WS_CLIPSIBLINGS | WS_OVERLAPPED, + CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, CW_USEDEFAULT, + NULL, NULL, hinst, 0); +} + /********************************************************************* * set_dataobject_format * @@ -1416,12 +1436,12 @@ static HRESULT set_dataobject_format(HWND hwnd) * CLIPBRD_E_CANT_SET SetClipboard failed */ -HRESULT WINAPI OleSetClipboard(IDataObject* pDataObj) +HRESULT WINAPI OleSetClipboard(IDataObject* data) { HRESULT hr; ole_clipbrd *clipbrd; - TRACE("(%p)\n", pDataObj); + TRACE("(%p)\n", data); if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; @@ -1444,25 +1464,8 @@ HRESULT WINAPI OleSetClipboard(IDataObject* pDataObj) goto end; } - /* - * If we are already holding on to an IDataObject first release that. - */ - if ( clipbrd->pIDataObjectSrc ) - { - IDataObject_Release(clipbrd->pIDataObjectSrc); - clipbrd->pIDataObjectSrc = NULL; - HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); - clipbrd->cached_enum = NULL; - } - - /* A NULL value indicates that the clipboard should be emptied. */ - clipbrd->pIDataObjectSrc = pDataObj; - if ( pDataObj ) - { - IDataObject_AddRef(clipbrd->pIDataObjectSrc); - hr = set_clipboard_formats(clipbrd, pDataObj); - if(FAILED(hr)) goto end; - } + hr = set_src_dataobject(clipbrd, data); + if(FAILED(hr)) goto end; hr = set_dataobject_format(clipbrd->hWndClipboard); @@ -1472,13 +1475,7 @@ end: if ( FAILED(hr) ) { - if (clipbrd->pIDataObjectSrc) - { - IDataObject_Release(clipbrd->pIDataObjectSrc); - clipbrd->pIDataObjectSrc = NULL; - HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); - clipbrd->cached_enum = NULL; - } + set_src_dataobject(clipbrd, NULL); } return hr; @@ -1561,10 +1558,7 @@ HRESULT WINAPI OleFlushClipboard(void) hr = set_dataobject_format(NULL); - IDataObject_Release(clipbrd->pIDataObjectSrc); - clipbrd->pIDataObjectSrc = NULL; - HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); - clipbrd->cached_enum = NULL; + set_src_dataobject(clipbrd, NULL); end:

1 0

Huw Davies : ole32: Store the marshalled IDataObject in a window property of the clipboard window .
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: 6d4a39409995c91372c182aa55f791d440ba34a1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6d4a39409995c91372c182aa5… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:12:03 2009 +0100 ole32: Store the marshalled IDataObject in a window property of the clipboard window. This is somewhat different from how Windows behaves. Windows stores two properties; an IID and an endpoint number. --- dlls/ole32/clipboard.c | 57 +++++++++++++++++++++++++++++++++++++++++++++++- 1 files changed, 56 insertions(+), 1 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 3cfb1a8..9e92abe 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -137,6 +137,7 @@ typedef struct ole_clipbrd HWND window; /* Hidden clipboard window */ IDataObject *src_data; /* Source object passed to OleSetClipboard */ ole_priv_data *cached_enum; /* Cached result from the enumeration of src data object */ + IStream *marshal_data; /* Stream onto which to marshal src_data */ } ole_clipbrd; static inline ole_clipbrd *impl_from_IDataObject(IDataObject *iface) @@ -174,6 +175,8 @@ static inline HRESULT get_ole_clipbrd(ole_clipbrd **clipbrd) */ static const WCHAR clipbrd_wndclass[] = {'C','L','I','P','B','R','D','W','N','D','C','L','A','S','S',0}; +static const WCHAR wine_marshal_dataobject[] = {'W','i','n','e',' ','m','a','r','s','h','a','l',' ','d','a','t','a','o','b','j','e','c','t',0}; + static UINT dataobject_clipboard_format; static UINT ole_priv_data_clipboard_format; static UINT embed_source_clipboard_format; @@ -829,6 +832,7 @@ static void OLEClipbrd_Destroy(ole_clipbrd* This) if ( This->window ) destroy_clipbrd_window(This->window); + IStream_Release(This->marshal_data); HeapFree(GetProcessHeap(), 0, This); } @@ -1119,6 +1123,7 @@ static const IDataObjectVtbl OLEClipbrd_IDataObject_VTable = static ole_clipbrd* OLEClipbrd_Construct(void) { ole_clipbrd* This; + HGLOBAL h; This = HeapAlloc( GetProcessHeap(), 0, sizeof(*This) ); if (!This) return NULL; @@ -1130,7 +1135,18 @@ static ole_clipbrd* OLEClipbrd_Construct(void) This->src_data = NULL; This->cached_enum = NULL; + h = GlobalAlloc(GMEM_DDESHARE | GMEM_MOVEABLE, 0); + if(!h) goto error; + + if(FAILED(CreateStreamOnHGlobal(h, TRUE, &This->marshal_data))) + goto error; + return This; + +error: + if(h) GlobalFree(h); + HeapFree(GetProcessHeap(), 0, This); + return NULL; } static void register_clipboard_formats(void) @@ -1291,17 +1307,45 @@ static inline HRESULT get_clipbrd_window(ole_clipbrd *clipbrd, HWND *wnd) return *wnd ? S_OK : E_FAIL; } + +/********************************************************************** + * release_marshal_data + * + * Releases the data and sets the stream back to zero size. + */ +static inline void release_marshal_data(IStream *stm) +{ + LARGE_INTEGER pos; + ULARGE_INTEGER size; + pos.QuadPart = size.QuadPart = 0; + + IStream_Seek(stm, pos, STREAM_SEEK_SET, NULL); + CoReleaseMarshalData(stm); + IStream_Seek(stm, pos, STREAM_SEEK_SET, NULL); + IStream_SetSize(stm, size); +} + /*********************************************************************** * 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. */ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) { - HRESULT hr = S_OK; + HRESULT hr; + HWND wnd; + + if(FAILED(hr = get_clipbrd_window(clipbrd, &wnd))) return hr; if(clipbrd->src_data) { + RemovePropW(wnd, wine_marshal_dataobject); + release_marshal_data(clipbrd->marshal_data); + IDataObject_Release(clipbrd->src_data); clipbrd->src_data = NULL; HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); @@ -1310,8 +1354,19 @@ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) if(data) { + HGLOBAL h; + IUnknown *unk; + IDataObject_AddRef(data); clipbrd->src_data = data; + + IDataObject_QueryInterface(data, &IID_IUnknown, (void**)&unk); + hr = CoMarshalInterface(clipbrd->marshal_data, &IID_IDataObject, unk, + 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;

1 0

Huw Davies : ole32: Rename a variable.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: 1ec12f68b184baf8b0cd28592085a3aafedc75d5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1ec12f68b184baf8b0cd28592… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:09:56 2009 +0100 ole32: Rename a variable. --- dlls/ole32/clipboard.c | 45 ++++++++++++++++++++++----------------------- 1 files changed, 22 insertions(+), 23 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 5990d90..3cfb1a8 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -135,7 +135,7 @@ typedef struct ole_clipbrd LONG ref; HWND window; /* Hidden clipboard window */ - IDataObject *pIDataObjectSrc; /* Source object passed to OleSetClipboard */ + IDataObject *src_data; /* Source object passed to OleSetClipboard */ ole_priv_data *cached_enum; /* Cached result from the enumeration of src data object */ } ole_clipbrd; @@ -461,7 +461,7 @@ static HRESULT render_embed_source_hack(IDataObject *data, LPFORMATETC fmt) hr = StgCreateDocfileOnILockBytes(ptrILockBytes, STGM_SHARE_EXCLUSIVE|STGM_READWRITE, 0, &std.u.pstg); ILockBytes_Release(ptrILockBytes); - if (FAILED(hr = IDataObject_GetDataHere(theOleClipboard->pIDataObjectSrc, fmt, &std))) + if (FAILED(hr = IDataObject_GetDataHere(theOleClipboard->src_data, fmt, &std))) { WARN("() : IDataObject_GetDataHere failed to render clipboard data! (%x)\n", hr); GlobalFree(hStorage); @@ -485,7 +485,7 @@ static HRESULT render_embed_source_hack(IDataObject *data, LPFORMATETC fmt) /* Get the metafile picture out of it */ - if (SUCCEEDED(hr = IDataObject_GetData(theOleClipboard->pIDataObjectSrc, &fmt2, &std2))) + if (SUCCEEDED(hr = IDataObject_GetData(theOleClipboard->src_data, &fmt2, &std2))) { mfp = GlobalLock(std2.u.hGlobal); } @@ -884,9 +884,9 @@ static HRESULT WINAPI OLEClipbrd_IDataObject_GetData( * NOTE: This code assumes that the IDataObject is in the same address space! * We will need to add marshalling support when Wine handles multiple processes. */ - if ( This->pIDataObjectSrc ) + if ( This->src_data ) { - return IDataObject_GetData(This->pIDataObjectSrc, pformatetcIn, pmedium); + return IDataObject_GetData(This->src_data, pformatetcIn, pmedium); } if ( pformatetcIn->lindex != -1 ) @@ -1127,7 +1127,7 @@ static ole_clipbrd* OLEClipbrd_Construct(void) This->ref = 1; This->window = NULL; - This->pIDataObjectSrc = NULL; + This->src_data = NULL; This->cached_enum = NULL; return This; @@ -1300,10 +1300,10 @@ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) { HRESULT hr = S_OK; - if(clipbrd->pIDataObjectSrc) + if(clipbrd->src_data) { - IDataObject_Release(clipbrd->pIDataObjectSrc); - clipbrd->pIDataObjectSrc = NULL; + IDataObject_Release(clipbrd->src_data); + clipbrd->src_data = NULL; HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); clipbrd->cached_enum = NULL; } @@ -1311,7 +1311,7 @@ static HRESULT set_src_dataobject(ole_clipbrd *clipbrd, IDataObject *data) if(data) { IDataObject_AddRef(data); - clipbrd->pIDataObjectSrc = data; + clipbrd->src_data = data; hr = set_clipboard_formats(clipbrd, data); } return hr; @@ -1337,7 +1337,7 @@ static LRESULT CALLBACK clipbrd_wndproc(HWND hwnd, UINT message, WPARAM wparam, entry = find_format_in_list(clipbrd->cached_enum->entries, clipbrd->cached_enum->count, cf); if(entry) - render_format(clipbrd->pIDataObjectSrc, &entry->fmtetc); + render_format(clipbrd->src_data, &entry->fmtetc); break; } @@ -1352,7 +1352,7 @@ static LRESULT CALLBACK clipbrd_wndproc(HWND hwnd, UINT message, WPARAM wparam, for(i = 0; i < clipbrd->cached_enum->count; i++) { if(entries[i].first_use) - render_format(clipbrd->pIDataObjectSrc, &entries[i].fmtetc); + render_format(clipbrd->src_data, &entries[i].fmtetc); } break; } @@ -1530,7 +1530,7 @@ HRESULT WINAPI OleFlushClipboard(void) /* * Already flushed or no source DataObject? Nothing to do. */ - if (!clipbrd->pIDataObjectSrc) return S_OK; + if (!clipbrd->src_data) return S_OK; if (!OpenClipboard(wnd)) return CLIPBRD_E_CANT_OPEN; @@ -1538,7 +1538,7 @@ HRESULT WINAPI OleFlushClipboard(void) * Render all HGLOBAL formats supported by the source into * the windows clipboard. */ - if ( FAILED( hr = IDataObject_EnumFormatEtc( clipbrd->pIDataObjectSrc, + if ( FAILED( hr = IDataObject_EnumFormatEtc( clipbrd->src_data, DATADIR_GET, &penumFormatetc) )) goto end; @@ -1553,7 +1553,7 @@ HRESULT WINAPI OleFlushClipboard(void) GetClipboardFormatNameA(rgelt.cfFormat, szFmtName, sizeof(szFmtName)-1) ? szFmtName : ""); - if ( FAILED(render_format( clipbrd->pIDataObjectSrc, &rgelt )) ) + if ( FAILED(render_format( clipbrd->src_data, &rgelt )) ) continue; } } @@ -1575,16 +1575,15 @@ end: /*********************************************************************** * OleIsCurrentClipboard [OLE32.@] */ -HRESULT WINAPI OleIsCurrentClipboard(IDataObject *pDataObject) +HRESULT WINAPI OleIsCurrentClipboard(IDataObject *data) { - HRESULT hr; - ole_clipbrd *clipbrd; - TRACE("()\n"); + HRESULT hr; + ole_clipbrd *clipbrd; + TRACE("()\n"); - if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; + if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; - if (pDataObject == NULL) - return S_FALSE; + if (data == NULL) return S_FALSE; - return (pDataObject == clipbrd->pIDataObjectSrc) ? S_OK : S_FALSE; + return (data == clipbrd->src_data) ? S_OK : S_FALSE; }

1 0

Huw Davies : ole32: Add a helper function to return the ole clipboard structure.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: bf8d85cee07e95e7c1e6e56a0fe3b4215dcdae3e URL: http://source.winehq.org/git/wine.git/?a=commit;h=bf8d85cee07e95e7c1e6e56a0… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:07:47 2009 +0100 ole32: Add a helper function to return the ole clipboard structure. --- dlls/ole32/clipboard.c | 122 +++++++++++++++++++++++------------------------- 1 files changed, 59 insertions(+), 63 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 196f220..2c39e45 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -157,6 +157,17 @@ typedef struct PresentationDataHeader */ static ole_clipbrd* theOleClipboard; +static inline HRESULT get_ole_clipbrd(ole_clipbrd **clipbrd) +{ + struct oletls *info = COM_CurrentInfo(); + *clipbrd = NULL; + + if(!info->ole_inits) + return CO_E_NOTINITIALIZED; + *clipbrd = theOleClipboard; + + return S_OK; +} /* * Name of our registered OLE clipboard window class @@ -747,7 +758,9 @@ static HRESULT render_format(IDataObject *data, LPFORMATETC fmt) */ static LRESULT CALLBACK clipbrd_wndproc(HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam) { - ole_clipbrd *clipbrd = theOleClipboard; + ole_clipbrd *clipbrd; + + get_ole_clipbrd(&clipbrd); switch (message) { @@ -876,8 +889,6 @@ static void OLEClipbrd_Destroy(ole_clipbrd* This) if (!This) return; - theOleClipboard = NULL; - if ( This->hWndClipboard ) OLEClipbrd_DestroyWindow(This->hWndClipboard); @@ -1183,7 +1194,6 @@ static ole_clipbrd* OLEClipbrd_Construct(void) This->pIDataObjectSrc = NULL; This->cached_enum = NULL; - theOleClipboard = This; return This; } @@ -1231,6 +1241,7 @@ void OLEClipbrd_UnInitialize(void) if ( theOleClipboard && (theOleClipboard->ref <= 1) ) { OLEClipbrd_Destroy( theOleClipboard ); + theOleClipboard = NULL; } else { @@ -1407,28 +1418,21 @@ static HRESULT set_dataobject_format(HWND hwnd) HRESULT WINAPI OleSetClipboard(IDataObject* pDataObj) { - HRESULT hr = S_OK; - struct oletls *info = COM_CurrentInfo(); + HRESULT hr; + ole_clipbrd *clipbrd; TRACE("(%p)\n", pDataObj); - if(!info) - WARN("Could not allocate tls\n"); - else - if(!info->ole_inits) - return CO_E_NOTINITIALIZED; - - OLEClipbrd_Initialize(); + if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; /* * If the Ole clipboard window hasn't been created yet, create it now. */ - if ( !theOleClipboard->hWndClipboard ) - theOleClipboard->hWndClipboard = OLEClipbrd_CreateWindow(); + if ( !clipbrd->hWndClipboard ) + clipbrd->hWndClipboard = OLEClipbrd_CreateWindow(); + if ( !clipbrd->hWndClipboard ) return E_FAIL; - if ( !theOleClipboard->hWndClipboard ) return E_FAIL; - - if ( !OpenClipboard(theOleClipboard->hWndClipboard) ) return CLIPBRD_E_CANT_OPEN; + if ( !OpenClipboard(clipbrd->hWndClipboard) ) return CLIPBRD_E_CANT_OPEN; /* * Empty the current clipboard and make our window the clipboard owner @@ -1443,37 +1447,37 @@ HRESULT WINAPI OleSetClipboard(IDataObject* pDataObj) /* * If we are already holding on to an IDataObject first release that. */ - if ( theOleClipboard->pIDataObjectSrc ) + if ( clipbrd->pIDataObjectSrc ) { - IDataObject_Release(theOleClipboard->pIDataObjectSrc); - theOleClipboard->pIDataObjectSrc = NULL; - HeapFree(GetProcessHeap(), 0, theOleClipboard->cached_enum); - theOleClipboard->cached_enum = NULL; + IDataObject_Release(clipbrd->pIDataObjectSrc); + clipbrd->pIDataObjectSrc = NULL; + HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); + clipbrd->cached_enum = NULL; } /* A NULL value indicates that the clipboard should be emptied. */ - theOleClipboard->pIDataObjectSrc = pDataObj; + clipbrd->pIDataObjectSrc = pDataObj; if ( pDataObj ) { - IDataObject_AddRef(theOleClipboard->pIDataObjectSrc); - hr = set_clipboard_formats(theOleClipboard, pDataObj); + IDataObject_AddRef(clipbrd->pIDataObjectSrc); + hr = set_clipboard_formats(clipbrd, pDataObj); if(FAILED(hr)) goto end; } - hr = set_dataobject_format(theOleClipboard->hWndClipboard); + hr = set_dataobject_format(clipbrd->hWndClipboard); end: - if ( !CloseClipboard() ) hr = CLIPBRD_E_CANT_CLOSE; + if ( !CloseClipboard() ) hr = CLIPBRD_E_CANT_CLOSE; if ( FAILED(hr) ) { - if (theOleClipboard->pIDataObjectSrc) + if (clipbrd->pIDataObjectSrc) { - IDataObject_Release(theOleClipboard->pIDataObjectSrc); - theOleClipboard->pIDataObjectSrc = NULL; - HeapFree(GetProcessHeap(), 0, theOleClipboard->cached_enum); - theOleClipboard->cached_enum = NULL; + IDataObject_Release(clipbrd->pIDataObjectSrc); + clipbrd->pIDataObjectSrc = NULL; + HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); + clipbrd->cached_enum = NULL; } } @@ -1489,21 +1493,15 @@ end: */ HRESULT WINAPI OleGetClipboard(IDataObject** ppDataObj) { - HRESULT hr = S_OK; - TRACE("()\n"); - - /* - * Make sure we have a clipboard object - */ - OLEClipbrd_Initialize(); + HRESULT hr; + ole_clipbrd *clipbrd; + TRACE("()\n"); - if (!theOleClipboard) - return E_OUTOFMEMORY; + if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; - /* Return a reference counted IDataObject */ - hr = IDataObject_QueryInterface( (IDataObject*)&(theOleClipboard->lpvtbl), - &IID_IDataObject, (void**)ppDataObj); - return hr; + hr = IDataObject_QueryInterface( (IDataObject*)&(clipbrd->lpvtbl), + &IID_IDataObject, (void**)ppDataObj); + return hr; } /****************************************************************************** @@ -1519,26 +1517,27 @@ HRESULT WINAPI OleFlushClipboard(void) { IEnumFORMATETC* penumFormatetc = NULL; FORMATETC rgelt; - HRESULT hr = S_OK; + HRESULT hr; + ole_clipbrd *clipbrd; TRACE("()\n"); - OLEClipbrd_Initialize(); + if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; /* * Already flushed or no source DataObject? Nothing to do. */ - if (!theOleClipboard->pIDataObjectSrc) + if (!clipbrd->pIDataObjectSrc) return S_OK; - if (!OpenClipboard(theOleClipboard->hWndClipboard)) + if (!OpenClipboard(clipbrd->hWndClipboard)) return CLIPBRD_E_CANT_OPEN; /* * Render all HGLOBAL formats supported by the source into * the windows clipboard. */ - if ( FAILED( hr = IDataObject_EnumFormatEtc( theOleClipboard->pIDataObjectSrc, + if ( FAILED( hr = IDataObject_EnumFormatEtc( clipbrd->pIDataObjectSrc, DATADIR_GET, &penumFormatetc) )) goto end; @@ -1553,7 +1552,7 @@ HRESULT WINAPI OleFlushClipboard(void) GetClipboardFormatNameA(rgelt.cfFormat, szFmtName, sizeof(szFmtName)-1) ? szFmtName : ""); - if ( FAILED(render_format( theOleClipboard->pIDataObjectSrc, &rgelt )) ) + if ( FAILED(render_format( clipbrd->pIDataObjectSrc, &rgelt )) ) continue; } } @@ -1562,10 +1561,10 @@ HRESULT WINAPI OleFlushClipboard(void) hr = set_dataobject_format(NULL); - IDataObject_Release(theOleClipboard->pIDataObjectSrc); - theOleClipboard->pIDataObjectSrc = NULL; - HeapFree(GetProcessHeap(), 0, theOleClipboard->cached_enum); - theOleClipboard->cached_enum = NULL; + IDataObject_Release(clipbrd->pIDataObjectSrc); + clipbrd->pIDataObjectSrc = NULL; + HeapFree(GetProcessHeap(), 0, clipbrd->cached_enum); + clipbrd->cached_enum = NULL; end: @@ -1580,17 +1579,14 @@ end: */ HRESULT WINAPI OleIsCurrentClipboard(IDataObject *pDataObject) { + HRESULT hr; + ole_clipbrd *clipbrd; TRACE("()\n"); - /* - * Make sure we have a clipboard object - */ - OLEClipbrd_Initialize(); - if (!theOleClipboard) - return E_OUTOFMEMORY; + if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; if (pDataObject == NULL) return S_FALSE; - return (pDataObject == theOleClipboard->pIDataObjectSrc) ? S_OK : S_FALSE; + return (pDataObject == clipbrd->pIDataObjectSrc) ? S_OK : S_FALSE; }

1 0

Huw Davies : ole32: Add a function to return (and possibly create) the clipboard's window.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: 6526b570e15dd0d31157e5432315b648cb2980f4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6526b570e15dd0d31157e5432… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:09:04 2009 +0100 ole32: Add a function to return (and possibly create) the clipboard's window. --- dlls/ole32/clipboard.c | 59 ++++++++++++++++++++++++++---------------------- 1 files changed, 32 insertions(+), 27 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 0b67476..0e2acf4 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -134,7 +134,7 @@ typedef struct ole_clipbrd LONG ref; - HWND hWndClipboard; /* Hidden clipboard window */ + HWND window; /* Hidden clipboard window */ IDataObject *pIDataObjectSrc; /* Source object passed to OleSetClipboard */ ole_priv_data *cached_enum; /* Cached result from the enumeration of src data object */ } ole_clipbrd; @@ -809,10 +809,10 @@ static ULONG WINAPI OLEClipbrd_IDataObject_AddRef( } /*********************************************************************** - * OLEClipbrd_DestroyWindow(HWND) + * destroy_clipbrd_window * Destroy the clipboard window and unregister its class */ -static void OLEClipbrd_DestroyWindow(HWND hwnd) +static void destroy_clipbrd_window(HWND hwnd) { static const WCHAR ole32W[] = {'o','l','e','3','2',0}; HINSTANCE hinst = GetModuleHandleW(ole32W); @@ -827,8 +827,7 @@ static void OLEClipbrd_Destroy(ole_clipbrd* This) if (!This) return; - if ( This->hWndClipboard ) - OLEClipbrd_DestroyWindow(This->hWndClipboard); + if ( This->window ) destroy_clipbrd_window(This->window); HeapFree(GetProcessHeap(), 0, This); } @@ -903,7 +902,7 @@ static HRESULT WINAPI OLEClipbrd_IDataObject_GetData( /* * Otherwise, get the data from the windows clipboard using GetClipboardData */ - if ( !OpenClipboard(theOleClipboard->hWndClipboard)) return CLIPBRD_E_CANT_OPEN; + if ( !OpenClipboard(theOleClipboard->window)) return CLIPBRD_E_CANT_OPEN; h = GetClipboardData(pformatetcIn->cfFormat); hr = dup_global_mem(h, GMEM_MOVEABLE, &hData); @@ -1016,7 +1015,7 @@ static HRESULT WINAPI OLEClipbrd_IDataObject_EnumFormatEtc( *enum_fmt = NULL; if ( dwDirection != DATADIR_GET ) return E_NOTIMPL; - if ( !OpenClipboard(This->hWndClipboard) ) return CLIPBRD_E_CANT_OPEN; + if ( !OpenClipboard(This->window) ) return CLIPBRD_E_CANT_OPEN; handle = GetClipboardData( ole_priv_data_clipboard_format ); if(handle) @@ -1128,7 +1127,7 @@ static ole_clipbrd* OLEClipbrd_Construct(void) This->lpvtbl = &OLEClipbrd_IDataObject_VTable; This->ref = 1; - This->hWndClipboard = NULL; + This->window = NULL; This->pIDataObjectSrc = NULL; This->cached_enum = NULL; @@ -1279,6 +1278,20 @@ static HRESULT set_clipboard_formats(ole_clipbrd *clipbrd, IDataObject *data) return S_OK; } +static HWND create_clipbrd_window(void); + +/*********************************************************************** + * get_clipbrd_window + */ +static inline HRESULT get_clipbrd_window(ole_clipbrd *clipbrd, HWND *wnd) +{ + if ( !clipbrd->window ) + clipbrd->window = create_clipbrd_window(); + + *wnd = clipbrd->window; + return *wnd ? S_OK : E_FAIL; +} + /*********************************************************************** * set_src_dataobject * @@ -1362,10 +1375,9 @@ static LRESULT CALLBACK clipbrd_wndproc(HWND hwnd, UINT message, WPARAM wparam, /*********************************************************************** - * OLEClipbrd_CreateWindow() - * Create the clipboard window + * create_clipbrd_window */ -static HWND OLEClipbrd_CreateWindow(void) +static HWND create_clipbrd_window(void) { WNDCLASSEXW class; static const WCHAR ole32W[] = {'o','l','e','3','2',0}; @@ -1440,24 +1452,16 @@ HRESULT WINAPI OleSetClipboard(IDataObject* data) { HRESULT hr; ole_clipbrd *clipbrd; + HWND wnd; TRACE("(%p)\n", data); if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; - /* - * If the Ole clipboard window hasn't been created yet, create it now. - */ - if ( !clipbrd->hWndClipboard ) - clipbrd->hWndClipboard = OLEClipbrd_CreateWindow(); - if ( !clipbrd->hWndClipboard ) return E_FAIL; + if(FAILED(hr = get_clipbrd_window(clipbrd, &wnd))) return hr; - if ( !OpenClipboard(clipbrd->hWndClipboard) ) return CLIPBRD_E_CANT_OPEN; + if ( !OpenClipboard(wnd) ) return CLIPBRD_E_CANT_OPEN; - /* - * Empty the current clipboard and make our window the clipboard owner - * NOTE: This will trigger a WM_DESTROYCLIPBOARD message - */ if ( !EmptyClipboard() ) { hr = CLIPBRD_E_CANT_EMPTY; @@ -1467,7 +1471,7 @@ HRESULT WINAPI OleSetClipboard(IDataObject* data) hr = set_src_dataobject(clipbrd, data); if(FAILED(hr)) goto end; - hr = set_dataobject_format(clipbrd->hWndClipboard); + hr = set_dataobject_format(wnd); end: @@ -1516,19 +1520,20 @@ HRESULT WINAPI OleFlushClipboard(void) FORMATETC rgelt; HRESULT hr; ole_clipbrd *clipbrd; + HWND wnd; TRACE("()\n"); if(FAILED(hr = get_ole_clipbrd(&clipbrd))) return hr; + if(FAILED(hr = get_clipbrd_window(clipbrd, &wnd))) return hr; + /* * Already flushed or no source DataObject? Nothing to do. */ - if (!clipbrd->pIDataObjectSrc) - return S_OK; + if (!clipbrd->pIDataObjectSrc) return S_OK; - if (!OpenClipboard(clipbrd->hWndClipboard)) - return CLIPBRD_E_CANT_OPEN; + if (!OpenClipboard(wnd)) return CLIPBRD_E_CANT_OPEN; /* * Render all HGLOBAL formats supported by the source into

1 0

Huw Davies : ole32: If there' s a current data object on the ole clipboard GetData should use it.
by Alexandre Julliard April 16, 2009

April 16, 2009

Module: wine Branch: master Commit: 4311691a88da680dd0c81078b8151d5300806323 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4311691a88da680dd0c81078b… Author: Huw Davies <huw(a)codeweavers.com> Date: Thu Apr 16 12:13:39 2009 +0100 ole32: If there's a current data object on the ole clipboard GetData should use it. --- dlls/ole32/clipboard.c | 110 +++++++++++++++++++++++++++++++++++++---------- 1 files changed, 86 insertions(+), 24 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 9e92abe..83d923e 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -859,14 +859,71 @@ static ULONG WINAPI OLEClipbrd_IDataObject_Release( return ref; } +/************************************************************ + * get_current_ole_clip_window + * + * Return the window that owns the ole clipboard. + * + * If the clipboard is flushed or not owned by ole this will + * return NULL. + */ +static HWND get_current_ole_clip_window(void) +{ + HGLOBAL h; + HWND *ptr, wnd; + + h = GetClipboardData(dataobject_clipboard_format); + if(!h) return NULL; + ptr = GlobalLock(h); + if(!ptr) return NULL; + wnd = *ptr; + GlobalUnlock(h); + return wnd; +} + +/************************************************************ + * get_current_dataobject + * + * Return an unmarshalled IDataObject if there is a current + * (ie non-flushed) object on the ole clipboard. + */ +static HRESULT get_current_dataobject(IDataObject **data) +{ + HRESULT hr = S_FALSE; + HWND wnd = get_current_ole_clip_window(); + HGLOBAL h; + void *ptr; + IStream *stm; + LARGE_INTEGER pos; + + *data = NULL; + if(!wnd) return S_FALSE; + + h = GetPropW(wnd, wine_marshal_dataobject); + if(!h) return S_FALSE; + ptr = GlobalLock(h); + if(!ptr) return S_FALSE; + + hr = CreateStreamOnHGlobal(NULL, TRUE, &stm); + if(FAILED(hr)) goto end; + + hr = IStream_Write(stm, ptr, GlobalSize(h), NULL); + if(SUCCEEDED(hr)) + { + pos.QuadPart = 0; + IStream_Seek(stm, pos, STREAM_SEEK_SET, NULL); + hr = CoUnmarshalInterface(stm, &IID_IDataObject, (void**)data); + } + IStream_Release(stm); + +end: + GlobalUnlock(h); + return hr; +} /************************************************************************ * OLEClipbrd_IDataObject_GetData (IDataObject) * - * The OLE Clipboard's implementation of this method delegates to - * a data source if there is one or wraps around the windows clipboard - * - * See Windows documentation for more details on IDataObject methods. */ static HRESULT WINAPI OLEClipbrd_IDataObject_GetData( IDataObject* iface, @@ -874,39 +931,40 @@ static HRESULT WINAPI OLEClipbrd_IDataObject_GetData( STGMEDIUM* pmedium) { HANDLE h, hData = 0; - ole_clipbrd *This = impl_from_IDataObject(iface); HRESULT hr; + IDataObject *data; TRACE("(%p,%p,%p)\n", iface, pformatetcIn, pmedium); if ( !pformatetcIn || !pmedium ) return E_INVALIDARG; - /* - * If we have a data source placed on the clipboard (via OleSetClipboard) - * simply delegate to the source object's QueryGetData - * NOTE: This code assumes that the IDataObject is in the same address space! - * We will need to add marshalling support when Wine handles multiple processes. - */ - if ( This->src_data ) + if ( !OpenClipboard(NULL)) return CLIPBRD_E_CANT_OPEN; + + hr = get_current_dataobject(&data); + if ( hr == S_OK ) { - return IDataObject_GetData(This->src_data, pformatetcIn, pmedium); + hr = IDataObject_GetData(data, pformatetcIn, pmedium); + IDataObject_Release(data); + CloseClipboard(); + return hr; } - if ( pformatetcIn->lindex != -1 ) - return DV_E_FORMATETC; - - if ( (pformatetcIn->tymed & TYMED_HGLOBAL) != TYMED_HGLOBAL ) - return DV_E_TYMED; -/* - if ( pformatetcIn->dwAspect != DVASPECT_CONTENT ) - return DV_E_DVASPECT; -*/ - /* * Otherwise, get the data from the windows clipboard using GetClipboardData */ - if ( !OpenClipboard(NULL)) return CLIPBRD_E_CANT_OPEN; + + if ( pformatetcIn->lindex != -1 ) + { + hr = DV_E_FORMATETC; + goto end; + } + + if ( (pformatetcIn->tymed & TYMED_HGLOBAL) != TYMED_HGLOBAL ) + { + hr = DV_E_TYMED; + goto end; + } h = GetClipboardData(pformatetcIn->cfFormat); hr = dup_global_mem(h, GMEM_MOVEABLE, &hData); @@ -918,12 +976,16 @@ static HRESULT WINAPI OLEClipbrd_IDataObject_GetData( pmedium->u.hGlobal = hData; pmedium->pUnkForRelease = NULL; +end: if ( !CloseClipboard() ) return CLIPBRD_E_CANT_CLOSE; if(FAILED(hr)) return hr; return (hData == 0) ? DV_E_FORMATETC : S_OK; } +/************************************************************************ + * OLEClipbrd_IDataObject_GetDataHere (IDataObject) + */ static HRESULT WINAPI OLEClipbrd_IDataObject_GetDataHere( IDataObject* iface, LPFORMATETC pformatetc,

1 0