February 2011 - Wine-commits

Rico Schüller : d3dcompiler/tests: Add reflection ps desc test.
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: d4798d0580cae7ab195f7fdf8080127816707f9a URL: http://source.winehq.org/git/wine.git/?a=commit;h=d4798d0580cae7ab195f7fdf8… Author: Rico Schüller <kgbricola(a)web.de> Date: Wed Feb 16 20:28:00 2011 +0100 d3dcompiler/tests: Add reflection ps desc test. --- dlls/d3dcompiler_43/tests/reflection.c | 301 ++++++++++++++++++++++++++++++++ 1 files changed, 301 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=d4798d0580cae7ab195f7…

1 0

André Hentschel : kernel32: Forward RtlCompareMemory to ntdll on x86_64.
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: 21332ccb850db1cd01715406f28a13c478fa6739 URL: http://source.winehq.org/git/wine.git/?a=commit;h=21332ccb850db1cd01715406f… Author: André Hentschel <nerv(a)dawncrow.de> Date: Wed Feb 16 21:20:32 2011 +0100 kernel32: Forward RtlCompareMemory to ntdll on x86_64. --- dlls/kernel32/kernel32.spec | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/kernel32.spec b/dlls/kernel32/kernel32.spec index c2f7bc2..d2f3a26 100644 --- a/dlls/kernel32/kernel32.spec +++ b/dlls/kernel32/kernel32.spec @@ -941,6 +941,7 @@ @ cdecl -arch=x86_64 RtlAddFunctionTable(ptr long long) ntdll.RtlAddFunctionTable @ stdcall -register RtlCaptureContext(ptr) ntdll.RtlCaptureContext @ stdcall RtlCaptureStackBackTrace(long long ptr ptr) ntdll.RtlCaptureStackBackTrace +@ stdcall -arch=x86_64 RtlCompareMemory(ptr ptr long) ntdll.RtlCompareMemory @ stdcall RtlFillMemory(ptr long long) ntdll.RtlFillMemory @ stdcall -arch=x86_64 RtlLookupFunctionEntry(long ptr ptr) ntdll.RtlLookupFunctionEntry @ stdcall RtlMoveMemory(ptr ptr long) ntdll.RtlMoveMemory

1 0

