wine-commits May 2011

wine-commits@winehq.org

2 participants
945 discussions

Hans Leidekker : msi: Fix the remaining actions to revert features and components to the installed state during rollback .
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 0ecb33e21ed5e848515438e2632b567e8550d8f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0ecb33e21ed5e848515438e26… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed May 25 10:39:41 2011 +0200 msi: Fix the remaining actions to revert features and components to the installed state during rollback. --- dlls/msi/action.c | 170 +++++++++++++++++----------------------------------- 1 files changed, 56 insertions(+), 114 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=0ecb33e21ed5e84851543…

1 0

Hans Leidekker : msi: Fix the assembly actions to revert components to the installed state during rollback .
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 39b7ce6b823a163be2fbd5c3044ff31cd669ffa7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=39b7ce6b823a163be2fbd5c30… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed May 25 10:39:11 2011 +0200 msi: Fix the assembly actions to revert components to the installed state during rollback. --- dlls/msi/assembly.c | 28 ++++++---------------------- 1 files changed, 6 insertions(+), 22 deletions(-) diff --git a/dlls/msi/assembly.c b/dlls/msi/assembly.c index 2beb005..806f89e 100644 --- a/dlls/msi/assembly.c +++ b/dlls/msi/assembly.c @@ -559,20 +559,12 @@ UINT ACTION_MsiPublishAssemblies( MSIPACKAGE *package ) if (!assembly || !comp->ComponentId) continue; - if (!comp->Enabled) + comp->Action = msi_get_component_action( package, comp ); + if (comp->Action != INSTALLSTATE_LOCAL) { - TRACE("component is disabled: %s\n", debugstr_w(comp->Component)); + TRACE("component not scheduled for installation %s\n", debugstr_w(comp->Component)); continue; } - - if (comp->ActionRequest != INSTALLSTATE_LOCAL) - { - TRACE("Component not scheduled for installation: %s\n", debugstr_w(comp->Component)); - comp->Action = comp->Installed; - continue; - } - comp->Action = INSTALLSTATE_LOCAL; - TRACE("publishing %s\n", debugstr_w(comp->Component)); CLSIDFromString( package->ProductCode, &guid ); @@ -628,20 +620,12 @@ UINT ACTION_MsiUnpublishAssemblies( MSIPACKAGE *package ) if (!assembly || !comp->ComponentId) continue; - if (!comp->Enabled) + comp->Action = msi_get_component_action( package, comp ); + if (comp->Action != INSTALLSTATE_ABSENT) { - TRACE("component is disabled: %s\n", debugstr_w(comp->Component)); + TRACE("component not scheduled for removal %s\n", debugstr_w(comp->Component)); continue; } - - if (comp->ActionRequest != INSTALLSTATE_ABSENT) - { - TRACE("Component not scheduled for removal: %s\n", debugstr_w(comp->Component)); - comp->Action = comp->Installed; - continue; - } - comp->Action = INSTALLSTATE_ABSENT; - TRACE("unpublishing %s\n", debugstr_w(comp->Component)); win32 = assembly->attributes & msidbAssemblyAttributesWin32;

1 0

David Hedberg : comdlg32: Add IFileDialogCustomize interface stub.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 4cc6107a7ba56f9af3822c083863fb516044a465 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4cc6107a7ba56f9af3822c083… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:53 2011 +0200 comdlg32: Add IFileDialogCustomize interface stub. --- dlls/comdlg32/itemdlg.c | 314 +++++++++++++++++++++++++++++++++++++++++ dlls/comdlg32/tests/itemdlg.c | 15 ++- 2 files changed, 327 insertions(+), 2 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=4cc6107a7ba56f9af3822…

1 0

