commit 35b59556064e773ee3e51c7b0cdb98caf2970c86
author Nikolay Sivov <nsivov@codeweavers.com> Thu Oct 06 10:35:07 2011 -0500
committer Alexandre Julliard <julliard@winehq.org> Thu Oct 06 16:46:40 2011 -0500
tree 268a1594586539ca0c6c2e2e745e1b238834bc60
parent 5931af0283c7426f2d19f7d1ae5f73cdef6e57ea

msxml3: Use a helper to get property values.

dlls/msxml3/mxwriter.c

diff --git a/dlls/msxml3/mxwriter.c b/dlls/msxml3/mxwriter.c
index 0214d80..cc75d1d 100644
--- a/dlls/msxml3/mxwriter.c
+++ b/dlls/msxml3/mxwriter.c
@@ -186,6 +186,13 @@
     return S_OK;
 }
 
+static HRESULT writer_get_property(const mxwriter *writer, MXWRITER_PROPS property, VARIANT_BOOL *value)
+{
+    if (!value) return E_POINTER;
+    *value = writer->props[property];
+    return S_OK;
+}
+
 static inline mxwriter *impl_from_IMXWriter(IMXWriter *iface)
 {
     return CONTAINING_RECORD(iface, mxwriter, IMXWriter_iface);
@@ -469,12 +476,7 @@
     mxwriter *This = impl_from_IMXWriter( iface );
 
     TRACE("(%p)->(%p)\n", This, value);
-
-    if (!value) return E_POINTER;
-
-    *value = This->props[MXWriter_BOM];
-
-    return S_OK;
+    return writer_get_property(This, MXWriter_BOM, value);
 }
 
 static HRESULT WINAPI mxwriter_put_indent(IMXWriter *iface, VARIANT_BOOL value)
@@ -490,12 +492,7 @@
     mxwriter *This = impl_from_IMXWriter( iface );
 
     TRACE("(%p)->(%p)\n", This, value);
-
-    if (!value) return E_POINTER;
-
-    *value = This->props[MXWriter_Indent];
-
-    return S_OK;
+    return writer_get_property(This, MXWriter_Indent, value);
 }
 
 static HRESULT WINAPI mxwriter_put_standalone(IMXWriter *iface, VARIANT_BOOL value)
@@ -511,12 +508,7 @@
     mxwriter *This = impl_from_IMXWriter( iface );
 
     TRACE("(%p)->(%p)\n", This, value);
-
-    if (!value) return E_POINTER;
-
-    *value = This->props[MXWriter_Standalone];
-
-    return S_OK;
+    return writer_get_property(This, MXWriter_Standalone, value);
 }
 
 static HRESULT WINAPI mxwriter_put_omitXMLDeclaration(IMXWriter *iface, VARIANT_BOOL value)
@@ -532,12 +524,7 @@
     mxwriter *This = impl_from_IMXWriter( iface );
 
     TRACE("(%p)->(%p)\n", This, value);
-
-    if (!value) return E_POINTER;
-
-    *value = This->props[MXWriter_OmitXmlDecl];
-
-    return S_OK;
+    return writer_get_property(This, MXWriter_OmitXmlDecl, value);
 }
 
 static HRESULT WINAPI mxwriter_put_version(IMXWriter *iface, BSTR version)
@@ -578,12 +565,7 @@
     mxwriter *This = impl_from_IMXWriter( iface );
 
     TRACE("(%p)->(%p)\n", This, value);
-
-    if (!value) return E_POINTER;
-
-    *value = This->props[MXWriter_DisableEscaping];
-
-    return S_OK;
+    return writer_get_property(This, MXWriter_DisableEscaping, value);
 }
 
 static HRESULT WINAPI mxwriter_flush(IMXWriter *iface)