Nikolay Sivov : msxml3: Null pointer for schema uri should be treated as empty.
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: edeff310b12072b957baf4a049408d28a22b4ccf URL: http://source.winehq.org/git/wine.git/?a=commit;h=edeff310b12072b957baf4a04… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Thu Feb 17 01:42:15 2011 +0300 msxml3: Null pointer for schema uri should be treated as empty. --- dlls/msxml3/schema.c | 18 ++++++++++-------- dlls/msxml3/tests/schema.c | 36 ++++++++++++++++++++++++++++++++++++ 2 files changed, 46 insertions(+), 8 deletions(-) diff --git a/dlls/msxml3/schema.c b/dlls/msxml3/schema.c index fa4208d..9239326 100644 --- a/dlls/msxml3/schema.c +++ b/dlls/msxml3/schema.c @@ -63,11 +63,13 @@ static const xmlChar XDR_schema[] = "Schema"; static const xmlChar XDR_nsURI[] = "urn:schemas-microsoft-com:xml-data"; static const xmlChar DT_nsURI[] = "urn:schemas-microsoft-com:datatypes"; -static xmlChar const* datatypes_src = NULL; -static int datatypes_len = 0; -static HGLOBAL datatypes_handle = NULL; -static HRSRC datatypes_rsrc = NULL; -static xmlSchemaPtr datatypes_schema = NULL; +static xmlChar const* datatypes_src; +static int datatypes_len; +static HGLOBAL datatypes_handle; +static HRSRC datatypes_rsrc; +static xmlSchemaPtr datatypes_schema; + +static const WCHAR emptyW[] = {0}; /* Supported Types: * msxml3 - XDR only @@ -1057,7 +1059,7 @@ static HRESULT WINAPI schema_cache_Invoke(IXMLDOMSchemaCollection2* iface, static HRESULT WINAPI schema_cache_add(IXMLDOMSchemaCollection2* iface, BSTR uri, VARIANT var) { schema_cache* This = impl_from_IXMLDOMSchemaCollection2(iface); - xmlChar* name = xmlChar_from_wchar(uri); + xmlChar* name = uri ? xmlChar_from_wchar(uri) : xmlChar_from_wchar(emptyW); TRACE("(%p)->(%s, var(vt %x))\n", This, debugstr_w(uri), V_VT(&var)); switch (V_VT(&var)) @@ -1158,7 +1160,7 @@ static HRESULT WINAPI schema_cache_get(IXMLDOMSchemaCollection2* iface, BSTR uri if (!node) return E_POINTER; - name = xmlChar_from_wchar(uri); + name = uri ? xmlChar_from_wchar(uri) : xmlChar_from_wchar(emptyW); entry = (cache_entry*) xmlHashLookup(This->cache, name); heap_free(name); @@ -1173,7 +1175,7 @@ static HRESULT WINAPI schema_cache_get(IXMLDOMSchemaCollection2* iface, BSTR uri static HRESULT WINAPI schema_cache_remove(IXMLDOMSchemaCollection2* iface, BSTR uri) { schema_cache* This = impl_from_IXMLDOMSchemaCollection2(iface); - xmlChar* name = xmlChar_from_wchar(uri); + xmlChar* name = uri ? xmlChar_from_wchar(uri) : xmlChar_from_wchar(emptyW); TRACE("(%p)->(%s)\n", This, wine_dbgstr_w(uri)); xmlHashRemoveEntry(This->cache, name, cache_free); diff --git a/dlls/msxml3/tests/schema.c b/dlls/msxml3/tests/schema.c index 2b20603..56a77e6 100644 --- a/dlls/msxml3/tests/schema.c +++ b/dlls/msxml3/tests/schema.c @@ -493,11 +493,14 @@ static void* _create_object(const GUID *clsid, const char *name, const IID *iid, static void test_schema_refs(void) { + static const WCHAR emptyW[] = {0}; IXMLDOMDocument2 *doc; + IXMLDOMNode *node; IXMLDOMSchemaCollection *cache; VARIANT v; VARIANT_BOOL b; BSTR str; + LONG len; doc = create_document(&IID_IXMLDOMDocument2); if (!doc) @@ -516,6 +519,39 @@ static void test_schema_refs(void) ok(b == VARIANT_TRUE, "b %04x\n", b); SysFreeString(str); + node = (void*)0xdeadbeef; + ole_check(IXMLDOMSchemaCollection_get(cache, NULL, &node)); + ok(node == NULL, "%p\n", node); + + /* NULL uri pointer, still adds a document */ + ole_check(IXMLDOMSchemaCollection_add(cache, NULL, _variantdoc_(doc))); + len = -1; + ole_check(IXMLDOMSchemaCollection_get_length(cache, &len)); + ok(len == 1, "got %d\n", len); + /* read back - empty valid BSTR */ + str = NULL; + ole_check(IXMLDOMSchemaCollection_get_namespaceURI(cache, 0, &str)); + ok(str && *str == 0, "got %p\n", str); + SysFreeString(str); + + node = NULL; + ole_check(IXMLDOMSchemaCollection_get(cache, NULL, &node)); + ok(node != NULL, "%p\n", node); + IXMLDOMNode_Release(node); + + node = NULL; + str = SysAllocString(emptyW); + ole_check(IXMLDOMSchemaCollection_get(cache, str, &node)); + ok(node != NULL, "%p\n", node); + IXMLDOMNode_Release(node); + SysFreeString(str); + + /* remove with NULL uri */ + ole_check(IXMLDOMSchemaCollection_remove(cache, NULL)); + len = -1; + ole_check(IXMLDOMSchemaCollection_get_length(cache, &len)); + ok(len == 0, "got %d\n", len); + str = SysAllocString(xdr_schema_uri); ole_check(IXMLDOMSchemaCollection_add(cache, str, _variantdoc_(doc)));

1 0

Alexandre Julliard : cabinet: Create one temp file per folder and avoid an extra copy of the data.
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: 682d9a292926691e4f86331bbc393c69c5d6f0c1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=682d9a292926691e4f86331bb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 17 16:48:19 2011 +0100 cabinet: Create one temp file per folder and avoid an extra copy of the data. --- dlls/cabinet/fci.c | 673 +++++++++++++++++----------------------------------- 1 files changed, 219 insertions(+), 454 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=682d9a292926691e4f863…

1 0

