December 2010 - Wine-Commits

Greg Geldorp : shdocvw/tests: Need to save . url before committing extra props for IE6.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: e2ce3cffe31fc79614f4ecd648ae444e1af055e7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e2ce3cffe31fc79614f4ecd64… Author: Greg Geldorp <ggeldorp(a)vmware.com> Date: Thu Dec 23 13:19:40 2010 +0100 shdocvw/tests: Need to save .url before committing extra props for IE6. --- dlls/shdocvw/intshcut.c | 8 +++++++- dlls/shdocvw/tests/intshcut.c | 34 +++++++++++++++++++--------------- 2 files changed, 26 insertions(+), 16 deletions(-) diff --git a/dlls/shdocvw/intshcut.c b/dlls/shdocvw/intshcut.c index 9820fec..6658c5d 100644 --- a/dlls/shdocvw/intshcut.c +++ b/dlls/shdocvw/intshcut.c @@ -610,7 +610,13 @@ static HRESULT WINAPI PersistFile_Save(IPersistFile *pFile, LPCOLESTR pszFileNam if SUCCEEDED(hr) { hr = IPropertyStorage_ReadMultiple(pPropStgRead, 2, ps, pvread); - if SUCCEEDED(hr) + if (hr == S_FALSE) + { + /* None of the properties are present, that's ok */ + hr = S_OK; + IPropertyStorage_Release(pPropStgRead); + } + else if SUCCEEDED(hr) { char indexString[50]; len = WideCharToMultiByte(CP_UTF8, 0, pvread[0].u.pwszVal, -1, NULL, 0, 0, 0); diff --git a/dlls/shdocvw/tests/intshcut.c b/dlls/shdocvw/tests/intshcut.c index c046601..360876e 100644 --- a/dlls/shdocvw/tests/intshcut.c +++ b/dlls/shdocvw/tests/intshcut.c @@ -200,6 +200,20 @@ static void test_ReadAndWriteProperties(void) IPropertyStorage *pPropStgWrite; IPropertySetStorage *pPropSetStg; PROPVARIANT pv[2]; + + /* We need to set a URL -- IPersistFile refuses to save without one. */ + hr = urlA->lpVtbl->SetURL(urlA, testurl, 0); + ok(hr == S_OK, "Failed to set a URL. hr=0x%x\n", hr); + + /* Write this shortcut out to a file so that we can test reading it in again. */ + hr = urlA->lpVtbl->QueryInterface(urlA, &IID_IPersistFile, (void **) &pf); + ok(hr == S_OK, "Failed to get the IPersistFile for writing. hr=0x%x\n", hr); + + hr = IPersistFile_Save(pf, fileNameW, TRUE); + ok(hr == S_OK, "Failed to save via IPersistFile. hr=0x%x\n", hr); + + IPersistFile_Release(pf); + pv[0].vt = VT_LPWSTR; pv[0].pwszVal = (void *) iconPath; pv[1].vt = VT_I4; @@ -217,19 +231,6 @@ static void test_ReadAndWriteProperties(void) ok(hr == S_OK, "Failed to commit properties, hr=0x%x\n", hr); pPropStgWrite->lpVtbl->Release(pPropStgWrite); - - /* We need to set a URL -- IPersistFile refuses to save without one. */ - hr = urlA->lpVtbl->SetURL(urlA, testurl, 0); - ok(hr == S_OK, "Failed to set a URL. hr=0x%x\n", hr); - - /* Write this shortcut out to a file so that we can test reading it in again. */ - hr = urlA->lpVtbl->QueryInterface(urlA, &IID_IPersistFile, (void **) &pf); - ok(hr == S_OK, "Failed to get the IPersistFile for writing. hr=0x%x\n", hr); - - hr = IPersistFile_Save(pf, fileNameW, TRUE); - ok(hr == S_OK, "Failed to save via IPersistFile. hr=0x%x\n", hr); - - IPersistFile_Release(pf); urlA->lpVtbl->Release(urlA); IPropertySetStorage_Release(pPropSetStg); } @@ -267,9 +268,12 @@ static void test_ReadAndWriteProperties(void) hr = IPropertyStorage_ReadMultiple(pPropStgRead, 2, ps, pvread); ok(hr == S_OK, "Unable to read properties, hr=0x%x\n", hr); - ok(pvread[1].iVal == iconIndex, "Read wrong icon index: %d\n", pvread[1].iVal); + todo_wine /* Wine doesn't yet support setting properties after save */ + { + ok(pvread[1].iVal == iconIndex, "Read wrong icon index: %d\n", pvread[1].iVal); - ok(lstrcmpW(pvread[0].pwszVal, iconPath) == 0, "Wrong icon path read: %s\n",wine_dbgstr_w(pvread[0].pwszVal)); + ok(lstrcmpW(pvread[0].pwszVal, iconPath) == 0, "Wrong icon path read: %s\n",wine_dbgstr_w(pvread[0].pwszVal)); + } PropVariantClear(&pvread[0]); PropVariantClear(&pvread[1]);

1 0

David Hedberg : shell32: Add IInputObject stub to ExplorerBrowser.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: 35a45d8190c7efd72ceb7a43edb34d4445fc62f2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=35a45d8190c7efd72ceb7a43e… Author: David Hedberg <david.hedberg(a)gmail.com> Date: Thu Dec 23 14:52:30 2010 +0100 shell32: Add IInputObject stub to ExplorerBrowser. --- dlls/shell32/ebrowser.c | 67 +++++++++++++++++++++++++ dlls/shell32/tests/ebrowser.c | 109 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 176 insertions(+), 0 deletions(-) diff --git a/dlls/shell32/ebrowser.c b/dlls/shell32/ebrowser.c index 50e5a4f..ea44337 100644 --- a/dlls/shell32/ebrowser.c +++ b/dlls/shell32/ebrowser.c @@ -58,6 +58,7 @@ typedef struct _ExplorerBrowserImpl { ICommDlgBrowser3 ICommDlgBrowser3_iface; IObjectWithSite IObjectWithSite_iface; INameSpaceTreeControlEvents INameSpaceTreeControlEvents_iface; + IInputObject IInputObject_iface; LONG ref; BOOL destroyed; @@ -790,6 +791,10 @@ static HRESULT WINAPI IExplorerBrowser_fnQueryInterface(IExplorerBrowser *iface, { *ppvObject = &This->IObjectWithSite_iface; } + else if(IsEqualIID(riid, &IID_IInputObject)) + { + *ppvObject = &This->IInputObject_iface; + } if(*ppvObject) { @@ -1971,6 +1976,67 @@ const INameSpaceTreeControlEventsVtbl vt_INameSpaceTreeControlEvents = { NSTCEvents_fnOnGetDefaultIconIndex }; +/************************************************************************** + * IInputObject Implementation + */ + +static inline ExplorerBrowserImpl *impl_from_IInputObject(IInputObject *iface) +{ + return CONTAINING_RECORD(iface, ExplorerBrowserImpl, IInputObject_iface); +} + +static HRESULT WINAPI IInputObject_fnQueryInterface(IInputObject *iface, + REFIID riid, void **ppvObject) +{ + ExplorerBrowserImpl *This = impl_from_IInputObject(iface); + TRACE("%p\n", This); + return IUnknown_QueryInterface((IUnknown*)This, riid, ppvObject); +} + +static ULONG WINAPI IInputObject_fnAddRef(IInputObject *iface) +{ + ExplorerBrowserImpl *This = impl_from_IInputObject(iface); + TRACE("%p\n", This); + return IUnknown_AddRef((IUnknown*)This); +} + +static ULONG WINAPI IInputObject_fnRelease(IInputObject *iface) +{ + ExplorerBrowserImpl *This = impl_from_IInputObject(iface); + TRACE("%p\n", This); + return IUnknown_Release((IUnknown*)This); +} + +static HRESULT WINAPI IInputObject_fnUIActivateIO(IInputObject *iface, BOOL fActivate, MSG *pMsg) +{ + ExplorerBrowserImpl *This = impl_from_IInputObject(iface); + FIXME("stub, %p (%d, %p)\n", This, fActivate, pMsg); + return E_NOTIMPL; +} + +static HRESULT WINAPI IInputObject_fnHasFocusIO(IInputObject *iface) +{ + ExplorerBrowserImpl *This = impl_from_IInputObject(iface); + FIXME("stub, %p\n", This); + return E_NOTIMPL; +} + +static HRESULT WINAPI IInputObject_fnTranslateAcceleratorIO(IInputObject *iface, MSG *pMsg) +{ + ExplorerBrowserImpl *This = impl_from_IInputObject(iface); + FIXME("stub, %p (%p)\n", This, pMsg); + return E_NOTIMPL; +} + +static IInputObjectVtbl vt_IInputObject = { + IInputObject_fnQueryInterface, + IInputObject_fnAddRef, + IInputObject_fnRelease, + IInputObject_fnUIActivateIO, + IInputObject_fnHasFocusIO, + IInputObject_fnTranslateAcceleratorIO +}; + HRESULT WINAPI ExplorerBrowser_Constructor(IUnknown *pUnkOuter, REFIID riid, void **ppv) { ExplorerBrowserImpl *eb; @@ -1990,6 +2056,7 @@ HRESULT WINAPI ExplorerBrowser_Constructor(IUnknown *pUnkOuter, REFIID riid, voi eb->ICommDlgBrowser3_iface.lpVtbl = &vt_ICommDlgBrowser3; eb->IObjectWithSite_iface.lpVtbl = &vt_IObjectWithSite; eb->INameSpaceTreeControlEvents_iface.lpVtbl = &vt_INameSpaceTreeControlEvents; + eb->IInputObject_iface.lpVtbl = &vt_IInputObject; /* Default settings */ eb->navpane.width = 150; diff --git a/dlls/shell32/tests/ebrowser.c b/dlls/shell32/tests/ebrowser.c index 15c2cb4..a221f97 100644 --- a/dlls/shell32/tests/ebrowser.c +++ b/dlls/shell32/tests/ebrowser.c @@ -1607,6 +1607,114 @@ static void test_GetCurrentView(void) IExplorerBrowser_Release(peb); } +static void test_InputObject(void) +{ + IExplorerBrowser *peb; + IShellFolder *psf; + IInputObject *pio; + HRESULT hr; + RECT rc; + UINT i; + WPARAM supported_key_accels_mode1[] = { + VK_BACK, VK_TAB, VK_RETURN, VK_PRIOR, VK_NEXT, VK_END, VK_HOME, + VK_LEFT, VK_UP, VK_RIGHT, VK_DOWN, VK_DELETE, VK_F1, VK_F2, + VK_F5, VK_F6, VK_F10, 0 }; + WPARAM supported_key_accels_mode2[] = { + VK_RETURN, VK_PRIOR, VK_NEXT, VK_END, VK_HOME, + VK_LEFT, VK_UP, VK_RIGHT, VK_DOWN, VK_DELETE, VK_F1, VK_F2, + VK_F10, 0 }; + WPARAM *key_accels; + MSG msg_a = { + hwnd, + WM_KEYDOWN, + VK_F5, 0, + GetTickCount(), + {5, 2} + }; + + ebrowser_instantiate(&peb); + hr = IExplorerBrowser_QueryInterface(peb, &IID_IInputObject, (void**)&pio); + ok(hr == S_OK, "Got 0x%08x\n", hr); + if(FAILED(hr)) + { + win_skip("IInputObject not supported.\n"); + return; + } + + /* Before initializing */ + hr = IInputObject_TranslateAcceleratorIO(pio, &msg_a); + todo_wine ok(hr == E_FAIL, "Got 0x%08x\n", hr); + + hr = IInputObject_HasFocusIO(pio); + todo_wine ok(hr == E_FAIL, "Got 0x%08x\n", hr); + + hr = IInputObject_UIActivateIO(pio, TRUE, &msg_a); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IInputObject_HasFocusIO(pio); + todo_wine ok(hr == E_FAIL, "Got 0x%08x\n", hr); + + hr = IInputObject_TranslateAcceleratorIO(pio, &msg_a); + todo_wine ok(hr == E_FAIL, "Got 0x%08x\n", hr); + + rc.left = 0; rc.top = 0; rc.right = 100; rc.bottom = 100; + hr = IExplorerBrowser_Initialize(peb, hwnd, &rc, NULL); + ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IInputObject_HasFocusIO(pio); + todo_wine ok(hr == E_FAIL, "Got 0x%08x\n", hr); + + hr = IInputObject_TranslateAcceleratorIO(pio, &msg_a); + todo_wine ok(hr == E_FAIL, "Got 0x%08x\n", hr); + + /* Browse to the desktop */ + SHGetDesktopFolder(&psf); + hr = IExplorerBrowser_BrowseToObject(peb, (IUnknown*)psf, SBSP_DEFBROWSER); + ok(hr == S_OK, "Got 0x%08x\n", hr); + IShellFolder_Release(psf); + + hr = IInputObject_UIActivateIO(pio, TRUE, &msg_a); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IInputObject_HasFocusIO(pio); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IInputObject_UIActivateIO(pio, FALSE, &msg_a); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IInputObject_HasFocusIO(pio); + todo_wine ok(hr == S_OK, "Got 0x%08x\n", hr); + + hr = IInputObject_TranslateAcceleratorIO(pio, &msg_a); + if(hr == S_OK) + key_accels = supported_key_accels_mode1; + else + key_accels = supported_key_accels_mode2; + + for(i = 0; i < 0x100; i++) + { + BOOL found = FALSE; + UINT j; + for(j = 0; key_accels[j] != 0; j++) + if(key_accels[j] == i) + { + found = TRUE; + break; + } + + msg_a.wParam = i; + process_msgs(); + hr = IInputObject_TranslateAcceleratorIO(pio, &msg_a); + todo_wine ok(hr == (found ? S_OK : S_FALSE), "Got 0x%08x (%04x)\n", hr, i); + } + + process_msgs(); + + IInputObject_Release(pio); + IExplorerBrowser_Destroy(peb); + IExplorerBrowser_Release(peb); +} + static BOOL test_instantiate_control(void) { IExplorerBrowser *peb; @@ -1658,6 +1766,7 @@ START_TEST(ebrowser) test_navigation(); test_GetCurrentView(); test_SetSite(); + test_InputObject(); DestroyWindow(hwnd); OleUninitialize();

1 0

Michael Stefaniuc : ole32: Use ifaces instead of vtbl pointers in DataCache .
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: feba7c37b417c76c63f1fa0375b89fd551f58c5c URL: http://source.winehq.org/git/wine.git/?a=commit;h=feba7c37b417c76c63f1fa037… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Dec 23 02:05:04 2010 +0100 ole32: Use ifaces instead of vtbl pointers in DataCache. --- dlls/ole32/datacache.c | 75 +++++++++++++++++++++++------------------------ 1 files changed, 37 insertions(+), 38 deletions(-) diff --git a/dlls/ole32/datacache.c b/dlls/ole32/datacache.c index 768dec7..9b32da6 100644 --- a/dlls/ole32/datacache.c +++ b/dlls/ole32/datacache.c @@ -121,18 +121,18 @@ typedef struct DataCacheEntry struct DataCache { /* - * List all interface VTables here + * List all interface here */ - const IDataObjectVtbl* lpVtbl; - const IUnknownVtbl* lpvtblNDIUnknown; - const IPersistStorageVtbl* lpvtblIPersistStorage; - const IViewObject2Vtbl* lpvtblIViewObject; - const IOleCache2Vtbl* lpvtblIOleCache2; - const IOleCacheControlVtbl* lpvtblIOleCacheControl; + IDataObject IDataObject_iface; + IUnknown IUnknown_iface; + IPersistStorage IPersistStorage_iface; + IViewObject2 IViewObject2_iface; + IOleCache2 IOleCache2_iface; + IOleCacheControl IOleCacheControl_iface; /* The sink that is connected to a remote object. The other interfaces are not available by QI'ing the sink and vice-versa */ - const IAdviseSinkVtbl* lpvtblIAdviseSink; + IAdviseSink IAdviseSink_iface; /* * Reference count of this object @@ -175,37 +175,37 @@ typedef struct DataCache DataCache; static inline DataCache *impl_from_IDataObject( IDataObject *iface ) { - return (DataCache *)((char*)iface - FIELD_OFFSET(DataCache, lpVtbl)); + return CONTAINING_RECORD(iface, DataCache, IDataObject_iface); } -static inline DataCache *impl_from_NDIUnknown( IUnknown *iface ) +static inline DataCache *impl_from_IUnknown( IUnknown *iface ) { - return (DataCache *)((char*)iface - FIELD_OFFSET(DataCache, lpvtblNDIUnknown)); + return CONTAINING_RECORD(iface, DataCache, IUnknown_iface); } static inline DataCache *impl_from_IPersistStorage( IPersistStorage *iface ) { - return (DataCache *)((char*)iface - FIELD_OFFSET(DataCache, lpvtblIPersistStorage)); + return CONTAINING_RECORD(iface, DataCache, IPersistStorage_iface); } static inline DataCache *impl_from_IViewObject2( IViewObject2 *iface ) { - return (DataCache *)((char*)iface - FIELD_OFFSET(DataCache, lpvtblIViewObject)); + return CONTAINING_RECORD(iface, DataCache, IViewObject2_iface); } static inline DataCache *impl_from_IOleCache2( IOleCache2 *iface ) { - return (DataCache *)((char*)iface - FIELD_OFFSET(DataCache, lpvtblIOleCache2)); + return CONTAINING_RECORD(iface, DataCache, IOleCache2_iface); } static inline DataCache *impl_from_IOleCacheControl( IOleCacheControl *iface ) { - return (DataCache *)((char*)iface - FIELD_OFFSET(DataCache, lpvtblIOleCacheControl)); + return CONTAINING_RECORD(iface, DataCache, IOleCacheControl_iface); } static inline DataCache *impl_from_IAdviseSink( IAdviseSink *iface ) { - return (DataCache *)((char*)iface - FIELD_OFFSET(DataCache, lpvtblIAdviseSink)); + return CONTAINING_RECORD(iface, DataCache, IAdviseSink_iface); } static const char * debugstr_formatetc(const FORMATETC *formatetc) @@ -879,7 +879,7 @@ static HRESULT WINAPI DataCache_NDIUnknown_QueryInterface( REFIID riid, void** ppvObject) { - DataCache *this = impl_from_NDIUnknown(iface); + DataCache *this = impl_from_IUnknown(iface); if ( ppvObject==0 ) return E_INVALIDARG; @@ -892,26 +892,26 @@ static HRESULT WINAPI DataCache_NDIUnknown_QueryInterface( } else if (IsEqualIID(&IID_IDataObject, riid)) { - *ppvObject = &this->lpVtbl; + *ppvObject = &this->IDataObject_iface; } else if ( IsEqualIID(&IID_IPersistStorage, riid) || IsEqualIID(&IID_IPersist, riid) ) { - *ppvObject = &this->lpvtblIPersistStorage; + *ppvObject = &this->IPersistStorage_iface; } else if ( IsEqualIID(&IID_IViewObject, riid) || IsEqualIID(&IID_IViewObject2, riid) ) { - *ppvObject = &this->lpvtblIViewObject; + *ppvObject = &this->IViewObject2_iface; } else if ( IsEqualIID(&IID_IOleCache, riid) || IsEqualIID(&IID_IOleCache2, riid) ) { - *ppvObject = &this->lpvtblIOleCache2; + *ppvObject = &this->IOleCache2_iface; } else if ( IsEqualIID(&IID_IOleCacheControl, riid) ) { - *ppvObject = &this->lpvtblIOleCacheControl; + *ppvObject = &this->IOleCacheControl_iface; } if ((*ppvObject)==0) @@ -934,7 +934,7 @@ static HRESULT WINAPI DataCache_NDIUnknown_QueryInterface( static ULONG WINAPI DataCache_NDIUnknown_AddRef( IUnknown* iface) { - DataCache *this = impl_from_NDIUnknown(iface); + DataCache *this = impl_from_IUnknown(iface); return InterlockedIncrement(&this->ref); } @@ -947,7 +947,7 @@ static ULONG WINAPI DataCache_NDIUnknown_AddRef( static ULONG WINAPI DataCache_NDIUnknown_Release( IUnknown* iface) { - DataCache *this = impl_from_NDIUnknown(iface); + DataCache *this = impl_from_IUnknown(iface); ULONG ref; ref = InterlockedDecrement(&this->ref); @@ -1882,7 +1882,7 @@ static HRESULT setup_sink(DataCache *This, DataCacheEntry *cache_entry) if(This->running_object) if(!(flags & ADVF_NODATA)) hr = IDataObject_DAdvise(This->running_object, &cache_entry->fmtetc, flags, - (IAdviseSink *)&This->lpvtblIAdviseSink, &cache_entry->sink_id); + &This->IAdviseSink_iface, &cache_entry->sink_id); return hr; } @@ -2012,8 +2012,7 @@ static HRESULT WINAPI DataCache_DiscardCache( TRACE("(%d)\n", dwDiscardOptions); if (dwDiscardOptions == DISCARDCACHE_SAVEIFDIRTY) - hr = DataCache_Save((IPersistStorage *)&This->lpvtblIPersistStorage, - This->presentationStorage, TRUE); + hr = DataCache_Save(&This->IPersistStorage_iface, This->presentationStorage, TRUE); LIST_FOR_EACH_ENTRY(cache_entry, &This->cache_list, DataCacheEntry, entry) { @@ -2152,7 +2151,7 @@ static void WINAPI DataCache_OnDataChange(IAdviseSink *iface, FORMATETC *fmt, ST { DataCache *This = impl_from_IAdviseSink(iface); TRACE("(%p)->(%s, %p)\n", This, debugstr_formatetc(fmt), med); - IOleCache_SetData((IOleCache2*)&This->lpvtblIOleCache2, fmt, med, FALSE); + IOleCache_SetData(&This->IOleCache2_iface, fmt, med, FALSE); } static void WINAPI DataCache_OnViewChange(IAdviseSink *iface, DWORD aspect, LONG index) @@ -2284,13 +2283,13 @@ static DataCache* DataCache_Construct( /* * Initialize the virtual function table. */ - newObject->lpVtbl = &DataCache_IDataObject_VTable; - newObject->lpvtblNDIUnknown = &DataCache_NDIUnknown_VTable; - newObject->lpvtblIPersistStorage = &DataCache_IPersistStorage_VTable; - newObject->lpvtblIViewObject = &DataCache_IViewObject2_VTable; - newObject->lpvtblIOleCache2 = &DataCache_IOleCache2_VTable; - newObject->lpvtblIOleCacheControl = &DataCache_IOleCacheControl_VTable; - newObject->lpvtblIAdviseSink = &DataCache_IAdviseSink_VTable; + newObject->IDataObject_iface.lpVtbl = &DataCache_IDataObject_VTable; + newObject->IUnknown_iface.lpVtbl = &DataCache_NDIUnknown_VTable; + newObject->IPersistStorage_iface.lpVtbl = &DataCache_IPersistStorage_VTable; + newObject->IViewObject2_iface.lpVtbl = &DataCache_IViewObject2_VTable; + newObject->IOleCache2_iface.lpVtbl = &DataCache_IOleCache2_VTable; + newObject->IOleCacheControl_iface.lpVtbl = &DataCache_IOleCacheControl_VTable; + newObject->IAdviseSink_iface.lpVtbl = &DataCache_IAdviseSink_VTable; /* * Start with one reference count. The caller of this function @@ -2305,7 +2304,7 @@ static DataCache* DataCache_Construct( * lifetime. */ if (pUnkOuter==NULL) - pUnkOuter = (IUnknown*)&(newObject->lpvtblNDIUnknown); + pUnkOuter = &newObject->IUnknown_iface; newObject->outerUnknown = pUnkOuter; @@ -2385,13 +2384,13 @@ HRESULT WINAPI CreateDataCache( /* * Make sure it supports the interface required by the caller. */ - hr = IUnknown_QueryInterface((IUnknown*)&(newCache->lpvtblNDIUnknown), riid, ppvObj); + hr = IUnknown_QueryInterface(&newCache->IUnknown_iface, riid, ppvObj); /* * Release the reference obtained in the constructor. If * the QueryInterface was unsuccessful, it will free the class. */ - IUnknown_Release((IUnknown*)&(newCache->lpvtblNDIUnknown)); + IUnknown_Release(&newCache->IUnknown_iface); return hr; }

1 0

Michael Stefaniuc : ole32: Standardize the COM usage in defaulthandler.c.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: 91ecb05f29c845cfc65225ddd9e3cd5d76cec5af URL: http://source.winehq.org/git/wine.git/?a=commit;h=91ecb05f29c845cfc65225ddd… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Dec 23 02:05:43 2010 +0100 ole32: Standardize the COM usage in defaulthandler.c. --- dlls/ole32/defaulthandler.c | 87 ++++++++++++++++++++++--------------------- 1 files changed, 45 insertions(+), 42 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=91ecb05f29c845cfc6522…

1 0

Alexandre Julliard : server: Use the syscall function instead of inline assembly.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: 0ae38162640084c432c92397c76a2c38dbd257ed URL: http://source.winehq.org/git/wine.git/?a=commit;h=0ae38162640084c432c92397c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 23 16:10:12 2010 +0100 server: Use the syscall function instead of inline assembly. --- server/change.c | 27 +++------------------------ server/fd.c | 24 +++--------------------- 2 files changed, 6 insertions(+), 45 deletions(-) diff --git a/server/change.c b/server/change.c index 2bdc444..5eb1df6 100644 --- a/server/change.c +++ b/server/change.c @@ -98,38 +98,17 @@ struct inotify_event { static inline int inotify_init( void ) { - int ret; - __asm__ __volatile__( "int $0x80" - : "=a" (ret) - : "0" (SYS_inotify_init)); - if (ret<0) { errno = -ret; ret = -1; } - return ret; + return syscall( SYS_inotify_init ); } static inline int inotify_add_watch( int fd, const char *name, unsigned int mask ) { - int ret; - __asm__ __volatile__( "pushl %%ebx;\n\t" - "movl %2,%%ebx;\n\t" - "int $0x80;\n\t" - "popl %%ebx" - : "=a" (ret) : "0" (SYS_inotify_add_watch), - "r" (fd), "c" (name), "d" (mask) ); - if (ret<0) { errno = -ret; ret = -1; } - return ret; + return syscall( SYS_inotify_add_watch, fd, name, mask ); } static inline int inotify_rm_watch( int fd, int wd ) { - int ret; - __asm__ __volatile__( "pushl %%ebx;\n\t" - "movl %2,%%ebx;\n\t" - "int $0x80;\n\t" - "popl %%ebx" - : "=a" (ret) : "0" (SYS_inotify_rm_watch), - "r" (fd), "c" (wd) ); - if (ret<0) { errno = -ret; ret = -1; } - return ret; + return syscall( SYS_inotify_rm_watch, fd, wd ); } #define USE_INOTIFY diff --git a/server/fd.c b/server/fd.c index bc1d853..e0ec708 100644 --- a/server/fd.c +++ b/server/fd.c @@ -127,38 +127,20 @@ struct epoll_event epoll_data_t data; }; -#define SYSCALL_RET(ret) do { \ - if (ret < 0) { errno = -ret; ret = -1; } \ - return ret; \ - } while(0) - static inline int epoll_create( int size ) { - int ret; - __asm__( "pushl %%ebx; movl %2,%%ebx; int $0x80; popl %%ebx" - : "=a" (ret) : "0" (254 /*NR_epoll_create*/), "r" (size) ); - SYSCALL_RET(ret); + return syscall( 254 /*NR_epoll_create*/, size ); } static inline int epoll_ctl( int epfd, int op, int fd, const struct epoll_event *event ) { - int ret; - __asm__( "pushl %%ebx; movl %2,%%ebx; int $0x80; popl %%ebx" - : "=a" (ret) - : "0" (255 /*NR_epoll_ctl*/), "r" (epfd), "c" (op), "d" (fd), "S" (event), "m" (*event) ); - SYSCALL_RET(ret); + return syscall( 255 /*NR_epoll_ctl*/, epfd, op, fd, event ); } static inline int epoll_wait( int epfd, struct epoll_event *events, int maxevents, int timeout ) { - int ret; - __asm__( "pushl %%ebx; movl %2,%%ebx; int $0x80; popl %%ebx" - : "=a" (ret) - : "0" (256 /*NR_epoll_wait*/), "r" (epfd), "c" (events), "d" (maxevents), "S" (timeout) - : "memory" ); - SYSCALL_RET(ret); + return syscall( 256 /*NR_epoll_wait*/, epfd, events, maxevents, timeout ); } -#undef SYSCALL_RET #endif /* linux && __i386__ && HAVE_STDINT_H */

1 0

Alexandre Julliard : ntdll: Use the syscall function instead of inline assembly.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: 37ae7a533c8df69789d4a3b3c66502179eacaca5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=37ae7a533c8df69789d4a3b3c… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 23 16:10:18 2010 +0100 ntdll: Use the syscall function instead of inline assembly. --- dlls/ntdll/directory.c | 12 +----------- 1 files changed, 1 insertions(+), 11 deletions(-) diff --git a/dlls/ntdll/directory.c b/dlls/ntdll/directory.c index 865c2fa..0cfa85d 100644 --- a/dlls/ntdll/directory.c +++ b/dlls/ntdll/directory.c @@ -112,17 +112,7 @@ typedef struct static inline int getdents64( int fd, char *de, unsigned int size ) { - int ret; - __asm__( "pushl %%ebx; movl %2,%%ebx; int $0x80; popl %%ebx" - : "=a" (ret) - : "0" (220 /*NR_getdents64*/), "r" (fd), "c" (de), "d" (size) - : "memory" ); - if (ret < 0) - { - errno = -ret; - ret = -1; - } - return ret; + return syscall( 220 /* NR_getdents64 */, fd, de, size ); } #define USE_GETDENTS

1 0

Alexandre Julliard : libwine: Use the syscall function instead of inline assembly.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: 378df94ef8f4fa5ca254c1aca408283342b25b16 URL: http://source.winehq.org/git/wine.git/?a=commit;h=378df94ef8f4fa5ca254c1aca… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Dec 23 17:07:35 2010 +0100 libwine: Use the syscall function instead of inline assembly. --- libs/wine/ldt.c | 26 ++------------------------ 1 files changed, 2 insertions(+), 24 deletions(-) diff --git a/libs/wine/ldt.c b/libs/wine/ldt.c index d56790f..c4dcb0e 100644 --- a/libs/wine/ldt.c +++ b/libs/wine/ldt.c @@ -70,34 +70,12 @@ static inline void fill_modify_ldt_struct( struct modify_ldt_s *ptr, const LDT_E static inline int modify_ldt( int func, struct modify_ldt_s *ptr, unsigned long count ) { - int res; - __asm__ __volatile__( "pushl %%ebx\n\t" - "movl %2,%%ebx\n\t" - "int $0x80\n\t" - "popl %%ebx" - : "=a" (res) - : "0" (SYS_modify_ldt), - "r" (func), - "c" (ptr), - "d" (count), - "m" (*ptr) ); - if (res >= 0) return res; - errno = -res; - return -1; + return syscall( SYS_modify_ldt, func, ptr, count ); } static inline int set_thread_area( struct modify_ldt_s *ptr ) { - int res; - __asm__ __volatile__( "pushl %%ebx\n\t" - "movl %3,%%ebx\n\t" - "int $0x80\n\t" - "popl %%ebx" - : "=a" (res), "=m" (*ptr) - : "0" (243) /* SYS_set_thread_area */, "q" (ptr), "m" (*ptr) ); - if (res >= 0) return res; - errno = -res; - return -1; + return syscall( 243 /* SYS_set_thread_area */, ptr ); } #endif /* linux */

1 0

André Hentschel : advapi32: Print a FIXME also for the unknown levels.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: c6682ef5dc1200844874309909f0de61754afb3c URL: http://source.winehq.org/git/wine.git/?a=commit;h=c6682ef5dc120084487430990… Author: André Hentschel <nerv(a)dawncrow.de> Date: Wed Dec 22 21:10:10 2010 +0100 advapi32: Print a FIXME also for the unknown levels. --- dlls/advapi32/service.c | 10 +++------- 1 files changed, 3 insertions(+), 7 deletions(-) diff --git a/dlls/advapi32/service.c b/dlls/advapi32/service.c index 92790c1..96b0848 100644 --- a/dlls/advapi32/service.c +++ b/dlls/advapi32/service.c @@ -1406,6 +1406,8 @@ cleanup: /****************************************************************************** * QueryServiceConfig2W [ADVAPI32.@] + * + * See QueryServiceConfig2A. */ BOOL WINAPI QueryServiceConfig2W(SC_HANDLE hService, DWORD dwLevel, LPBYTE buffer, DWORD size, LPDWORD needed) @@ -1413,13 +1415,7 @@ BOOL WINAPI QueryServiceConfig2W(SC_HANDLE hService, DWORD dwLevel, LPBYTE buffe DWORD err; if(dwLevel != SERVICE_CONFIG_DESCRIPTION) { - if((dwLevel == SERVICE_CONFIG_DELAYED_AUTO_START_INFO) || - (dwLevel == SERVICE_CONFIG_FAILURE_ACTIONS) || - (dwLevel == SERVICE_CONFIG_FAILURE_ACTIONS_FLAG) || - (dwLevel == SERVICE_CONFIG_PRESHUTDOWN_INFO) || - (dwLevel == SERVICE_CONFIG_REQUIRED_PRIVILEGES_INFO) || - (dwLevel == SERVICE_CONFIG_SERVICE_SID_INFO)) - FIXME("Level %d not implemented\n", dwLevel); + FIXME("Level %d not implemented\n", dwLevel); SetLastError(ERROR_INVALID_LEVEL); return FALSE; }

1 0

Michael Stefaniuc : mshtml: COM cleanup for the IOleInPlaceActiveObject iface.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: b09dc2e5b144876e6e87b1a1ac554c6fa82e2458 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b09dc2e5b144876e6e87b1a1a… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Thu Dec 23 02:20:05 2010 +0100 mshtml: COM cleanup for the IOleInPlaceActiveObject iface. --- dlls/mshtml/htmldoc.c | 4 ++-- dlls/mshtml/mshtml_private.h | 4 +--- dlls/mshtml/olecmd.c | 5 +++-- dlls/mshtml/olewnd.c | 33 +++++++++++++++++---------------- dlls/mshtml/view.c | 8 +++++--- 5 files changed, 28 insertions(+), 26 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b09dc2e5b144876e6e87b…

1 0

Hwang YunSong : iccvid: Update Korean resource.
by Alexandre Julliard Dec. 23, 2010

Dec. 23, 2010

Module: wine Branch: master Commit: 36cbe780ec30b116628f6837d73b22523229aad0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=36cbe780ec30b116628f6837d… Author: Hwang YunSong <hys545(a)dreamwiz.com> Date: Thu Dec 23 16:39:56 2010 +0900 iccvid: Update Korean resource. --- dlls/iccvid/iccvid_Ko.rc | 8 +++++--- 1 files changed, 5 insertions(+), 3 deletions(-) diff --git a/dlls/iccvid/iccvid_Ko.rc b/dlls/iccvid/iccvid_Ko.rc index 1d1ba86..e4f6069 100644 --- a/dlls/iccvid/iccvid_Ko.rc +++ b/dlls/iccvid/iccvid_Ko.rc @@ -1,6 +1,6 @@ /* * Copyright 2005 Dmitry Timoshkov - * Copyright 2005 YunSomg Hwang + * Copyright 2005,2010 YunSomg Hwang * * This library is free software; you can redistribute it and/or * modify it under the terms of the GNU Lesser General Public @@ -19,10 +19,12 @@ #include "iccvid_private.h" +#pragma code_page(65001) + LANGUAGE LANG_KOREAN, SUBLANG_DEFAULT STRINGTABLE { - IDS_NAME "�ó�� ڵ�" - IDS_DESCRIPTION "�ó�� ڵ�" + IDS_NAME "시네팩 비디오 코덱" + IDS_DESCRIPTION "시네팩 비디오 코덱" }

1 0