August 2011 - Wine-Commits - list.winehq.org

Alexandre Julliard : msdaps: Specify the PSFactoryBuffer class in the idl files.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: 9a5141b32590fcdec8a4b96d8822ea4b25072392 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9a5141b32590fcdec8a4b96d8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Aug 1 14:26:36 2011 +0200 msdaps: Specify the PSFactoryBuffer class in the idl files. --- .gitignore | 2 -- dlls/msdaps/Makefile.in | 6 +----- dlls/msdaps/main.c | 1 + dlls/msdaps/msdaps.idl | 6 ++++++ dlls/msdaps/row_server.idl | 7 +++++++ 5 files changed, 15 insertions(+), 7 deletions(-) diff --git a/.gitignore b/.gitignore index ed67e11..8a77f70 100644 --- a/.gitignore +++ b/.gitignore @@ -67,10 +67,8 @@ dlls/libkernel.def dlls/libwinspool.def dlls/libxinput.def dlls/msdaps/msdaps.h -dlls/msdaps/msdaps_i.c dlls/msdaps/msdaps_p.c dlls/msdaps/row_server.h -dlls/msdaps/row_server_i.c dlls/msdaps/row_server_p.c dlls/mshtml.tlb/mshtml_tlb.tlb dlls/mshtml/nsiface.h diff --git a/dlls/msdaps/Makefile.in b/dlls/msdaps/Makefile.in index 01b0b72..3f3d619 100644 --- a/dlls/msdaps/Makefile.in +++ b/dlls/msdaps/Makefile.in @@ -1,6 +1,6 @@ MODULE = msdaps.dll IMPORTS = uuid oleaut32 ole32 rpcrt4 advapi32 -EXTRADEFS = -DWINE_REGISTER_DLL -DPROXY_CLSID_IS="{ 0x06210e88, 0x01f5, 0x11d1, { 0xb5, 0x12, 0x00, 0x80, 0xc7, 0x81, 0xc3, 0x84 } }" -DENTRY_PREFIX=msdaps_ +EXTRADEFS = -DWINE_REGISTER_DLL -DPROXY_CLSID=CLSID_PSFactoryBuffer -DENTRY_PREFIX=msdaps_ EXTRAIDLFLAGS = --win32-align=2 EXTRA_OBJS = dlldata.o @@ -10,10 +10,6 @@ C_SRCS = \ row_server.c \ usrmarshal.c -IDL_I_SRCS = \ - msdaps.idl \ - row_server.idl - IDL_P_SRCS = \ msdaps.idl \ row_server.idl diff --git a/dlls/msdaps/main.c b/dlls/msdaps/main.c index b673b58..14c6c01 100644 --- a/dlls/msdaps/main.c +++ b/dlls/msdaps/main.c @@ -29,6 +29,7 @@ #include "wingdi.h" #include "winuser.h" #include "winerror.h" +#include "initguid.h" #include "objbase.h" #include "oleauto.h" #define DBINITCONSTANTS diff --git a/dlls/msdaps/msdaps.idl b/dlls/msdaps/msdaps.idl index c0a8e11..dbbe75c 100644 --- a/dlls/msdaps/msdaps.idl +++ b/dlls/msdaps/msdaps.idl @@ -1 +1,7 @@ #include "oledb.idl" + +[ + threading(both), + uuid(06210e88-01f5-11d1-b512-0080c781c384) +] +coclass PSFactoryBuffer { interface IFactoryBuffer; } diff --git a/dlls/msdaps/row_server.idl b/dlls/msdaps/row_server.idl index 6fb49b0..afe9463 100644 --- a/dlls/msdaps/row_server.idl +++ b/dlls/msdaps/row_server.idl @@ -20,6 +20,13 @@ import "oledb.idl"; + +[ + threading(both), + uuid(06210e88-01f5-11d1-b512-0080c781c384) +] +coclass PSFactoryBuffer { interface IFactoryBuffer; } + [ object, uuid(38248178-cf6d-11de-abe5-000c2916d865),

1 0

