September 2010 - Wine-Commits

Gerald Pfeifer : urlmon/tests: Simplify test_IUriBuilder_GetPorttest_IUriBuilder_GetPort a bit.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: 9387fe19a0a874b888bd7c67cb051eca9d7ad4a9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9387fe19a0a874b888bd7c67c… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Sat Sep 4 23:56:38 2010 +0200 urlmon/tests: Simplify test_IUriBuilder_GetPorttest_IUriBuilder_GetPort a bit. --- dlls/urlmon/tests/uri.c | 2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/dlls/urlmon/tests/uri.c b/dlls/urlmon/tests/uri.c index 09cc3b6..9206ce0 100644 --- a/dlls/urlmon/tests/uri.c +++ b/dlls/urlmon/tests/uri.c @@ -6445,13 +6445,11 @@ static void test_IUriBuilder_GetPort(IUriBuilder *builder, const uri_builder_tes } if(SUCCEEDED(hr)) { DWORD expected; - BOOL got_port = FALSE; hr = IUri_GetPort(uri, &expected); ok(SUCCEEDED(hr), "Error: Expected IUri_Port to succeed, but got 0x%08x instead on uri_builder_tests[%d].\n", hr, test_index); - got_port = hr == S_OK; hr = IUriBuilder_GetPort(builder, &has_port, &received); todo_wine {

1 0

Jacek Caban : msxml3: Added node_query_inerface function for common xmlnode child objects implementation .
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: 0670d092f137da408fe2f3ed0119a1a08ea1a932 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0670d092f137da408fe2f3ed0… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat Sep 4 17:54:46 2010 +0200 msxml3: Added node_query_inerface function for common xmlnode child objects implementation. --- dlls/msxml3/attribute.c | 4 ++++ dlls/msxml3/cdata.c | 4 ++++ dlls/msxml3/comment.c | 4 ++++ dlls/msxml3/docfrag.c | 4 ++++ dlls/msxml3/domdoc.c | 2 +- dlls/msxml3/element.c | 2 +- dlls/msxml3/entityref.c | 4 ++++ dlls/msxml3/msxml_private.h | 1 + dlls/msxml3/node.c | 11 +++++++++++ dlls/msxml3/pi.c | 4 ++++ dlls/msxml3/text.c | 4 ++++ 11 files changed, 42 insertions(+), 2 deletions(-) diff --git a/dlls/msxml3/attribute.c b/dlls/msxml3/attribute.c index 327f91c..3554859 100644 --- a/dlls/msxml3/attribute.c +++ b/dlls/msxml3/attribute.c @@ -67,6 +67,10 @@ static HRESULT WINAPI domattr_QueryInterface( { *ppvObject = IXMLDOMNode_from_impl(&This->node); } + else if(node_query_interface(&This->node, riid, ppvObject)) + { + return *ppvObject ? S_OK : E_NOINTERFACE; + } else { FIXME("Unsupported interface %s\n", debugstr_guid(riid)); diff --git a/dlls/msxml3/cdata.c b/dlls/msxml3/cdata.c index b23d19d..e378295 100644 --- a/dlls/msxml3/cdata.c +++ b/dlls/msxml3/cdata.c @@ -74,6 +74,10 @@ static HRESULT WINAPI domcdata_QueryInterface( TRACE("Unsupported interface\n"); return E_NOINTERFACE; } + else if(node_query_interface(&This->node, riid, ppvObject)) + { + return *ppvObject ? S_OK : E_NOINTERFACE; + } else { FIXME("Unsupported interface %s\n", debugstr_guid(riid)); diff --git a/dlls/msxml3/comment.c b/dlls/msxml3/comment.c index 6e7b77e..f496625 100644 --- a/dlls/msxml3/comment.c +++ b/dlls/msxml3/comment.c @@ -68,6 +68,10 @@ static HRESULT WINAPI domcomment_QueryInterface( { *ppvObject = IXMLDOMNode_from_impl(&This->node); } + else if(node_query_interface(&This->node, riid, ppvObject)) + { + return *ppvObject ? S_OK : E_NOINTERFACE; + } else { FIXME("Unsupported interface %s\n", debugstr_guid(riid)); diff --git a/dlls/msxml3/docfrag.c b/dlls/msxml3/docfrag.c index 9ca63d3..b013deb 100644 --- a/dlls/msxml3/docfrag.c +++ b/dlls/msxml3/docfrag.c @@ -67,6 +67,10 @@ static HRESULT WINAPI domfrag_QueryInterface( { *ppvObject = IXMLDOMNode_from_impl(&This->node); } + else if(node_query_interface(&This->node, riid, ppvObject)) + { + return *ppvObject ? S_OK : E_NOINTERFACE; + } else { FIXME("Unsupported interface %s\n", debugstr_guid(riid)); diff --git a/dlls/msxml3/domdoc.c b/dlls/msxml3/domdoc.c index 3f01329..8fdf5c1 100644 --- a/dlls/msxml3/domdoc.c +++ b/dlls/msxml3/domdoc.c @@ -545,7 +545,7 @@ static HRESULT WINAPI domdoc_QueryInterface( IXMLDOMDocument3 *iface, REFIID rii { *ppvObject = &This->lpvtblISupportErrorInfo; } - else if(dispex_query_interface(&This->node.dispex, riid, ppvObject)) + else if(node_query_interface(&This->node, riid, ppvObject)) { return *ppvObject ? S_OK : E_NOINTERFACE; } diff --git a/dlls/msxml3/element.c b/dlls/msxml3/element.c index 5b4a604..6247269 100644 --- a/dlls/msxml3/element.c +++ b/dlls/msxml3/element.c @@ -74,7 +74,7 @@ static HRESULT WINAPI domelem_QueryInterface( { *ppvObject = IXMLDOMNode_from_impl(&This->node); } - else if(dispex_query_interface(&This->node.dispex, riid, ppvObject)) + else if(node_query_interface(&This->node, riid, ppvObject)) { return *ppvObject ? S_OK : E_NOINTERFACE; } diff --git a/dlls/msxml3/entityref.c b/dlls/msxml3/entityref.c index c55fd18..396db9b 100644 --- a/dlls/msxml3/entityref.c +++ b/dlls/msxml3/entityref.c @@ -67,6 +67,10 @@ static HRESULT WINAPI entityref_QueryInterface( { *ppvObject = IXMLDOMNode_from_impl(&This->node); } + else if(node_query_interface(&This->node, riid, ppvObject)) + { + return *ppvObject ? S_OK : E_NOINTERFACE; + } else { FIXME("Unsupported interface %s\n", debugstr_guid(riid)); diff --git a/dlls/msxml3/msxml_private.h b/dlls/msxml3/msxml_private.h index fe843b8..e8251bb 100644 --- a/dlls/msxml3/msxml_private.h +++ b/dlls/msxml3/msxml_private.h @@ -162,6 +162,7 @@ static inline IXMLDOMNode *IXMLDOMNode_from_impl(xmlnode *This) extern void init_xmlnode(xmlnode*,xmlNodePtr,IXMLDOMNode*,dispex_static_data_t*); extern void destroy_xmlnode(xmlnode*); +extern BOOL node_query_interface(xmlnode*,REFIID,void**); extern HRESULT DOMDocument_create_from_xmldoc(xmlDocPtr xmldoc, IXMLDOMDocument3 **document); diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c index 5b9eee4..a2e175e 100644 --- a/dlls/msxml3/node.c +++ b/dlls/msxml3/node.c @@ -80,6 +80,13 @@ xmlNodePtr xmlNodePtr_from_domnode( IXMLDOMNode *iface, xmlElementType type ) return This->node; } +BOOL node_query_interface(xmlnode *This, REFIID riid, void **ppv) +{ + if(This->dispex.outer) + return dispex_query_interface(&This->dispex, riid, ppv); + return FALSE; +} + static HRESULT WINAPI xmlnode_QueryInterface( IXMLDOMNode *iface, REFIID riid, @@ -97,6 +104,8 @@ static HRESULT WINAPI xmlnode_QueryInterface( }else if (IsEqualGUID( riid, &IID_IDispatch) || IsEqualGUID( riid, &IID_IXMLDOMNode)) { *ppvObject = &This->lpVtbl; + }else if(node_query_interface(This, riid, ppvObject)) { + return *ppvObject ? S_OK : E_NOINTERFACE; }else { FIXME("interface %s not implemented\n", debugstr_guid(riid)); *ppvObject = NULL; @@ -1725,6 +1734,8 @@ void init_xmlnode(xmlnode *This, xmlNodePtr node, IXMLDOMNode *node_iface, dispe if(dispex_data) init_dispex(&This->dispex, (IUnknown*)This->iface, dispex_data); + else + This->dispex.outer = NULL; } IXMLDOMNode *create_node( xmlNodePtr node ) diff --git a/dlls/msxml3/pi.c b/dlls/msxml3/pi.c index 2c70cc7..0b2a7e7 100644 --- a/dlls/msxml3/pi.c +++ b/dlls/msxml3/pi.c @@ -67,6 +67,10 @@ static HRESULT WINAPI dom_pi_QueryInterface( { *ppvObject = IXMLDOMNode_from_impl(&This->node); } + else if(node_query_interface(&This->node, riid, ppvObject)) + { + return *ppvObject ? S_OK : E_NOINTERFACE; + } else { FIXME("Unsupported interface %s\n", debugstr_guid(riid)); diff --git a/dlls/msxml3/text.c b/dlls/msxml3/text.c index d3da258..0ae5aa6 100644 --- a/dlls/msxml3/text.c +++ b/dlls/msxml3/text.c @@ -69,6 +69,10 @@ static HRESULT WINAPI domtext_QueryInterface( { *ppvObject = IXMLDOMNode_from_impl(&This->node); } + else if(node_query_interface(&This->node, riid, ppvObject)) + { + return *ppvObject ? S_OK : E_NOINTERFACE; + } else if ( IsEqualGUID( riid, &IID_IXMLDOMElement ) || IsEqualGUID( riid, &IID_IXMLDOMCDATASection ) ) {

1 0

Jacek Caban : msxml3: Use fake IID to safely cast IXMLDOMNode interface to xmlnode object.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: a92e95eab16e0c6e0a97b241c6c5b0b9a96ee1fc URL: http://source.winehq.org/git/wine.git/?a=commit;h=a92e95eab16e0c6e0a97b241c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat Sep 4 17:55:04 2010 +0200 msxml3: Use fake IID to safely cast IXMLDOMNode interface to xmlnode object. --- dlls/msxml3/domdoc.c | 6 ++- dlls/msxml3/msxml_private.h | 6 +-- dlls/msxml3/node.c | 146 +++++++++++++++++++++++++----------------- dlls/msxml3/nodemap.c | 38 +++++------- 4 files changed, 108 insertions(+), 88 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=a92e95eab16e0c6e0a97b…

1 0

Jacek Caban : msxml3: Use separate object for unknown node type implementation.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: 4f61cd2840538483eaf2feb48fd734ec3a4bfa14 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4f61cd2840538483eaf2feb48… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat Sep 4 17:55:27 2010 +0200 msxml3: Use separate object for unknown node type implementation. --- dlls/msxml3/msxml_private.h | 1 - dlls/msxml3/node.c | 518 +++++++++++++++++++++++++++++++++++++++++-- 2 files changed, 496 insertions(+), 23 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=4f61cd2840538483eaf2f…

1 0

Andrew Talbot : dmusic: Constify a variable.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: ab683fee33b6a80bf0a0c38b0594477fe40177e2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ab683fee33b6a80bf0a0c38b0… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Sun Sep 5 12:36:31 2010 +0100 dmusic: Constify a variable. --- dlls/dmusic/dmusic_main.c | 2 +- dlls/dmusic/dmusic_private.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/dmusic/dmusic_main.c b/dlls/dmusic/dmusic_main.c index 9c0e78f..6cd92ab 100644 --- a/dlls/dmusic/dmusic_main.c +++ b/dlls/dmusic/dmusic_main.c @@ -187,7 +187,7 @@ HRESULT WINAPI DllGetClassObject(REFCLSID rclsid, REFIID riid, LPVOID *ppv) * */ /* dwPatch from MIDILOCALE */ -DWORD MIDILOCALE2Patch (LPMIDILOCALE pLocale) { +DWORD MIDILOCALE2Patch (const MIDILOCALE *pLocale) { DWORD dwPatch = 0; if (!pLocale) return 0; dwPatch |= (pLocale->ulBank & F_INSTRUMENT_DRUMS); /* set drum bit */ diff --git a/dlls/dmusic/dmusic_private.h b/dlls/dmusic/dmusic_private.h index 2de7e8a..e7668a4 100644 --- a/dlls/dmusic/dmusic_private.h +++ b/dlls/dmusic/dmusic_private.h @@ -266,7 +266,7 @@ typedef struct { #define GE(x) { &x, #x } /* dwPatch from MIDILOCALE */ -extern DWORD MIDILOCALE2Patch (LPMIDILOCALE pLocale); +extern DWORD MIDILOCALE2Patch (const MIDILOCALE *pLocale); /* MIDILOCALE from dwPatch */ extern void Patch2MIDILOCALE (DWORD dwPatch, LPMIDILOCALE pLocale);

1 0

Gerald Pfeifer : server: Remove pollev from set_socket_event.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: 87624c2c217994f563dfd317d596fa76579a80c8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=87624c2c217994f563dfd317d… Author: Gerald Pfeifer <gerald(a)pfeifer.com> Date: Sat Sep 4 22:17:39 2010 +0200 server: Remove pollev from set_socket_event. --- server/sock.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/server/sock.c b/server/sock.c index 4f54c46..04ae748 100644 --- a/server/sock.c +++ b/server/sock.c @@ -967,7 +967,6 @@ DECL_HANDLER(set_socket_event) { struct sock *sock; struct event *old_event; - int pollev; if (!(sock = (struct sock *)get_handle_obj( current->process, req->handle, FILE_WRITE_ATTRIBUTES, &sock_ops))) return; @@ -982,7 +981,7 @@ DECL_HANDLER(set_socket_event) if (debug_level && sock->event) fprintf(stderr, "event ptr: %p\n", sock->event); - pollev = sock_reselect( sock ); + sock_reselect( sock ); if (sock->mask) sock->state |= FD_WINE_NONBLOCKING;

1 0

Jacek Caban : msxml3: Use xmlnode child object' s IXMLDOMNode implementation as base interface.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: 27b4f11c0bb9f0b9d9023259ca6a2f9852a2d033 URL: http://source.winehq.org/git/wine.git/?a=commit;h=27b4f11c0bb9f0b9d9023259c… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat Sep 4 17:56:08 2010 +0200 msxml3: Use xmlnode child object's IXMLDOMNode implementation as base interface. The plan is to get rid of IXMLDOMNode interface in xmlnode object and use proper inheritance. --- dlls/msxml3/attribute.c | 5 +-- dlls/msxml3/cdata.c | 10 +--- dlls/msxml3/comment.c | 7 +-- dlls/msxml3/docfrag.c | 5 +-- dlls/msxml3/domdoc.c | 11 ++--- dlls/msxml3/element.c | 5 +-- dlls/msxml3/entityref.c | 5 +-- dlls/msxml3/node.c | 100 +++++++++-------------------------------------- dlls/msxml3/pi.c | 10 +--- dlls/msxml3/text.c | 8 +--- 10 files changed, 38 insertions(+), 128 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=27b4f11c0bb9f0b9d9023…

1 0

Jacek Caban : msxml3: Use get_node_obj to get xmlnode from iface in replaceChild implementation.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: 8a82ae668acdcef442b1c490107def73a58cc725 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8a82ae668acdcef442b1c4901… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat Sep 4 17:55:42 2010 +0200 msxml3: Use get_node_obj to get xmlnode from iface in replaceChild implementation. --- dlls/msxml3/node.c | 39 +++++++++++++++++++++------------------ 1 files changed, 21 insertions(+), 18 deletions(-) diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c index 1236fae..a7d2ddf 100644 --- a/dlls/msxml3/node.c +++ b/dlls/msxml3/node.c @@ -644,11 +644,9 @@ static HRESULT WINAPI xmlnode_replaceChild( IXMLDOMNode** outOldChild) { xmlnode *This = impl_from_IXMLDOMNode( iface ); - xmlNode *old_child_ptr, *new_child_ptr; + xmlnode *old_child, *new_child; xmlDocPtr leaving_doc; xmlNode *my_ancestor; - IXMLDOMNode *realOldChild; - HRESULT hr; TRACE("(%p)->(%p %p %p)\n", This, newChild, oldChild, outOldChild); @@ -660,23 +658,28 @@ static HRESULT WINAPI xmlnode_replaceChild( if(outOldChild) *outOldChild = NULL; - hr = IXMLDOMNode_QueryInterface(oldChild,&IID_IXMLDOMNode,(LPVOID*)&realOldChild); - if(FAILED(hr)) - return hr; + old_child = get_node_obj(oldChild); + if(!old_child) { + FIXME("oldChild is not our node implementation\n"); + return E_FAIL; + } - old_child_ptr = impl_from_IXMLDOMNode(realOldChild)->node; - IXMLDOMNode_Release(realOldChild); - if(old_child_ptr->parent != This->node) + if(old_child->node->parent != This->node) { WARN("childNode %p is not a child of %p\n", oldChild, iface); return E_INVALIDARG; } - new_child_ptr = impl_from_IXMLDOMNode(newChild)->node; + new_child = get_node_obj(newChild); + if(!new_child) { + FIXME("newChild is not our node implementation\n"); + return E_FAIL; + } + my_ancestor = This->node; while(my_ancestor) { - if(my_ancestor == new_child_ptr) + if(my_ancestor == new_child->node) { WARN("tried to create loop\n"); return E_FAIL; @@ -684,16 +687,16 @@ static HRESULT WINAPI xmlnode_replaceChild( my_ancestor = my_ancestor->parent; } - if(!new_child_ptr->parent) - if(xmldoc_remove_orphan(new_child_ptr->doc, new_child_ptr) != S_OK) - WARN("%p is not an orphan of %p\n", new_child_ptr, new_child_ptr->doc); + if(!new_child->node->parent) + if(xmldoc_remove_orphan(new_child->node->doc, new_child->node) != S_OK) + WARN("%p is not an orphan of %p\n", new_child->node, new_child->node->doc); - leaving_doc = new_child_ptr->doc; - xmldoc_add_ref(old_child_ptr->doc); - xmlReplaceNode(old_child_ptr, new_child_ptr); + leaving_doc = new_child->node->doc; + xmldoc_add_ref(old_child->node->doc); + xmlReplaceNode(old_child->node, new_child->node); xmldoc_release(leaving_doc); - xmldoc_add_orphan(old_child_ptr->doc, old_child_ptr); + xmldoc_add_orphan(old_child->node->doc, old_child->node); if(outOldChild) {

1 0

Alexandre Julliard : ws2_32/tests: Fix some buffer overflows in traces.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: 9f2c51800dacf11cf93dc4aeb5d5ea6c3c9d6cfd URL: http://source.winehq.org/git/wine.git/?a=commit;h=9f2c51800dacf11cf93dc4aeb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 3 21:07:54 2010 +0200 ws2_32/tests: Fix some buffer overflows in traces. --- dlls/ws2_32/tests/sock.c | 22 +++++++++++----------- 1 files changed, 11 insertions(+), 11 deletions(-) diff --git a/dlls/ws2_32/tests/sock.c b/dlls/ws2_32/tests/sock.c index e61d550..631fcda 100644 --- a/dlls/ws2_32/tests/sock.c +++ b/dlls/ws2_32/tests/sock.c @@ -2629,23 +2629,22 @@ static void get_event_details(int event, int *bit, char *name) } } -static char *dbgstr_event_seq(const LPARAM *seq) +static const char *dbgstr_event_seq(const LPARAM *seq) { static char message[1024]; - char name[10]; + char name[12]; + int len = 1; message[0] = '['; message[1] = 0; while (*seq) { get_event_details(WSAGETSELECTEVENT(*seq), NULL, name); - - sprintf(message, "%s%s%s(%d)", message, message[1] == 0 ? "" : " ", - name, WSAGETSELECTERROR(*seq)); - + len += sprintf(message + len, "%s(%d) ", name, WSAGETSELECTERROR(*seq)); seq++; } - strcat(message, "]"); + if (len > 1) len--; + strcpy( message + len, "]" ); return message; } @@ -2654,7 +2653,8 @@ static char *dbgstr_event_seq_result(SOCKET s, WSANETWORKEVENTS *netEvents) static char message[1024]; struct async_message *curr = messages_received; int index, error, bit = 0; - char name[10]; + char name[12]; + int len = 1; message[0] = '['; message[1] = 0; @@ -2685,10 +2685,10 @@ static char *dbgstr_event_seq_result(SOCKET s, WSANETWORKEVENTS *netEvents) curr = curr->next; } - sprintf(message, "%s%s%s(%d)", message, message[1] == 0 ? "" : " ", - name, error); + len += sprintf(message + len, "%s(%d) ", name, error); } - strcat(message, "]"); + if (len > 1) len--; + strcpy( message + len, "]" ); return message; }

1 0

Jacek Caban : msxml3: Store outer iface as IXMLDOMNode instead of IUnknown in xmlnode.
by Alexandre Julliard Sept. 6, 2010

Sept. 6, 2010

Module: wine Branch: master Commit: f5348b25aff4642956891ef267a06341aed0f1a5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f5348b25aff4642956891ef26… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Sat Sep 4 17:54:13 2010 +0200 msxml3: Store outer iface as IXMLDOMNode instead of IUnknown in xmlnode. --- dlls/msxml3/attribute.c | 2 +- dlls/msxml3/cdata.c | 2 +- dlls/msxml3/comment.c | 2 +- dlls/msxml3/docfrag.c | 2 +- dlls/msxml3/domdoc.c | 2 +- dlls/msxml3/element.c | 2 +- dlls/msxml3/entityref.c | 2 +- dlls/msxml3/msxml_private.h | 4 ++-- dlls/msxml3/node.c | 18 +++++++++--------- dlls/msxml3/pi.c | 2 +- dlls/msxml3/text.c | 2 +- 11 files changed, 20 insertions(+), 20 deletions(-) diff --git a/dlls/msxml3/attribute.c b/dlls/msxml3/attribute.c index 0bdc691..327f91c 100644 --- a/dlls/msxml3/attribute.c +++ b/dlls/msxml3/attribute.c @@ -556,7 +556,7 @@ IUnknown* create_attribute( xmlNodePtr attribute ) This->lpVtbl = &domattr_vtbl; This->ref = 1; - init_xmlnode(&This->node, attribute, (IUnknown*)&This->lpVtbl, NULL); + init_xmlnode(&This->node, attribute, (IXMLDOMNode*)&This->lpVtbl, NULL); return (IUnknown*) &This->lpVtbl; } diff --git a/dlls/msxml3/cdata.c b/dlls/msxml3/cdata.c index 373bbc8..b23d19d 100644 --- a/dlls/msxml3/cdata.c +++ b/dlls/msxml3/cdata.c @@ -835,7 +835,7 @@ IUnknown* create_cdata( xmlNodePtr text ) This->lpVtbl = &domcdata_vtbl; This->ref = 1; - init_xmlnode(&This->node, text, (IUnknown*)&This->lpVtbl, NULL); + init_xmlnode(&This->node, text, (IXMLDOMNode*)&This->lpVtbl, NULL); return (IUnknown*) &This->lpVtbl; } diff --git a/dlls/msxml3/comment.c b/dlls/msxml3/comment.c index 74d036d..6e7b77e 100644 --- a/dlls/msxml3/comment.c +++ b/dlls/msxml3/comment.c @@ -781,7 +781,7 @@ IUnknown* create_comment( xmlNodePtr comment ) This->lpVtbl = &domcomment_vtbl; This->ref = 1; - init_xmlnode(&This->node, comment, (IUnknown*)&This->lpVtbl, NULL); + init_xmlnode(&This->node, comment, (IXMLDOMNode*)&This->lpVtbl, NULL); return (IUnknown*) &This->lpVtbl; } diff --git a/dlls/msxml3/docfrag.c b/dlls/msxml3/docfrag.c index d1b5b8c..9ca63d3 100644 --- a/dlls/msxml3/docfrag.c +++ b/dlls/msxml3/docfrag.c @@ -526,7 +526,7 @@ IUnknown* create_doc_fragment( xmlNodePtr fragment ) This->lpVtbl = &domfrag_vtbl; This->ref = 1; - init_xmlnode(&This->node, fragment, (IUnknown*)&This->lpVtbl, NULL); + init_xmlnode(&This->node, fragment, (IXMLDOMNode*)&This->lpVtbl, NULL); return (IUnknown*) &This->lpVtbl; } diff --git a/dlls/msxml3/domdoc.c b/dlls/msxml3/domdoc.c index 98045a0..3f01329 100644 --- a/dlls/msxml3/domdoc.c +++ b/dlls/msxml3/domdoc.c @@ -2404,7 +2404,7 @@ HRESULT DOMDocument_create_from_xmldoc(xmlDocPtr xmldoc, IXMLDOMDocument3 **docu doc->safeopt = 0; doc->bsc = NULL; - init_xmlnode(&doc->node, (xmlNodePtr)xmldoc, (IUnknown*)&doc->lpVtbl, &domdoc_dispex); + init_xmlnode(&doc->node, (xmlNodePtr)xmldoc, (IXMLDOMNode*)&doc->lpVtbl, &domdoc_dispex); *document = (IXMLDOMDocument3*)&doc->lpVtbl; diff --git a/dlls/msxml3/element.c b/dlls/msxml3/element.c index 4c557ef..5b4a604 100644 --- a/dlls/msxml3/element.c +++ b/dlls/msxml3/element.c @@ -801,7 +801,7 @@ IUnknown* create_element( xmlNodePtr element ) This->lpVtbl = &domelem_vtbl; This->ref = 1; - init_xmlnode(&This->node, element, (IUnknown*)&This->lpVtbl, &domelem_dispex); + init_xmlnode(&This->node, element, (IXMLDOMNode*)&This->lpVtbl, &domelem_dispex); return (IUnknown*) &This->lpVtbl; } diff --git a/dlls/msxml3/entityref.c b/dlls/msxml3/entityref.c index f583141..c55fd18 100644 --- a/dlls/msxml3/entityref.c +++ b/dlls/msxml3/entityref.c @@ -525,7 +525,7 @@ IUnknown* create_doc_entity_ref( xmlNodePtr entity ) This->lpVtbl = &entityref_vtbl; This->ref = 1; - init_xmlnode(&This->node, entity, (IUnknown*)&This->lpVtbl, NULL); + init_xmlnode(&This->node, entity, (IXMLDOMNode*)&This->lpVtbl, NULL); return (IUnknown*) &This->lpVtbl; } diff --git a/dlls/msxml3/msxml_private.h b/dlls/msxml3/msxml_private.h index 2210574..fe843b8 100644 --- a/dlls/msxml3/msxml_private.h +++ b/dlls/msxml3/msxml_private.h @@ -145,7 +145,7 @@ typedef struct _xmlnode { DispatchEx dispex; const struct IXMLDOMNodeVtbl *lpVtbl; - IUnknown *pUnkOuter; + IXMLDOMNode *iface; LONG ref; xmlNodePtr node; } xmlnode; @@ -160,7 +160,7 @@ static inline IXMLDOMNode *IXMLDOMNode_from_impl(xmlnode *This) return (IXMLDOMNode*)&This->lpVtbl; } -extern void init_xmlnode(xmlnode*,xmlNodePtr,IUnknown*,dispex_static_data_t*); +extern void init_xmlnode(xmlnode*,xmlNodePtr,IXMLDOMNode*,dispex_static_data_t*); extern void destroy_xmlnode(xmlnode*); extern HRESULT DOMDocument_create_from_xmldoc(xmlDocPtr xmldoc, IXMLDOMDocument3 **document); diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c index 2c6df32..5b9eee4 100644 --- a/dlls/msxml3/node.c +++ b/dlls/msxml3/node.c @@ -89,8 +89,8 @@ static HRESULT WINAPI xmlnode_QueryInterface( TRACE("(%p)->(%s %p)\n", This, debugstr_guid(riid), ppvObject); - if(This->pUnkOuter) - return IUnknown_QueryInterface(This->pUnkOuter, riid, ppvObject); + if(This->iface) + return IXMLDOMNode_QueryInterface(This->iface, riid, ppvObject); if (IsEqualGUID(riid, &IID_IUnknown)) { *ppvObject = iface; @@ -112,8 +112,8 @@ static ULONG WINAPI xmlnode_AddRef( { xmlnode *This = impl_from_IXMLDOMNode( iface ); - if(This->pUnkOuter) - return IUnknown_AddRef(This->pUnkOuter); + if(This->iface) + return IXMLDOMNode_AddRef(This->iface); return InterlockedIncrement(&This->ref); } @@ -124,8 +124,8 @@ static ULONG WINAPI xmlnode_Release( xmlnode *This = impl_from_IXMLDOMNode( iface ); LONG ref; - if(This->pUnkOuter) - return IUnknown_Release(This->pUnkOuter); + if(This->iface) + return IXMLDOMNode_Release(This->iface); ref = InterlockedDecrement( &This->ref ); if(!ref) { @@ -1713,7 +1713,7 @@ void destroy_xmlnode(xmlnode *This) xmldoc_release(This->node->doc); } -void init_xmlnode(xmlnode *This, xmlNodePtr node, IUnknown *outer, dispex_static_data_t *dispex_data ) +void init_xmlnode(xmlnode *This, xmlNodePtr node, IXMLDOMNode *node_iface, dispex_static_data_t *dispex_data) { if(node) xmldoc_add_ref( node->doc ); @@ -1721,10 +1721,10 @@ void init_xmlnode(xmlnode *This, xmlNodePtr node, IUnknown *outer, dispex_static This->lpVtbl = &xmlnode_vtbl; This->ref = 1; This->node = node; - This->pUnkOuter = outer; + This->iface = node_iface; if(dispex_data) - init_dispex(&This->dispex, This->pUnkOuter, dispex_data); + init_dispex(&This->dispex, (IUnknown*)This->iface, dispex_data); } IXMLDOMNode *create_node( xmlNodePtr node ) diff --git a/dlls/msxml3/pi.c b/dlls/msxml3/pi.c index 4a5e621..2c70cc7 100644 --- a/dlls/msxml3/pi.c +++ b/dlls/msxml3/pi.c @@ -611,7 +611,7 @@ IUnknown* create_pi( xmlNodePtr pi ) This->lpVtbl = &dom_pi_vtbl; This->ref = 1; - init_xmlnode(&This->node, pi, (IUnknown*)&This->lpVtbl, NULL); + init_xmlnode(&This->node, pi, (IXMLDOMNode*)&This->lpVtbl, NULL); return (IUnknown*) &This->lpVtbl; } diff --git a/dlls/msxml3/text.c b/dlls/msxml3/text.c index a087076..d3da258 100644 --- a/dlls/msxml3/text.c +++ b/dlls/msxml3/text.c @@ -811,7 +811,7 @@ IUnknown* create_text( xmlNodePtr text ) This->lpVtbl = &domtext_vtbl; This->ref = 1; - init_xmlnode(&This->node, text, (IUnknown*)&This->lpVtbl, NULL); + init_xmlnode(&This->node, text, (IXMLDOMNode*)&This->lpVtbl, NULL); return (IUnknown*) &This->lpVtbl; }

1 0