[4/7] msxml3: Simplify IXMLDOMNode::removeChild.

Michael Karcher wine at mkarcher.dialup.fu-berlin.de
Sun Oct 5 05:13:24 CDT 2008


---
 dlls/msxml3/node.c |   11 +----------
 1 files changed, 1 insertions(+), 10 deletions(-)

diff --git a/dlls/msxml3/node.c b/dlls/msxml3/node.c
index d9a2a0c..25ffe45 100644
--- a/dlls/msxml3/node.c
+++ b/dlls/msxml3/node.c
@@ -620,8 +620,6 @@ static HRESULT WINAPI xmlnode_removeChild(
 {
     xmlnode *This = impl_from_IXMLDOMNode( iface );
     xmlNode *child_node_ptr;
-    HRESULT hr;
-    IXMLDOMNode *child;
 
     TRACE("%p->(%p, %p)\n", This, childNode, oldChild);
 
@@ -630,22 +628,15 @@ static HRESULT WINAPI xmlnode_removeChild(
     if(oldChild)
         *oldChild = NULL;
 
-    hr = IXMLDOMNode_QueryInterface(childNode, &IID_IXMLDOMNode, (LPVOID)&child);
-    if(FAILED(hr))
-        return hr;
-
-    child_node_ptr = impl_from_IXMLDOMNode(child)->node;
+    child_node_ptr = impl_from_IXMLDOMNode(childNode)->node;
     if(child_node_ptr->parent != This->node)
     {
         WARN("childNode %p is not a child of %p\n", childNode, iface);
-        IXMLDOMNode_Release(child);
         return E_INVALIDARG;
     }
 
     xmlUnlinkNode(child_node_ptr);
 
-    IXMLDOMNode_Release(child);
-
     if(oldChild)
     {
         IXMLDOMNode_AddRef(childNode);
-- 
1.5.6.5




More information about the wine-patches mailing list