Alexandre Julliard : makefiles: Also install the program manpages from the top-level makefile.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: 41107a940f59302ae93145727e6a7ba590fe7671 URL: http://source.winehq.org/git/wine.git/?a=commit;h=41107a940f59302ae93145727… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Aug 1 13:40:19 2011 +0200 makefiles: Also install the program manpages from the top-level makefile. --- Makefile.in | 3 ++- aclocal.m4 | 9 +++++++++ configure | 49 +++++++++++++++++++++++++++++-------------------- configure.ac | 40 ++++++++++++++++++++-------------------- tools/make_makefiles | 3 ++- 5 files changed, 62 insertions(+), 42 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=41107a940f59302ae9314…

1 0

Jacek Caban : mshtml: Added IHTMLPluginsCollection:: get_length implementation.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: 7d4e9f77929dbb233db6e69e593e91ecb936494e URL: http://source.winehq.org/git/wine.git/?a=commit;h=7d4e9f77929dbb233db6e69e5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sun Jul 31 16:45:08 2011 +0200 mshtml: Added IHTMLPluginsCollection::get_length implementation. --- dlls/mshtml/omnavigator.c | 8 ++++++-- dlls/mshtml/tests/dom.c | 20 ++++++++++++++++++-- 2 files changed, 24 insertions(+), 4 deletions(-) diff --git a/dlls/mshtml/omnavigator.c b/dlls/mshtml/omnavigator.c index 79db598..b4b5c15 100644 --- a/dlls/mshtml/omnavigator.c +++ b/dlls/mshtml/omnavigator.c @@ -145,8 +145,12 @@ static HRESULT WINAPI HTMLPluginsCollection_Invoke(IHTMLPluginsCollection *iface static HRESULT WINAPI HTMLPluginsCollection_get_length(IHTMLPluginsCollection *iface, LONG *p) { HTMLPluginsCollection *This = impl_from_IHTMLPluginsCollection(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, p); + + /* IE always returns 0 here */ + *p = 0; + return S_OK; } static HRESULT WINAPI HTMLPluginsCollection_refresh(IHTMLPluginsCollection *iface, VARIANT_BOOL reload) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index adffac2..08be27d 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -3760,12 +3760,19 @@ static void test_location(IHTMLDocument2 *doc) ok(!ref, "location chould be destroyed here\n"); } -static void test_plugins_col(IOmNavigator *nav) +static void test_plugins_col(IHTMLDocument2 *doc) { IHTMLPluginsCollection *col, *col2; + IHTMLWindow2 *window; + IOmNavigator *nav; ULONG ref; + LONG len; HRESULT hres; + window = get_doc_window(doc); + hres = IHTMLWindow2_get_navigator(window, &nav); + IHTMLWindow2_Release(window); + hres = IOmNavigator_get_plugins(nav, &col); ok(hres == S_OK, "get_plugins failed: %08x\n", hres); @@ -3776,8 +3783,15 @@ static void test_plugins_col(IOmNavigator *nav) test_disp2((IUnknown*)col, &DIID_DispCPlugins, &IID_IHTMLPluginsCollection, "[object]"); + len = 0xdeadbeef; + hres = IHTMLPluginsCollection_get_length(col, &len); + ok(hres == S_OK, "get_length failed: %08x\n", hres); + ok(!len, "length = %d\n", len); + ref = IHTMLPluginsCollection_Release(col); ok(!ref, "ref=%d\n", ref); + + IOmNavigator_Release(nav); } static void test_mime_types_col(IOmNavigator *nav) @@ -3892,7 +3906,6 @@ static void test_navigator(IHTMLDocument2 *doc) skip("nonstandard user agent\n"); } - test_plugins_col(navigator); test_mime_types_col(navigator); ref = IOmNavigator_Release(navigator); @@ -4263,6 +4276,7 @@ static void test_defaults(IHTMLDocument2 *doc) test_compatmode(doc); test_location(doc); test_navigator(doc); + test_plugins_col(doc); elem2 = get_elem2_iface((IUnknown*)elem); hres = IHTMLElement2_get_currentStyle(elem2, &cstyle); @@ -4653,6 +4667,8 @@ static void test_elems(IHTMLDocument2 *doc) IHTMLElementCollection_Release(collection); } + test_plugins_col(doc); + elem = get_doc_elem(doc); test_elem_all((IUnknown*)elem, all_types+1, sizeof(all_types)/sizeof(all_types[0])-1); IHTMLElement_Release(elem);

1 0