David Hedberg : comdlg32/tests: Fix some copypaste errors.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 93b947fd1ee4fdeecd8e5337086e1c9a0b03385a URL: http://source.winehq.org/git/wine.git/?a=commit;h=93b947fd1ee4fdeecd8e53370… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:52 2011 +0200 comdlg32/tests: Fix some copypaste errors. --- dlls/comdlg32/tests/itemdlg.c | 20 ++++++++++---------- 1 files changed, 10 insertions(+), 10 deletions(-) diff --git a/dlls/comdlg32/tests/itemdlg.c b/dlls/comdlg32/tests/itemdlg.c index 5d438ae..c57e166 100644 --- a/dlls/comdlg32/tests/itemdlg.c +++ b/dlls/comdlg32/tests/itemdlg.c @@ -217,13 +217,13 @@ static BOOL test_instantiation(void) ok(hr == S_OK, "got 0x%08x.\n", hr); if(SUCCEEDED(hr)) IFileDialog_Release(pfd); - hr = IFileOpenDialog_QueryInterface(pfod, &IID_IFileDialogCustomize, (void**)&pfd); + hr = IFileOpenDialog_QueryInterface(pfod, &IID_IFileDialogCustomize, (void**)&punk); todo_wine ok(hr == S_OK, "got 0x%08x.\n", hr); - if(SUCCEEDED(hr)) IFileDialog_Release(pfd); + if(SUCCEEDED(hr)) IUnknown_Release(punk); hr = IFileOpenDialog_QueryInterface(pfod, &IID_IFileSaveDialog, (void**)&pfsd); ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); - if(SUCCEEDED(hr)) IFileOpenDialog_Release(pfsd); + if(SUCCEEDED(hr)) IFileSaveDialog_Release(pfsd); hr = IFileOpenDialog_QueryInterface(pfod, &IID_IServiceProvider, (void**)&psp); ok(hr == S_OK, "got 0x%08x.\n", hr); @@ -248,9 +248,9 @@ static BOOL test_instantiation(void) IServiceProvider_Release(psp); } - hr = IFileOpenDialog_QueryInterface(pfod, &IID_IFileDialogEvents, (void**)&pfd); + hr = IFileOpenDialog_QueryInterface(pfod, &IID_IFileDialogEvents, (void**)&punk); ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); - if(SUCCEEDED(hr)) IFileDialogEvents_Release(pfd); + if(SUCCEEDED(hr)) IUnknown_Release(punk); hr = IFileOpenDialog_QueryInterface(pfod, &IID_IExplorerBrowser, (void**)&punk); ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); @@ -310,17 +310,17 @@ static BOOL test_instantiation(void) ok(hr == S_OK, "got 0x%08x.\n", hr); if(SUCCEEDED(hr)) IFileDialog_Release(pfd); - hr = IFileSaveDialog_QueryInterface(pfsd, &IID_IFileDialogCustomize, (void**)&pfd); + hr = IFileSaveDialog_QueryInterface(pfsd, &IID_IFileDialogCustomize, (void**)&punk); todo_wine ok(hr == S_OK, "got 0x%08x.\n", hr); - if(SUCCEEDED(hr)) IFileDialogCustomize_Release(pfd); + if(SUCCEEDED(hr)) IUnknown_Release(punk); - hr = IFileOpenDialog_QueryInterface(pfsd, &IID_IFileOpenDialog, (void**)&pfod); + hr = IFileSaveDialog_QueryInterface(pfsd, &IID_IFileOpenDialog, (void**)&pfod); ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); if(SUCCEEDED(hr)) IFileOpenDialog_Release(pfod); - hr = IFileSaveDialog_QueryInterface(pfsd, &IID_IFileDialogEvents, (void**)&pfd); + hr = IFileSaveDialog_QueryInterface(pfsd, &IID_IFileDialogEvents, (void**)&punk); ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); - if(SUCCEEDED(hr)) IFileDialogEvents_Release(pfd); + if(SUCCEEDED(hr)) IUnknown_Release(pfd); hr = IFileSaveDialog_QueryInterface(pfsd, &IID_IExplorerBrowser, (void**)&punk); ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr);

1 0

David Hedberg : comdlg32: Improve handling of file extension changes in the Item Dialog.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 0d96d26f0d1f1a4d27b28ad00b9927966f3b8feb URL: http://source.winehq.org/git/wine.git/?a=commit;h=0d96d26f0d1f1a4d27b28ad00… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:51 2011 +0200 comdlg32: Improve handling of file extension changes in the Item Dialog. --- dlls/comdlg32/itemdlg.c | 23 +++++++++++++++++++++++ 1 files changed, 23 insertions(+), 0 deletions(-) diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c index 80b0d86..11db9c9 100644 --- a/dlls/comdlg32/itemdlg.c +++ b/dlls/comdlg32/itemdlg.c @@ -872,16 +872,39 @@ static LRESULT on_command_filetype(FileDialogImpl *This, WPARAM wparam, LPARAM l { IShellView *psv; HRESULT hr; + LPWSTR filename; + UINT prev_index = This->filetypeindex; This->filetypeindex = SendMessageW((HWND)lparam, CB_GETCURSEL, 0, 0); TRACE("File type selection changed to %d.\n", This->filetypeindex); + if(prev_index == This->filetypeindex) + return FALSE; + hr = IExplorerBrowser_GetCurrentView(This->peb, &IID_IShellView, (void**)&psv); if(SUCCEEDED(hr)) { IShellView_Refresh(psv); IShellView_Release(psv); } + + if(This->dlg_type == ITEMDLG_TYPE_SAVE && get_file_name(This, &filename)) + { + WCHAR buf[MAX_PATH], extbuf[MAX_PATH], *ext; + + ext = get_first_ext_from_spec(extbuf, This->filterspecs[This->filetypeindex].pszSpec); + if(ext) + { + lstrcpyW(buf, filename); + + if(PathMatchSpecW(buf, This->filterspecs[prev_index].pszSpec)) + PathRemoveExtensionW(buf); + + lstrcatW(buf, ext); + set_file_name(This, buf); + } + CoTaskMemFree(filename); + } } return FALSE;