Juan Lang : crypt32/tests: Remove an unused assignment (clang).
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: d4748065b310458dde4cac2de89aac3a7428e4a3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d4748065b310458dde4cac2de… Author: Juan Lang <juan.lang(a)gmail.com> Date: Tue Feb 15 10:54:09 2011 -0800 crypt32/tests: Remove an unused assignment (clang). --- dlls/crypt32/tests/message.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/crypt32/tests/message.c b/dlls/crypt32/tests/message.c index 1fe9a57..be3a03b 100644 --- a/dlls/crypt32/tests/message.c +++ b/dlls/crypt32/tests/message.c @@ -124,7 +124,7 @@ static void test_verify_detached_message_hash(void) if (0) { - ret = CryptVerifyDetachedMessageHash(NULL, NULL, 0, 0, NULL, NULL, NULL, + CryptVerifyDetachedMessageHash(NULL, NULL, 0, 0, NULL, NULL, NULL, NULL); } memset(&para, 0, sizeof(para));

1 0

Juan Lang : crypt32/tests: Test a return value (clang).
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: bcace6fb7c70034d8bbd009be809745b007d585c URL: http://source.winehq.org/git/wine.git/?a=commit;h=bcace6fb7c70034d8bbd009be… Author: Juan Lang <juan.lang(a)gmail.com> Date: Tue Feb 15 11:08:36 2011 -0800 crypt32/tests: Test a return value (clang). --- dlls/crypt32/tests/store.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/crypt32/tests/store.c b/dlls/crypt32/tests/store.c index 521fc48..db31190 100644 --- a/dlls/crypt32/tests/store.c +++ b/dlls/crypt32/tests/store.c @@ -2271,6 +2271,8 @@ static void testAddCertificateLink(void) { ret = CertGetCertificateContextProperty(linked, CERT_FRIENDLY_NAME_PROP_ID, buf, &size); + ok(ret, "CertGetCertificateContextProperty failed: %08x\n", + GetLastError()); ok(!lstrcmpW((LPCWSTR)buf, WineTestW), "unexpected friendly name\n"); HeapFree(GetProcessHeap(), 0, buf);

1 0

Juan Lang : crypt32/tests: Test return value (clang).
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: 04f74d7e43336afd62feb918a9c2ee7e9fab8d6d URL: http://source.winehq.org/git/wine.git/?a=commit;h=04f74d7e43336afd62feb918a… Author: Juan Lang <juan.lang(a)gmail.com> Date: Tue Feb 15 11:07:19 2011 -0800 crypt32/tests: Test return value (clang). --- dlls/crypt32/tests/msg.c | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/dlls/crypt32/tests/msg.c b/dlls/crypt32/tests/msg.c index de55d6b..ddb94bd 100644 --- a/dlls/crypt32/tests/msg.c +++ b/dlls/crypt32/tests/msg.c @@ -853,6 +853,8 @@ static void test_hash_msg_get_param(void) ok(ret, "CryptMsgGetParam failed: %08x\n", GetLastError()); ok(size == sizeof(buf), "Unexpected size %d\n", size); ret = CryptMsgGetParam(msg, CMSG_COMPUTED_HASH_PARAM, 0, buf, &size); + ok(ret, "CryptMsgGetParam failed: %08x\n", GetLastError()); + ok(size == sizeof(buf), "Unexpected size %d\n", size); if (size == sizeof(buf)) ok(!memcmp(buf, emptyHashParam, size), "Unexpected value\n"); /* By getting the hash, further updates are not allowed */

1 0

Austin English : d3dx9_36/tests: Free memory after use (Valgrind).
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: e8a2fa28ea78039248a9aa5f0223e8d37977db53 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e8a2fa28ea78039248a9aa5f0… Author: Austin English <austinenglish(a)gmail.com> Date: Wed Feb 16 16:37:14 2011 -0800 d3dx9_36/tests: Free memory after use (Valgrind). --- dlls/d3dx9_36/tests/mesh.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/d3dx9_36/tests/mesh.c b/dlls/d3dx9_36/tests/mesh.c index 5d1425b..05d68eb 100644 --- a/dlls/d3dx9_36/tests/mesh.c +++ b/dlls/d3dx9_36/tests/mesh.c @@ -1417,6 +1417,7 @@ static void D3DXCreateBoxTest(void) end: IDirect3DDevice9_Release(device); IDirect3D9_Release(d3d); + ID3DXBuffer_Release(ppBuffer); DestroyWindow(wnd); }

1 0

Alexandre Julliard : kernel32: Don' t try to initialize curses if the TERM variable is not set.
by Alexandre Julliard 17 Feb '11

17 Feb '11

Module: wine Branch: master Commit: b0cebda36b0ad3aeacfe6a87394d024b0a6939d3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b0cebda36b0ad3aeacfe6a873… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 16 23:30:32 2011 +0100 kernel32: Don't try to initialize curses if the TERM variable is not set. --- dlls/kernel32/term.c | 1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/dlls/kernel32/term.c b/dlls/kernel32/term.c index df0719a..18eb4b2 100644 --- a/dlls/kernel32/term.c +++ b/dlls/kernel32/term.c @@ -415,6 +415,7 @@ BOOL TERM_Init(void) { /* if we're not attached to a tty, don't fire the curses support */ if (!isatty(0) || !isatty(1)) return FALSE; + if (!getenv("TERM")) return FALSE; if (!TERM_bind_libcurses()) return FALSE; if (setupterm(NULL, 1 /* really ?? */, NULL) == -1) return FALSE; TERM_init_done = TRUE;