Michael Stefaniuc : dxdiagn: COM cleanup for the IClassFactory iface.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: 08031fdc209135bcde10af0b47dae3e9c5c2423d URL: http://source.winehq.org/git/wine.git/?a=commit;h=08031fdc209135bcde10af0b4… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Aug 1 10:39:18 2011 +0200 dxdiagn: COM cleanup for the IClassFactory iface. --- dlls/dxdiagn/dxdiag_main.c | 50 +++++++++++++++++++++---------------------- 1 files changed, 24 insertions(+), 26 deletions(-) diff --git a/dlls/dxdiagn/dxdiag_main.c b/dlls/dxdiagn/dxdiag_main.c index 2d8b0ea..9cfe2ab 100644 --- a/dlls/dxdiagn/dxdiag_main.c +++ b/dlls/dxdiagn/dxdiag_main.c @@ -19,6 +19,8 @@ * */ +#define COBJMACROS + #include "config.h" #include <stdarg.h> @@ -53,39 +55,42 @@ BOOL WINAPI DllMain(HINSTANCE hInstDLL, DWORD fdwReason, LPVOID lpvReserved) * DXDiag ClassFactory */ typedef struct { - const IClassFactoryVtbl *lpVtbl; - REFCLSID rclsid; - HRESULT (*pfnCreateInstanceFactory)(LPCLASSFACTORY iface, LPUNKNOWN punkOuter, REFIID riid, LPVOID *ppobj); + IClassFactory IClassFactory_iface; } IClassFactoryImpl; -static HRESULT WINAPI DXDiagCF_QueryInterface(LPCLASSFACTORY iface,REFIID riid,LPVOID *ppobj) { +static HRESULT WINAPI DXDiagCF_QueryInterface(IClassFactory *iface, REFIID riid, void **ppv) +{ FIXME("- no interface\n\tIID:\t%s\n", debugstr_guid(riid)); - if (ppobj == NULL) return E_POINTER; + if (ppv == NULL) return E_POINTER; return E_NOINTERFACE; } -static ULONG WINAPI DXDiagCF_AddRef(LPCLASSFACTORY iface) { +static ULONG WINAPI DXDiagCF_AddRef(IClassFactory *iface) +{ DXDIAGN_LockModule(); return 2; /* non-heap based object */ } -static ULONG WINAPI DXDiagCF_Release(LPCLASSFACTORY iface) { +static ULONG WINAPI DXDiagCF_Release(IClassFactory * iface) +{ DXDIAGN_UnlockModule(); return 1; /* non-heap based object */ } -static HRESULT WINAPI DXDiagCF_CreateInstance(LPCLASSFACTORY iface,LPUNKNOWN pOuter,REFIID riid,LPVOID *ppobj) { - IClassFactoryImpl *This = (IClassFactoryImpl *)iface; - TRACE("(%p)->(%p,%s,%p)\n",This,pOuter,debugstr_guid(riid),ppobj); - - return This->pfnCreateInstanceFactory(iface, pOuter, riid, ppobj); +static HRESULT WINAPI DXDiagCF_CreateInstance(IClassFactory *iface, IUnknown *pOuter, REFIID riid, + void **ppv) +{ + TRACE("(%p)->(%p,%s,%p)\n", iface, pOuter, debugstr_guid(riid), ppv); + + return DXDiag_CreateDXDiagProvider(iface, pOuter, riid, ppv); } -static HRESULT WINAPI DXDiagCF_LockServer(LPCLASSFACTORY iface,BOOL dolock) { +static HRESULT WINAPI DXDiagCF_LockServer(IClassFactory *iface, BOOL dolock) +{ TRACE("(%d)\n", dolock); if (dolock) @@ -104,10 +109,7 @@ static const IClassFactoryVtbl DXDiagCF_Vtbl = { DXDiagCF_LockServer }; -static IClassFactoryImpl DXDiag_CFS[] = { - { &DXDiagCF_Vtbl, &CLSID_DxDiagProvider, DXDiag_CreateDXDiagProvider }, - { NULL, NULL, NULL } -}; +static IClassFactoryImpl DXDiag_CF = { { &DXDiagCF_Vtbl } }; /*********************************************************************** * DllCanUnloadNow (DXDIAGN.@) @@ -122,16 +124,12 @@ HRESULT WINAPI DllCanUnloadNow(void) */ HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID riid, LPVOID *ppv) { - int i = 0; - TRACE("(%s,%s,%p)\n", debugstr_guid(rclsid), debugstr_guid(riid), ppv); - while (NULL != DXDiag_CFS[i].rclsid) { - if (IsEqualGUID(rclsid, DXDiag_CFS[i].rclsid)) { - DXDiagCF_AddRef((IClassFactory*) &DXDiag_CFS[i]); - *ppv = &DXDiag_CFS[i]; - return S_OK; - } - ++i; + + if (IsEqualGUID(rclsid, &CLSID_DxDiagProvider)) { + IClassFactory_AddRef(&DXDiag_CF.IClassFactory_iface); + *ppv = &DXDiag_CF.IClassFactory_iface; + return S_OK; } FIXME("(%s,%s,%p): no interface found.\n", debugstr_guid(rclsid), debugstr_guid(riid), ppv);

