Alexandre Julliard : msdaps: Remove the redundant IRowset interface.

Alexandre Julliard julliard at winehq.org
Mon Nov 29 13:38:38 CST 2010


Module: wine
Branch: master
Commit: 624ca877f1a2637ed96a9776766059a10985110f
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=624ca877f1a2637ed96a9776766059a10985110f

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Mon Nov 29 12:17:42 2010 +0100

msdaps: Remove the redundant IRowset interface.

---

 dlls/msdaps/row_server.c |  119 +++++++++-------------------------------------
 1 files changed, 23 insertions(+), 96 deletions(-)

diff --git a/dlls/msdaps/row_server.c b/dlls/msdaps/row_server.c
index 47d4e53..d1e1427 100644
--- a/dlls/msdaps/row_server.c
+++ b/dlls/msdaps/row_server.c
@@ -844,7 +844,6 @@ static HRESULT create_row_proxy(IWineRowServer *server, IUnknown **obj)
 
 typedef struct
 {
-    IRowset IRowset_iface;
     IRowsetLocate IRowsetLocate_iface;
     IRowsetInfo IRowsetInfo_iface;
     IAccessor IAccessor_iface;
@@ -854,11 +853,6 @@ typedef struct
     IWineRowServer *server;
 } rowset_proxy;
 
-static inline rowset_proxy *impl_from_IRowset(IRowset *iface)
-{
-    return CONTAINING_RECORD(iface, rowset_proxy, IRowset_iface);
-}
-
 static inline rowset_proxy *impl_from_IRowsetLocate(IRowsetLocate *iface)
 {
     return CONTAINING_RECORD(iface, rowset_proxy, IRowsetLocate_iface);
@@ -874,19 +868,16 @@ static inline rowset_proxy *impl_from_IAccessor(IAccessor *iface)
     return CONTAINING_RECORD(iface, rowset_proxy, IAccessor_iface);
 }
 
-static HRESULT WINAPI rowset_QueryInterface(IRowset *iface, REFIID iid, void **obj)
+static HRESULT WINAPI rowsetlocate_QueryInterface(IRowsetLocate *iface, REFIID iid, void **obj)
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
     TRACE("(%p)->(%s, %p)\n", This, debugstr_guid(iid), obj);
 
     *obj = NULL;
 
     if(IsEqualIID(iid, &IID_IUnknown) ||
-       IsEqualIID(iid, &IID_IRowset))
-    {
-        *obj = &This->IRowset_iface;
-    }
-    else if(IsEqualIID(iid, &IID_IRowsetLocate))
+       IsEqualIID(iid, &IID_IRowset) ||
+       IsEqualIID(iid, &IID_IRowsetLocate))
     {
         *obj = &This->IRowsetLocate_iface;
     }
@@ -908,17 +899,17 @@ static HRESULT WINAPI rowset_QueryInterface(IRowset *iface, REFIID iid, void **o
     return S_OK;
 }
 
-static ULONG WINAPI rowset_AddRef(IRowset *iface)
+static ULONG WINAPI rowsetlocate_AddRef(IRowsetLocate *iface)
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
     TRACE("(%p)\n", This);
 
     return InterlockedIncrement(&This->ref);
 }
 
-static ULONG WINAPI rowset_Release(IRowset *iface)
+static ULONG WINAPI rowsetlocate_Release(IRowsetLocate *iface)
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
     LONG ref;
 
     TRACE("(%p)\n", This);
@@ -933,10 +924,10 @@ static ULONG WINAPI rowset_Release(IRowset *iface)
     return ref;
 }
 
-static HRESULT WINAPI rowset_AddRefRows(IRowset *iface, DBCOUNTITEM cRows, const HROW rghRows[],
-                                        DBREFCOUNT rgRefCounts[], DBROWSTATUS rgRowStatus[])
+static HRESULT WINAPI rowsetlocate_AddRefRows(IRowsetLocate *iface, DBCOUNTITEM cRows, const HROW rghRows[],
+                                              DBREFCOUNT rgRefCounts[], DBROWSTATUS rgRowStatus[])
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
     HRESULT hr;
     DBREFCOUNT *refs = rgRefCounts;
     DBSTATUS *stats = rgRowStatus;