1 0

Nikolay Sivov : msxml3: Basic put_input() method for IXSLProcessor.
by Alexandre Julliard 16 Feb '11

16 Feb '11

Module: wine Branch: master Commit: 392934af901223ae5f0a0d1e83d69c270721ddc8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=392934af901223ae5f0a0d1e8… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Feb 16 16:44:04 2011 +0300 msxml3: Basic put_input() method for IXSLProcessor. --- dlls/msxml3/stylesheet.c | 40 ++++++++++++++++++++++++++++++++++++++-- dlls/msxml3/tests/domdoc.c | 10 ++++++++++ 2 files changed, 48 insertions(+), 2 deletions(-) diff --git a/dlls/msxml3/stylesheet.c b/dlls/msxml3/stylesheet.c index 1add7db..6d7507e 100644 --- a/dlls/msxml3/stylesheet.c +++ b/dlls/msxml3/stylesheet.c @@ -49,6 +49,8 @@ typedef struct _xslprocessor { IXSLProcessor IXSLProcessor_iface; LONG ref; + + IXMLDOMNode *input; } xslprocessor; static HRESULT XSLProcessor_create(IXSLProcessor**); @@ -302,7 +304,10 @@ static ULONG WINAPI xslprocessor_Release( IXSLProcessor *iface ) ref = InterlockedDecrement( &This->ref ); if ( ref == 0 ) + { + if (This->input) IXMLDOMNode_Release(This->input); heap_free( This ); + } return ref; } @@ -381,9 +386,39 @@ static HRESULT WINAPI xslprocessor_Invoke( static HRESULT WINAPI xslprocessor_put_input( IXSLProcessor *iface, VARIANT input ) { xslprocessor *This = impl_from_IXSLProcessor( iface ); + IXMLDOMNode *input_node; + HRESULT hr; - FIXME("(%p): stub\n", This); - return E_NOTIMPL; + TRACE("(%p)->(%s)\n", This, debugstr_variant(&input)); + + /* try IXMLDOMNode directly first */ + if (V_VT(&input) == VT_UNKNOWN) + hr = IUnknown_QueryInterface(V_UNKNOWN(&input), &IID_IXMLDOMNode, (void**)&input_node); + else if (V_VT(&input) == VT_DISPATCH) + hr = IDispatch_QueryInterface(V_DISPATCH(&input), &IID_IXMLDOMNode, (void**)&input_node); + else + { + IXMLDOMDocument *doc; + + hr = DOMDocument_create(&CLSID_DOMDocument, NULL, (void**)&doc); + if (hr == S_OK) + { + VARIANT_BOOL b; + + hr = IXMLDOMDocument_load(doc, input, &b); + if (hr == S_OK) + hr = IXMLDOMDocument_QueryInterface(doc, &IID_IXMLDOMNode, (void**)&input_node); + IXMLDOMDocument_Release(doc); + } + } + + if (hr == S_OK) + { + if (This->input) IXMLDOMNode_Release(This->input); + This->input = input_node; + } + + return hr; } static HRESULT WINAPI xslprocessor_get_input( IXSLProcessor *iface, VARIANT *input ) @@ -554,6 +589,7 @@ HRESULT XSLProcessor_create(IXSLProcessor **ppObj) This->IXSLProcessor_iface.lpVtbl = &xslprocessor_vtbl; This->ref = 1; + This->input = NULL; *ppObj = &This->IXSLProcessor_iface; diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index b4dc2e1..0c8fff2 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -7666,6 +7666,7 @@ static void test_xsltemplate(void) VARIANT_BOOL b; HRESULT hr; ULONG ref1, ref2; + VARIANT v; template = create_xsltemplate(&IID_IXSLTemplate); if (!template) return; @@ -7724,6 +7725,15 @@ static void test_xsltemplate(void) hr = IXSLTemplate_createProcessor(template, &processor); ok(hr == S_OK, "got 0x%08x\n", hr); + + /* input no set yet */ + V_VT(&v) = VT_BSTR; + V_BSTR(&v) = NULL; + hr = IXSLProcessor_get_input(processor, &v); +todo_wine { + ok(hr == S_OK, "got 0x%08x\n", hr); + ok(V_VT(&v) == VT_EMPTY, "got %d\n", V_VT(&v)); +} IXSLProcessor_Release(processor); /* drop reference */

1 0