1 0

Michael Stefaniuc : dxdiagn: Implement IClassFactory::QueryInterface.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: a84870259f20cb14762139c28615c30feb1df63f URL: http://source.winehq.org/git/wine.git/?a=commit;h=a84870259f20cb14762139c28… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Aug 1 10:41:56 2011 +0200 dxdiagn: Implement IClassFactory::QueryInterface. --- dlls/dxdiagn/dxdiag_main.c | 19 +++++++++++++++---- 1 files changed, 15 insertions(+), 4 deletions(-) diff --git a/dlls/dxdiagn/dxdiag_main.c b/dlls/dxdiagn/dxdiag_main.c index 9cfe2ab..9f10a60 100644 --- a/dlls/dxdiagn/dxdiag_main.c +++ b/dlls/dxdiagn/dxdiag_main.c @@ -60,11 +60,22 @@ typedef struct { static HRESULT WINAPI DXDiagCF_QueryInterface(IClassFactory *iface, REFIID riid, void **ppv) { - FIXME("- no interface\n\tIID:\t%s\n", debugstr_guid(riid)); + if (ppv == NULL) + return E_POINTER; + + if (IsEqualGUID(&IID_IUnknown, riid)) + TRACE("(%p)->(IID_IUnknown %p)\n", iface, ppv); + else if (IsEqualGUID(&IID_IClassFactory, riid)) + TRACE("(%p)->(IID_IClassFactory %p)\n", iface, ppv); + else { + FIXME("(%p)->(%s %p)\n", iface, debugstr_guid(riid), ppv); + *ppv = NULL; + return E_NOINTERFACE; + } - if (ppv == NULL) return E_POINTER; - - return E_NOINTERFACE; + *ppv = iface; + IClassFactory_AddRef(iface); + return S_OK; } static ULONG WINAPI DXDiagCF_AddRef(IClassFactory *iface)

1 0