@@ -954,9 +945,9 @@ static HRESULT WINAPI rowset_AddRefRows(IRowset *iface, DBCOUNTITEM cRows, const
     return hr;
 }
 
-static HRESULT WINAPI rowset_GetData(IRowset *iface, HROW hRow, HACCESSOR hAccessor, void *pData)
+static HRESULT WINAPI rowsetlocate_GetData(IRowsetLocate *iface, HROW hRow, HACCESSOR hAccessor, void *pData)
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
     HRESULT hr;
     IAccessor *accessor;
     DBACCESSORFLAGS flags;
@@ -966,7 +957,7 @@ static HRESULT WINAPI rowset_GetData(IRowset *iface, HROW hRow, HACCESSOR hAcces
 
     TRACE("(%p)->(%lx, %lx, %p)\n", This, hRow, hAccessor, pData);
 
-    hr = IRowset_QueryInterface(iface, &IID_IAccessor, (void**)&accessor);
+    hr = IRowsetLocate_QueryInterface(iface, &IID_IAccessor, (void**)&accessor);
     if(FAILED(hr)) return hr;
 
     hr = IAccessor_GetBindings(accessor, hAccessor, &flags, &count, &bindings);
@@ -996,10 +987,10 @@ static HRESULT WINAPI rowset_GetData(IRowset *iface, HROW hRow, HACCESSOR hAcces
     return hr;
 }
 
-static HRESULT WINAPI rowset_GetNextRows(IRowset *iface, HCHAPTER hReserved, DBROWOFFSET lRowsOffset,
-                                         DBROWCOUNT cRows, DBCOUNTITEM *pcRowObtained, HROW **prghRows)
+static HRESULT WINAPI rowsetlocate_GetNextRows(IRowsetLocate *iface, HCHAPTER hReserved, DBROWOFFSET lRowsOffset,
+                                               DBROWCOUNT cRows, DBCOUNTITEM *pcRowObtained, HROW **prghRows)
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
     HRESULT hr;
     HROW *rows = NULL;
 
@@ -1017,10 +1008,10 @@ static HRESULT WINAPI rowset_GetNextRows(IRowset *iface, HCHAPTER hReserved, DBR
     return hr;
 }
 
-static HRESULT WINAPI rowset_ReleaseRows(IRowset *iface, DBCOUNTITEM cRows, const HROW rghRows[],
-                                         DBROWOPTIONS rgRowOptions[], DBREFCOUNT rgRefCounts[], DBROWSTATUS rgRowStatus[])
+static HRESULT WINAPI rowsetlocate_ReleaseRows(IRowsetLocate *iface, DBCOUNTITEM cRows, const HROW rghRows[],
+                                               DBROWOPTIONS rgRowOptions[], DBREFCOUNT rgRefCounts[], DBROWSTATUS rgRowStatus[])
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
     HRESULT hr;
     DBROWOPTIONS *options = rgRowOptions;
     DBREFCOUNT *refs = rgRefCounts;
@@ -1045,78 +1036,15 @@ static HRESULT WINAPI rowset_ReleaseRows(IRowset *iface, DBCOUNTITEM cRows, cons
     return hr;
 }
 
-static HRESULT WINAPI rowset_RestartPosition(IRowset* iface, HCHAPTER hReserved)
+static HRESULT WINAPI rowsetlocate_RestartPosition(IRowsetLocate* iface, HCHAPTER hReserved)
 {
-    rowset_proxy *This = impl_from_IRowset(iface);
+    rowset_proxy *This = impl_from_IRowsetLocate(iface);
 
     FIXME("(%p)->(%lx): stub\n", This, hReserved);
 
     return E_NOTIMPL;
 }
 
-static const IRowsetVtbl rowset_vtbl =
-{
-    rowset_QueryInterface,
-    rowset_AddRef,
-    rowset_Release,
-    rowset_AddRefRows,
-    rowset_GetData,
-    rowset_GetNextRows,
-    rowset_ReleaseRows,
-    rowset_RestartPosition
-};
-
-static HRESULT WINAPI rowsetlocate_QueryInterface(IRowsetLocate *iface, REFIID iid, void **obj)
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IUnknown_QueryInterface((IUnknown *)This, iid, obj);
-}
-
-static ULONG WINAPI rowsetlocate_AddRef(IRowsetLocate *iface)
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IUnknown_AddRef((IUnknown *)This);
-}
-
-static ULONG WINAPI rowsetlocate_Release(IRowsetLocate *iface)
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IUnknown_Release((IUnknown *)This);
-}
-
-static HRESULT WINAPI rowsetlocate_AddRefRows(IRowsetLocate *iface, DBCOUNTITEM cRows, const HROW rghRows[],
-                                              DBREFCOUNT rgRefCounts[], DBROWSTATUS rgRowStatus[])
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IRowset_AddRefRows((IRowset*)This, cRows, rghRows, rgRefCounts, rgRowStatus);
-}
-
-static HRESULT WINAPI rowsetlocate_GetData(IRowsetLocate *iface, HROW hRow, HACCESSOR hAccessor, void *pData)
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IRowset_GetData((IRowset*)This, hRow, hAccessor, pData);
-}
-
-static HRESULT WINAPI rowsetlocate_GetNextRows(IRowsetLocate *iface, HCHAPTER hReserved, DBROWOFFSET lRowsOffset,
-                                               DBROWCOUNT cRows, DBCOUNTITEM *pcRowObtained, HROW **prghRows)
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IRowset_GetNextRows((IRowset*)This, hReserved, lRowsOffset, cRows, pcRowObtained, prghRows);
-}
-
-static HRESULT WINAPI rowsetlocate_ReleaseRows(IRowsetLocate *iface, DBCOUNTITEM cRows, const HROW rghRows[],
-                                               DBROWOPTIONS rgRowOptions[], DBREFCOUNT rgRefCounts[], DBROWSTATUS rgRowStatus[])
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IRowset_ReleaseRows((IRowset*)This, cRows, rghRows, rgRowOptions, rgRefCounts, rgRowStatus);
-}
-
-static HRESULT WINAPI rowsetlocate_RestartPosition(IRowsetLocate *iface, HCHAPTER hReserved)
-{
-    rowset_proxy *This = impl_from_IRowsetLocate(iface);
-    return IRowset_RestartPosition((IRowset*)This, hReserved);
-}
-
 static HRESULT WINAPI rowsetlocate_Compare(IRowsetLocate *iface, HCHAPTER hReserved, DBBKMARK cbBookmark1, const BYTE *pBookmark1,
                                            DBBKMARK cbBookmark2, const BYTE *pBookmark2, DBCOMPARE *pComparison)
 {
@@ -1330,7 +1258,6 @@ static HRESULT create_rowset_proxy(IWineRowServer *server, IUnknown **obj)
     proxy = HeapAlloc(GetProcessHeap(), 0, sizeof(*proxy));
     if(!proxy) return E_OUTOFMEMORY;
 
-    proxy->IRowset_iface.lpVtbl = &rowset_vtbl;
     proxy->IRowsetLocate_iface.lpVtbl = &rowsetlocate_vtbl;
     proxy->IRowsetInfo_iface.lpVtbl = &rowsetinfo_vtbl;
     proxy->IAccessor_iface.lpVtbl = &accessor_vtbl;
@@ -1338,7 +1265,7 @@ static HRESULT create_rowset_proxy(IWineRowServer *server, IUnknown **obj)
     IWineRowServer_AddRef(server);
     proxy->server = server;
 
-    *obj = (IUnknown *)&proxy->IRowset_iface;
+    *obj = (IUnknown *)&proxy->IRowsetLocate_iface;
     TRACE("returning %p\n", *obj);
     return S_OK;
 }




More information about the wine-cvs mailing list