1 0

David Hedberg : comdlg32: Implement SetDefaultFilename.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: d441d3b8ba74ac27b59614ff02076efa0cad2278 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d441d3b8ba74ac27b59614ff0… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:50 2011 +0200 comdlg32: Implement SetDefaultFilename. --- dlls/comdlg32/cdlg.h | 1 + dlls/comdlg32/filedlg.c | 2 +- dlls/comdlg32/itemdlg.c | 81 ++++++++++++- dlls/comdlg32/tests/itemdlg.c | 275 ++++++++++++++++++++++++++++++++++++++++- 4 files changed, 352 insertions(+), 7 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d441d3b8ba74ac27b5961…

1 0

David Hedberg : comdlg32: Add IOleWindow implementation to the Item Dialog.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 5fd6180a19bf3aa3eef132d9f9b05a01f85cae44 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5fd6180a19bf3aa3eef132d9f… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:49 2011 +0200 comdlg32: Add IOleWindow implementation to the Item Dialog. --- dlls/comdlg32/itemdlg.c | 55 +++++++++++++++++++++++++++++++++++++++++ dlls/comdlg32/tests/itemdlg.c | 52 ++++++++++++++++++++++++++++++++++++++ 2 files changed, 107 insertions(+), 0 deletions(-) diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c index dc447c2..a04b86a 100644 --- a/dlls/comdlg32/itemdlg.c +++ b/dlls/comdlg32/itemdlg.c @@ -65,6 +65,7 @@ typedef struct FileDialogImpl { IExplorerBrowserEvents IExplorerBrowserEvents_iface; IServiceProvider IServiceProvider_iface; ICommDlgBrowser3 ICommDlgBrowser3_iface; + IOleWindow IOleWindow_iface; LONG ref; FILEOPENDIALOGOPTIONS options; @@ -910,6 +911,10 @@ static HRESULT WINAPI IFileDialog2_fnQueryInterface(IFileDialog2 *iface, { *ppvObject = &This->ICommDlgBrowser3_iface; } + else if(IsEqualGUID(&IID_IOleWindow, riid)) + { + *ppvObject = &This->IOleWindow_iface; + } else FIXME("Unknown interface requested: %s.\n", debugstr_guid(riid)); @@ -2206,6 +2211,55 @@ static const ICommDlgBrowser3Vtbl vt_ICommDlgBrowser3 = { ICommDlgBrowser3_fnOnPreviewCreated }; +/************************************************************************** + * IOleWindow implementation + */ +static inline FileDialogImpl *impl_from_IOleWindow(IOleWindow *iface) +{ + return CONTAINING_RECORD(iface, FileDialogImpl, IOleWindow_iface); +} + +static HRESULT WINAPI IOleWindow_fnQueryInterface(IOleWindow *iface, REFIID riid, void **ppvObject) +{ + FileDialogImpl *This = impl_from_IOleWindow(iface); + return IFileDialog2_QueryInterface(&This->IFileDialog2_iface, riid, ppvObject); +} + +static ULONG WINAPI IOleWindow_fnAddRef(IOleWindow *iface) +{ + FileDialogImpl *This = impl_from_IOleWindow(iface); + return IFileDialog2_AddRef(&This->IFileDialog2_iface); +} + +static ULONG WINAPI IOleWindow_fnRelease(IOleWindow *iface) +{ + FileDialogImpl *This = impl_from_IOleWindow(iface); + return IFileDialog2_Release(&This->IFileDialog2_iface); +} + +static HRESULT WINAPI IOleWindow_fnContextSensitiveHelp(IOleWindow *iface, BOOL fEnterMOde) +{ + FileDialogImpl *This = impl_from_IOleWindow(iface); + FIXME("Stub: %p (%d)\n", This, fEnterMOde); + return E_NOTIMPL; +} + +static HRESULT WINAPI IOleWindow_fnGetWindow(IOleWindow *iface, HWND *phwnd) +{ + FileDialogImpl *This = impl_from_IOleWindow(iface); + TRACE("%p (%p)\n", This, phwnd); + *phwnd = This->dlg_hwnd; + return S_OK; +} + +static const IOleWindowVtbl vt_IOleWindow = { + IOleWindow_fnQueryInterface, + IOleWindow_fnAddRef, + IOleWindow_fnRelease, + IOleWindow_fnGetWindow, + IOleWindow_fnContextSensitiveHelp +}; + static HRESULT FileDialog_constructor(IUnknown *pUnkOuter, REFIID riid, void **ppv, enum ITEMDLG_TYPE type) { FileDialogImpl *fdimpl; @@ -2227,6 +2281,7 @@ static HRESULT FileDialog_constructor(IUnknown *pUnkOuter, REFIID riid, void **p fdimpl->IExplorerBrowserEvents_iface.lpVtbl = &vt_IExplorerBrowserEvents; fdimpl->IServiceProvider_iface.lpVtbl = &vt_IServiceProvider; fdimpl->ICommDlgBrowser3_iface.lpVtbl = &vt_ICommDlgBrowser3; + fdimpl->IOleWindow_iface.lpVtbl = &vt_IOleWindow; if(type == ITEMDLG_TYPE_OPEN) { diff --git a/dlls/comdlg32/tests/itemdlg.c b/dlls/comdlg32/tests/itemdlg.c index fcd0e50..7c52868 100644 --- a/dlls/comdlg32/tests/itemdlg.c +++ b/dlls/comdlg32/tests/itemdlg.c @@ -166,6 +166,7 @@ static BOOL test_instantiation(void) IFileOpenDialog *pfod; IFileSaveDialog *pfsd; IServiceProvider *psp; + IOleWindow *pow; IUnknown *punk; HRESULT hr; LONG ref; @@ -235,6 +236,31 @@ static BOOL test_instantiation(void) ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); if(SUCCEEDED(hr)) IUnknown_Release(punk); + hr = IFileOpenDialog_QueryInterface(pfod, &IID_IOleWindow, (void**)&pow); + ok(hr == S_OK, "got 0x%08x.\n", hr); + if(SUCCEEDED(hr)) + { + HWND hwnd; + + hr = IOleWindow_ContextSensitiveHelp(pow, TRUE); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IOleWindow_ContextSensitiveHelp(pow, FALSE); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + if(0) + { + /* Crashes on win7 */ + IOleWindow_GetWindow(pow, NULL); + } + + hr = IOleWindow_GetWindow(pow, &hwnd); + ok(hr == S_OK, "Got 0x%08x\n", hr); + ok(hwnd == NULL, "Got %p\n", hwnd); + + IOleWindow_Release(pow); + } + ref = IFileOpenDialog_Release(pfod); ok(!ref, "Got refcount %d, should have been released.\n", ref); @@ -280,6 +306,32 @@ static BOOL test_instantiation(void) ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); if(SUCCEEDED(hr)) IUnknown_Release(punk); + hr = IFileSaveDialog_QueryInterface(pfsd, &IID_IOleWindow, (void**)&pow); + ok(hr == S_OK, "got 0x%08x.\n", hr); + if(SUCCEEDED(hr)) + { + HWND hwnd; + + hr = IOleWindow_ContextSensitiveHelp(pow, TRUE); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IOleWindow_ContextSensitiveHelp(pow, FALSE); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + if(0) + { + /* Crashes on win7 */ + IOleWindow_GetWindow(pow, NULL); + } + + hr = IOleWindow_GetWindow(pow, &hwnd); + ok(hr == S_OK, "Got 0x%08x\n", hr); + ok(hwnd == NULL, "Got %p\n", hwnd); + + IOleWindow_Release(pow); + } + + ref = IFileSaveDialog_Release(pfsd); ok(!ref, "Got refcount %d, should have been released.\n", ref); return TRUE;

1 0

David Hedberg : comdlg32: Support some events in the item dialog.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 1597dec04856174f39f8b127b2f0b51fd7cbbe87 URL: http://source.winehq.org/git/wine.git/?a=commit;h=1597dec04856174f39f8b127b… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:48 2011 +0200 comdlg32: Support some events in the item dialog. --- dlls/comdlg32/itemdlg.c | 94 ++++++++++++++++++++++++++++++++++++++++- dlls/comdlg32/tests/itemdlg.c | 49 ++++++++++++++++++---- 2 files changed, 134 insertions(+), 9 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=1597dec04856174f39f8b…

1 0

David Hedberg : comdlg32: Fix reference counting when querying services.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: 77cc26878c2c4ca3c801af8a642c123c67514daa URL: http://source.winehq.org/git/wine.git/?a=commit;h=77cc26878c2c4ca3c801af8a6… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:47 2011 +0200 comdlg32: Fix reference counting when querying services. --- dlls/comdlg32/itemdlg.c | 8 +------- dlls/comdlg32/tests/itemdlg.c | 21 +++++++++++++++------ 2 files changed, 16 insertions(+), 13 deletions(-) diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c index 8612904..69d9661 100644 --- a/dlls/comdlg32/itemdlg.c +++ b/dlls/comdlg32/itemdlg.c @@ -1915,13 +1915,7 @@ static HRESULT WINAPI IServiceProvider_fnQueryService(IServiceProvider *iface, FIXME("Interface %s requested from unknown service %s\n", debugstr_guid(riid), debugstr_guid(guidService)); - if(SUCCEEDED(hr) && *ppv) - { - IUnknown_AddRef((IUnknown*)*ppv); - return S_OK; - } - - return E_FAIL; + return hr; } static const IServiceProviderVtbl vt_IServiceProvider = { diff --git a/dlls/comdlg32/tests/itemdlg.c b/dlls/comdlg32/tests/itemdlg.c index 3653014..4dafa9b 100644 --- a/dlls/comdlg32/tests/itemdlg.c +++ b/dlls/comdlg32/tests/itemdlg.c @@ -157,6 +157,7 @@ static BOOL test_instantiation(void) IServiceProvider *psp; IUnknown *punk; HRESULT hr; + LONG ref; /* Instantiate FileOpenDialog */ hr = CoCreateInstance(&CLSID_FileOpenDialog, NULL, CLSCTX_INPROC_SERVER, @@ -223,7 +224,8 @@ static BOOL test_instantiation(void) ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); if(SUCCEEDED(hr)) IUnknown_Release(punk); - IFileOpenDialog_Release(pfod); + ref = IFileOpenDialog_Release(pfod); + ok(!ref, "Got refcount %d, should have been released.\n", ref); /* Instantiate FileSaveDialog */ hr = CoCreateInstance(&CLSID_FileSaveDialog, NULL, CLSCTX_INPROC_SERVER, @@ -267,7 +269,8 @@ static BOOL test_instantiation(void) ok(hr == E_NOINTERFACE, "got 0x%08x.\n", hr); if(SUCCEEDED(hr)) IUnknown_Release(punk); - IFileSaveDialog_Release(pfsd); + ref = IFileSaveDialog_Release(pfsd); + ok(!ref, "Got refcount %d, should have been released.\n", ref); return TRUE; } @@ -284,6 +287,7 @@ static void test_basics(void) LPITEMIDLIST pidl; WCHAR *filename; UINT filetype; + LONG ref; HRESULT hr; const WCHAR txt[] = {'t','x','t', 0}; const WCHAR null[] = {0}; @@ -735,8 +739,10 @@ static void test_basics(void) /* Cleanup */ IShellItem_Release(psidesktop); - IFileOpenDialog_Release(pfod); - IFileSaveDialog_Release(pfsd); + ref = IFileOpenDialog_Release(pfod); + ok(!ref, "Got refcount %d, should have been released.\n", ref); + ref = IFileSaveDialog_Release(pfsd); + ok(!ref, "Got refcount %d, should have been released.\n", ref); } static void test_advise_helper(IFileDialog *pfd) @@ -811,20 +817,23 @@ static void test_advise(void) { IFileDialog *pfd; HRESULT hr; + LONG ref; trace("Testing FileOpenDialog (advise)\n"); hr = CoCreateInstance(&CLSID_FileOpenDialog, NULL, CLSCTX_INPROC_SERVER, &IID_IFileDialog, (void**)&pfd); ok(hr == S_OK, "got 0x%08x.\n", hr); test_advise_helper(pfd); - IFileDialog_Release(pfd); + ref = IFileDialog_Release(pfd); + ok(!ref, "Got refcount %d, should have been released.\n", ref); trace("Testing FileSaveDialog (advise)\n"); hr = CoCreateInstance(&CLSID_FileSaveDialog, NULL, CLSCTX_INPROC_SERVER, &IID_IFileDialog, (void**)&pfd); ok(hr == S_OK, "got 0x%08x.\n", hr); test_advise_helper(pfd); - IFileDialog_Release(pfd); + ref = IFileDialog_Release(pfd); + ok(!ref, "Got refcount %d, should have been released.\n", ref); } START_TEST(itemdlg)

1 0

David Hedberg : comdlg32: Change some trace statements to dump strings.
by Alexandre Julliard 25 May '11

25 May '11

Module: wine Branch: master Commit: f539d4c9281cb6042a841201da12775da442c2c8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f539d4c9281cb6042a841201d… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Tue May 24 21:09:46 2011 +0200 comdlg32: Change some trace statements to dump strings. --- dlls/comdlg32/itemdlg.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/comdlg32/itemdlg.c b/dlls/comdlg32/itemdlg.c index 51767a7..8612904 100644 --- a/dlls/comdlg32/itemdlg.c +++ b/dlls/comdlg32/itemdlg.c @@ -1109,7 +1109,7 @@ static HRESULT WINAPI IFileDialog2_fnGetCurrentSelection(IFileDialog2 *iface, IS static HRESULT WINAPI IFileDialog2_fnSetFileName(IFileDialog2 *iface, LPCWSTR pszName) { FileDialogImpl *This = impl_from_IFileDialog2(iface); - TRACE("%p (%p)\n", iface, pszName); + TRACE("%p (%s)\n", iface, debugstr_w(pszName)); set_file_name(This, pszName); @@ -1134,7 +1134,7 @@ static HRESULT WINAPI IFileDialog2_fnGetFileName(IFileDialog2 *iface, LPWSTR *ps static HRESULT WINAPI IFileDialog2_fnSetTitle(IFileDialog2 *iface, LPCWSTR pszTitle) { FileDialogImpl *This = impl_from_IFileDialog2(iface); - TRACE("%p (%p)\n", This, pszTitle); + TRACE("%p (%s)\n", This, debugstr_w(pszTitle)); LocalFree(This->custom_title); This->custom_title = StrDupW(pszTitle); @@ -1146,7 +1146,7 @@ static HRESULT WINAPI IFileDialog2_fnSetTitle(IFileDialog2 *iface, LPCWSTR pszTi static HRESULT WINAPI IFileDialog2_fnSetOkButtonLabel(IFileDialog2 *iface, LPCWSTR pszText) { FileDialogImpl *This = impl_from_IFileDialog2(iface); - TRACE("%p (%p)\n", This, pszText); + TRACE("%p (%s)\n", This, debugstr_w(pszText)); LocalFree(This->custom_okbutton); This->custom_okbutton = StrDupW(pszText); @@ -1159,7 +1159,7 @@ static HRESULT WINAPI IFileDialog2_fnSetOkButtonLabel(IFileDialog2 *iface, LPCWS static HRESULT WINAPI IFileDialog2_fnSetFileNameLabel(IFileDialog2 *iface, LPCWSTR pszLabel) { FileDialogImpl *This = impl_from_IFileDialog2(iface); - TRACE("%p (%p)\n", This, pszLabel); + TRACE("%p (%s)\n", This, debugstr_w(pszLabel)); LocalFree(This->custom_filenamelabel); This->custom_filenamelabel = StrDupW(pszLabel); @@ -1246,7 +1246,7 @@ static HRESULT WINAPI IFileDialog2_fnSetFilter(IFileDialog2 *iface, IShellItemFi static HRESULT WINAPI IFileDialog2_fnSetCancelButtonLabel(IFileDialog2 *iface, LPCWSTR pszLabel) { FileDialogImpl *This = impl_from_IFileDialog2(iface); - TRACE("%p (%p)\n", This, pszLabel); + TRACE("%p (%s)\n", This, debugstr_w(pszLabel)); LocalFree(This->custom_cancelbutton); This->custom_cancelbutton = StrDupW(pszLabel);

1 0

← Newer
1
...
14
15
16
17
18
19
20
...
95
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits May 2011