Michael Stefaniuc : devenum: COM cleanup for the IClassFactory iface.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: aa591dcd9d45bc0a50066f493092de7cdd822cf8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=aa591dcd9d45bc0a50066f493… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Aug 1 10:34:09 2011 +0200 devenum: COM cleanup for the IClassFactory iface. --- dlls/devenum/devenum_main.c | 2 +- dlls/devenum/devenum_private.h | 2 +- dlls/devenum/factory.c | 23 ++++++++--------------- 3 files changed, 10 insertions(+), 17 deletions(-) diff --git a/dlls/devenum/devenum_main.c b/dlls/devenum/devenum_main.c index 6867fea..e19e1e0 100644 --- a/dlls/devenum/devenum_main.c +++ b/dlls/devenum/devenum_main.c @@ -75,7 +75,7 @@ HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID iid, LPVOID *ppv) * Oh well - works just fine as it is */ if (IsEqualGUID(rclsid, &CLSID_SystemDeviceEnum) || IsEqualGUID(rclsid, &CLSID_CDeviceMoniker)) - return IClassFactory_QueryInterface((IClassFactory*)&DEVENUM_ClassFactory, iid, ppv); + return IClassFactory_QueryInterface(&DEVENUM_ClassFactory.IClassFactory_iface, iid, ppv); FIXME("CLSID: %s, IID: %s\n", debugstr_guid(rclsid), debugstr_guid(iid)); return CLASS_E_CLASSNOTAVAILABLE; diff --git a/dlls/devenum/devenum_private.h b/dlls/devenum/devenum_private.h index 1363e91..7515877 100644 --- a/dlls/devenum/devenum_private.h +++ b/dlls/devenum/devenum_private.h @@ -57,7 +57,7 @@ static inline void DEVENUM_UnlockModule(void) { InterlockedDecrement(&dll_refs); */ typedef struct { - const IClassFactoryVtbl *lpVtbl; + IClassFactory IClassFactory_iface; } ClassFactoryImpl; typedef struct diff --git a/dlls/devenum/factory.c b/dlls/devenum/factory.c index 09861fa..215eb3a 100644 --- a/dlls/devenum/factory.c +++ b/dlls/devenum/factory.c @@ -28,10 +28,8 @@ WINE_DEFAULT_DEBUG_CHANNEL(devenum); /********************************************************************** * DEVENUM_IClassFactory_QueryInterface (also IUnknown) */ -static HRESULT WINAPI DEVENUM_IClassFactory_QueryInterface( - LPCLASSFACTORY iface, - REFIID riid, - LPVOID *ppvObj) +static HRESULT WINAPI DEVENUM_IClassFactory_QueryInterface(IClassFactory *iface, REFIID riid, + void **ppvObj) { TRACE("\n\tIID:\t%s\n",debugstr_guid(riid)); @@ -56,7 +54,7 @@ static HRESULT WINAPI DEVENUM_IClassFactory_QueryInterface( /********************************************************************** * DEVENUM_IClassFactory_AddRef (also IUnknown) */ -static ULONG WINAPI DEVENUM_IClassFactory_AddRef(LPCLASSFACTORY iface) +static ULONG WINAPI DEVENUM_IClassFactory_AddRef(IClassFactory *iface) { TRACE("\n"); @@ -68,7 +66,7 @@ static ULONG WINAPI DEVENUM_IClassFactory_AddRef(LPCLASSFACTORY iface) /********************************************************************** * DEVENUM_IClassFactory_Release (also IUnknown) */ -static ULONG WINAPI DEVENUM_IClassFactory_Release(LPCLASSFACTORY iface) +static ULONG WINAPI DEVENUM_IClassFactory_Release(IClassFactory *iface) { TRACE("\n"); @@ -80,11 +78,8 @@ static ULONG WINAPI DEVENUM_IClassFactory_Release(LPCLASSFACTORY iface) /********************************************************************** * DEVENUM_IClassFactory_CreateInstance */ -static HRESULT WINAPI DEVENUM_IClassFactory_CreateInstance( - LPCLASSFACTORY iface, - LPUNKNOWN pUnkOuter, - REFIID riid, - LPVOID *ppvObj) +static HRESULT WINAPI DEVENUM_IClassFactory_CreateInstance(IClassFactory *iface, + IUnknown *pUnkOuter, REFIID riid, void **ppvObj) { TRACE("\n\tIID:\t%s\n",debugstr_guid(riid)); @@ -110,9 +105,7 @@ static HRESULT WINAPI DEVENUM_IClassFactory_CreateInstance( /********************************************************************** * DEVENUM_IClassFactory_LockServer */ -static HRESULT WINAPI DEVENUM_IClassFactory_LockServer( - LPCLASSFACTORY iface, - BOOL fLock) +static HRESULT WINAPI DEVENUM_IClassFactory_LockServer(IClassFactory *iface, BOOL fLock) { TRACE("\n"); @@ -138,4 +131,4 @@ static const IClassFactoryVtbl IClassFactory_Vtbl = /********************************************************************** * static ClassFactory instance */ -ClassFactoryImpl DEVENUM_ClassFactory = { &IClassFactory_Vtbl }; +ClassFactoryImpl DEVENUM_ClassFactory = { { &IClassFactory_Vtbl } };

1 0

Jacek Caban : mshtml: Added semi-stub implementation of IHTMLWindow2:: status property.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: b25d49c6c16bd20371a5ea77fda477051794d162 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b25d49c6c16bd20371a5ea77f… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sun Jul 31 16:44:43 2011 +0200 mshtml: Added semi-stub implementation of IHTMLWindow2::status property. --- dlls/mshtml/htmlwindow.c | 18 ++++++++++++++---- dlls/mshtml/tests/dom.c | 27 +++++++++++++++++++++++++++ 2 files changed, 41 insertions(+), 4 deletions(-) diff --git a/dlls/mshtml/htmlwindow.c b/dlls/mshtml/htmlwindow.c index ba292ab..465db09 100644 --- a/dlls/mshtml/htmlwindow.c +++ b/dlls/mshtml/htmlwindow.c @@ -491,15 +491,25 @@ static HRESULT WINAPI HTMLWindow2_get_defaultStatus(IHTMLWindow2 *iface, BSTR *p static HRESULT WINAPI HTMLWindow2_put_status(IHTMLWindow2 *iface, BSTR v) { HTMLWindow *This = impl_from_IHTMLWindow2(iface); - FIXME("(%p)->(%s)\n", This, debugstr_w(v)); - return E_NOTIMPL; + + WARN("(%p)->(%s)\n", This, debugstr_w(v)); + + /* + * FIXME: Since IE7, setting status is blocked, but still possible in certain circumstances. + * Ignoring the call should be enough for us. + */ + return S_OK; } static HRESULT WINAPI HTMLWindow2_get_status(IHTMLWindow2 *iface, BSTR *p) { HTMLWindow *This = impl_from_IHTMLWindow2(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, p); + + /* See put_status */ + *p = NULL; + return S_OK; } static HRESULT WINAPI HTMLWindow2_setTimeout(IHTMLWindow2 *iface, BSTR expression, diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index 00fdb14..adffac2 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -1058,6 +1058,31 @@ static void _set_window_name(unsigned line, IHTMLWindow2 *window, const char *na _test_window_name(line, window, name); } +#define test_window_status(d) _test_window_status(__LINE__,d) +static void _test_window_status(unsigned line, IHTMLWindow2 *window) +{ + BSTR status; + HRESULT hres; + + status = (void*)0xdeadbeef; + hres = IHTMLWindow2_get_status(window, &status); + ok_(__FILE__,line)(hres == S_OK, "get_status failed: %08x\n", hres); + ok_(__FILE__,line)(!status, "status = %s\n", wine_dbgstr_w(status)); + SysFreeString(status); +} + +#define set_window_status(w,n) _set_window_status(__LINE__,w,n) +static void _set_window_status(unsigned line, IHTMLWindow2 *window, const char *status) +{ + BSTR str; + HRESULT hres; + + str = a2bstr(status); + hres = IHTMLWindow2_put_status(window, str); + SysFreeString(str); + ok_(__FILE__,line)(hres == S_OK, "put_status failed: %08x\n", hres); +} + #define test_window_length(w,l) _test_window_length(__LINE__,w,l) static void _test_window_length(unsigned line, IHTMLWindow2 *window, LONG exlen) { @@ -4146,6 +4171,8 @@ static void test_window(IHTMLDocument2 *doc) set_window_name(window, "test"); test_window_length(window, 0); test_screen(window); + test_window_status(window); + set_window_status(window, "Test!"); IHTMLWindow2_Release(window); }

1 0

Henri Verbeet : wined3d: Get rid of context_bind_fbo() calls outside context.c.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: 33fafc6166c72624f159824145d1a8d5274ddf9d URL: http://source.winehq.org/git/wine.git/?a=commit;h=33fafc6166c72624f15982414… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Jul 31 17:33:44 2011 +0200 wined3d: Get rid of context_bind_fbo() calls outside context.c. --- dlls/wined3d/context.c | 2 +- dlls/wined3d/surface.c | 37 ++++++++++++++----------------------- dlls/wined3d/swapchain.c | 4 ++-- dlls/wined3d/wined3d_private.h | 1 - 4 files changed, 17 insertions(+), 27 deletions(-) diff --git a/dlls/wined3d/context.c b/dlls/wined3d/context.c index b0bc5f6..620a89b 100644 --- a/dlls/wined3d/context.c +++ b/dlls/wined3d/context.c @@ -33,7 +33,7 @@ static DWORD wined3d_context_tls_idx; /* FBO helper functions */ /* GL locking is done by the caller */ -void context_bind_fbo(struct wined3d_context *context, GLenum target, GLuint *fbo) +static void context_bind_fbo(struct wined3d_context *context, GLenum target, GLuint *fbo) { const struct wined3d_gl_info *gl_info = context->gl_info; GLuint f; diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 9422a0b..7c5b6d9 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -5010,6 +5010,7 @@ static void surface_blt_fbo(struct wined3d_device *device, const WINED3DTEXTUREF struct wined3d_context *context; RECT src_rect, dst_rect; GLenum gl_filter; + GLenum buffer; TRACE("device %p, filter %s,\n", device, debug_d3dtexturefiltertype(filter)); TRACE("src_surface %p, src_location %s, src_rect %s,\n", @@ -5062,48 +5063,38 @@ static void surface_blt_fbo(struct wined3d_device *device, const WINED3DTEXTUREF if (src_location == SFLAG_INDRAWABLE) { - GLenum buffer = surface_get_gl_buffer(src_surface); - TRACE("Source surface %p is onscreen.\n", src_surface); - + buffer = surface_get_gl_buffer(src_surface); surface_translate_drawable_coords(src_surface, context->win_handle, &src_rect); - - ENTER_GL(); - context_bind_fbo(context, GL_READ_FRAMEBUFFER, NULL); - glReadBuffer(buffer); - checkGLcall("glReadBuffer()"); } else { TRACE("Source surface %p is offscreen.\n", src_surface); - ENTER_GL(); - context_apply_fbo_state_blit(context, GL_READ_FRAMEBUFFER, src_surface, NULL, src_location); - glReadBuffer(GL_COLOR_ATTACHMENT0); - checkGLcall("glReadBuffer()"); + buffer = GL_COLOR_ATTACHMENT0; } + + ENTER_GL(); + context_apply_fbo_state_blit(context, GL_READ_FRAMEBUFFER, src_surface, NULL, src_location); + glReadBuffer(buffer); + checkGLcall("glReadBuffer()"); context_check_fbo_status(context, GL_READ_FRAMEBUFFER); LEAVE_GL(); if (dst_location == SFLAG_INDRAWABLE) { - GLenum buffer = surface_get_gl_buffer(dst_surface); - TRACE("Destination surface %p is onscreen.\n", dst_surface); - + buffer = surface_get_gl_buffer(dst_surface); surface_translate_drawable_coords(dst_surface, context->win_handle, &dst_rect); - - ENTER_GL(); - context_bind_fbo(context, GL_DRAW_FRAMEBUFFER, NULL); - context_set_draw_buffer(context, buffer); } else { TRACE("Destination surface %p is offscreen.\n", dst_surface); - - ENTER_GL(); - context_apply_fbo_state_blit(context, GL_DRAW_FRAMEBUFFER, dst_surface, NULL, dst_location); - context_set_draw_buffer(context, GL_COLOR_ATTACHMENT0); + buffer = GL_COLOR_ATTACHMENT0; } + + ENTER_GL(); + context_apply_fbo_state_blit(context, GL_DRAW_FRAMEBUFFER, dst_surface, NULL, dst_location); + context_set_draw_buffer(context, buffer); context_check_fbo_status(context, GL_DRAW_FRAMEBUFFER); context_invalidate_state(context, STATE_FRAMEBUFFER); diff --git a/dlls/wined3d/swapchain.c b/dlls/wined3d/swapchain.c index a7e61fb..5ad6966 100644 --- a/dlls/wined3d/swapchain.c +++ b/dlls/wined3d/swapchain.c @@ -306,7 +306,7 @@ static void swapchain_blit(const struct wined3d_swapchain *swapchain, glReadBuffer(GL_COLOR_ATTACHMENT0); context_check_fbo_status(context, GL_READ_FRAMEBUFFER); - context_bind_fbo(context, GL_DRAW_FRAMEBUFFER, NULL); + context_apply_fbo_state_blit(context, GL_DRAW_FRAMEBUFFER, swapchain->front_buffer, NULL, SFLAG_INDRAWABLE); context_set_draw_buffer(context, GL_BACK); context_invalidate_state(context, STATE_FRAMEBUFFER); @@ -350,7 +350,7 @@ static void swapchain_blit(const struct wined3d_swapchain *swapchain, gl_filter = GL_NEAREST; ENTER_GL(); - context_bind_fbo(context2, GL_FRAMEBUFFER, NULL); + context_apply_fbo_state_blit(context2, GL_FRAMEBUFFER, swapchain->front_buffer, NULL, SFLAG_INDRAWABLE); /* Set up the texture. The surface is not in a wined3d_texture * container, so there are no D3D texture settings to dirtify. */ diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index fbf8641..17e8b41 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1233,7 +1233,6 @@ BOOL context_apply_clear_state(struct wined3d_context *context, struct wined3d_d BOOL context_apply_draw_state(struct wined3d_context *context, struct wined3d_device *device) DECLSPEC_HIDDEN; void context_apply_fbo_state_blit(struct wined3d_context *context, GLenum target, struct wined3d_surface *render_target, struct wined3d_surface *depth_stencil, DWORD location) DECLSPEC_HIDDEN; -void context_bind_fbo(struct wined3d_context *context, GLenum target, GLuint *fbo) DECLSPEC_HIDDEN; void context_check_fbo_status(const struct wined3d_context *context, GLenum target) DECLSPEC_HIDDEN; struct wined3d_context *context_create(struct wined3d_swapchain *swapchain, struct wined3d_surface *target, const struct wined3d_format *ds_format) DECLSPEC_HIDDEN;

1 0

Andrew Talbot : riched20: Declare some variables unsigned.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: faa05713ad4cb299b8d2c35337d5513a54a36d55 URL: http://source.winehq.org/git/wine.git/?a=commit;h=faa05713ad4cb299b8d2c3533… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Sun Jul 31 21:13:13 2011 +0100 riched20: Declare some variables unsigned. --- dlls/riched20/run.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/riched20/run.c b/dlls/riched20/run.c index 3682d13..4a9ae7a 100644 --- a/dlls/riched20/run.c +++ b/dlls/riched20/run.c @@ -894,8 +894,8 @@ void ME_GetCharFormat(ME_TextEditor *editor, const ME_Cursor *from, do { /* FIXME add more style feature comparisons */ - int nAttribs = CFM_SIZE | CFM_FACE | CFM_COLOR | CFM_UNDERLINETYPE; - int nEffects = CFM_BOLD | CFM_ITALIC | CFM_UNDERLINE | CFM_STRIKEOUT | CFM_PROTECTED | CFM_LINK | CFM_SUPERSCRIPT; + DWORD dwAttribs = CFM_SIZE | CFM_FACE | CFM_COLOR | CFM_UNDERLINETYPE; + DWORD dwEffects = CFM_BOLD | CFM_ITALIC | CFM_UNDERLINE | CFM_STRIKEOUT | CFM_PROTECTED | CFM_LINK | CFM_SUPERSCRIPT; run = ME_FindItemFwd(run, diRun); @@ -903,7 +903,7 @@ void ME_GetCharFormat(ME_TextEditor *editor, const ME_Cursor *from, tmp.cbSize = sizeof(tmp); ME_GetRunCharFormat(editor, run, &tmp); - assert((tmp.dwMask & nAttribs) == nAttribs); + assert((tmp.dwMask & dwAttribs) == dwAttribs); /* reset flags that differ */ if (pFmt->yHeight != tmp.yHeight) @@ -929,7 +929,7 @@ void ME_GetCharFormat(ME_TextEditor *editor, const ME_Cursor *from, } } - pFmt->dwMask &= ~((pFmt->dwEffects ^ tmp.dwEffects) & nEffects); + pFmt->dwMask &= ~((pFmt->dwEffects ^ tmp.dwEffects) & dwEffects); pFmt->dwEffects = tmp.dwEffects; } while(run != run_end);

1 0

Andrew Talbot : mscms: Move a misplaced brace.
by Alexandre Julliard Aug. 1, 2011

Aug. 1, 2011

Module: wine Branch: master Commit: 322f3037677198b66cc87e790beeceded246023f URL: http://source.winehq.org/git/wine.git/?a=commit;h=322f3037677198b66cc87e790… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Sun Jul 31 21:13:08 2011 +0100 mscms: Move a misplaced brace. --- dlls/mscms/profile.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/mscms/profile.c b/dlls/mscms/profile.c index c9a5b42..49fc24b 100644 --- a/dlls/mscms/profile.c +++ b/dlls/mscms/profile.c @@ -660,9 +660,9 @@ BOOL WINAPI GetStandardColorSpaceProfileW( PCWSTR machine, DWORD id, PWSTR profi GetColorDirectoryW( machine, rgbprofile, &len ); switch (id) + { case LCS_sRGB: case LCS_WINDOWS_COLOR_SPACE: /* FIXME */ - { lstrcatW( rgbprofile, rgbprofilefile ); len = lstrlenW( rgbprofile ) * sizeof(WCHAR);

1 0