The problems with having the macros in the public headers are that they won't work on x86 machines, and are not included in the public headers. Therefore someone developing for another platform will have their code break when using the native public headers or when porting the code to an x86 where the thiscall calling convention is used.<br>
<br>On Sun, Nov 30, 2008 at 5:42 AM, Maarten Lankhorst <span dir="ltr"><<a href="mailto:m.b.lankhorst@gmail.com">m.b.lankhorst@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Commit e4ef9c5c removed the ITextServices macros from wine/include/textserv.h. With this they are added again, and COBJMACROS before including that header, so that they won't conflict.<br>
---<br>
<br>From c04fcab5168309395602f4c4fd07a319a9a532ab Mon Sep 17 00:00:00 2001<br>
From: Maarten Lankhorst <<a href="mailto:maarten@codeweavers.com">maarten@codeweavers.com</a>><br>
Date: Sun, 30 Nov 2008 11:37:54 +0100<br>
Subject: [PATCH] includes: Fix macros for ITextServices<br>
<br>
---<br>
dlls/riched20/tests/txtsrv.c | 8 +++++++-<br>
include/textserv.h | 19 +++++++++++++++++++<br>
2 files changed, 26 insertions(+), 1 deletions(-)<br>
<br>
diff --git a/dlls/riched20/tests/txtsrv.c b/dlls/riched20/tests/txtsrv.c<br>
index 3d47490..c28f5a7 100644<br>
--- a/dlls/riched20/tests/txtsrv.c<br>
+++ b/dlls/riched20/tests/txtsrv.c<br>
@@ -29,6 +29,9 @@<br>
#include <objbase.h><br>
#include <richedit.h><br>
#include <initguid.h><br>
+<br>
+#undef COBJMACROS<br>
+/* Don't define ITextServices, we override it here */<br>
#include <textserv.h><br>
#include <wine/test.h><br>
<br>
@@ -44,6 +47,9 @@ static HMODULE hmoduleRichEdit;<br>
#define TXTSERV_VTABLE(This) (This)->lpVtbl<br>
#endif /* __i386__ */<br>
<br>
+#define ITextServices_QueryInterface(p,a,b) (p)->lpVtbl->QueryInterface(p,a,b)<br>
+#define ITextServices_AddRef(p) (p)->lpVtbl->AddRef(p)<br>
+#define ITextServices_Release(p) (p)->lpVtbl->Release(p)<br>
#define ITextServices_TxSendMessage(This,a,b,c,d) TXTSERV_VTABLE(This)->TxSendMessage(This,a,b,c,d)<br>
#define ITextServices_TxDraw(This,a,b,c,d,e,f,g,h,i,j,k,l) TXTSERV_VTABLE(This)->TxDraw(This,a,b,c,d,e,f,g,h,i,j,k,l)<br>
#define ITextServices_TxGetHScroll(This,a,b,c,d,e) TXTSERV_VTABLE(This)->TxGetHScroll(This,a,b,c,d,e)<br>
@@ -84,7 +90,7 @@ static HRESULT WINAPI ITextHostImpl_QueryInterface(ITextHost *iface,<br>
<br>
if (IsEqualIID(riid, &IID_IUnknown) || IsEqualIID(riid, &IID_ITextHost)) {<br>
*ppvObject = This;<br>
- ITextHost_AddRef((ITextHost *)*ppvObject);<br>
+ IUnknown_AddRef((IUnknown *)*ppvObject);<br>
return S_OK;<br>
}<br>
<br>
diff --git a/include/textserv.h b/include/textserv.h<br>
index 977b742..820fc0a 100644<br>
--- a/include/textserv.h<br>
+++ b/include/textserv.h<br>
@@ -147,6 +147,25 @@ DECLARE_INTERFACE_(ITextServices,IUnknown)<br>
#define ITextServices_QueryInterface(p,a,b) (p)->lpVtbl->QueryInterface(p,a,b)<br>
#define ITextServices_AddRef(p) (p)->lpVtbl->AddRef(p)<br>
#define ITextServices_Release(p) (p)->lpVtbl->Release(p)<br>
+/*** ITextServices methods ***/<br>
+#define ITextServices_TxSendMessage(p,a,b,c,d) (p)->lpVtbl->TxSendMessage(p,a,b<br>
+#define ITextServices_TxDraw(p,a,b,c,d,e,f,g,h,i,j,k,l) (p)->lpVtbl->TxDraw(p,a<br>
+#define ITextServices_TxGetHScroll(p,a,b,c,d,e) (p)->lpVtbl->TxGetHScroll(p,a,b<br>
+#define ITextServices_TxGetVScroll(p,a,b,c,d,e) (p)->lpVtbl->TxGetVScroll(p,a,b<br>
+#define ITextServices_OnTxSetCursor(p,a,b,c,d,e,f,g,h,i) (p)->lpVtbl->OnTxSetCu<br>
+#define ITextServices_TxQueryHitPoint(p,a,b,c,d,e,f,g,h,i,j) (p)->lpVtbl->TxQue<br>
+#define ITextServices_OnTxInplaceActivate(p,a) (p)->lpVtbl->OnTxInplaceActivate<br>
+#define ITextServices_OnTxInplaceDeactivate(p) (p)->lpVtbl->OnTxInplaceDeactiva<br>
+#define ITextServices_OnTxUIActivate(p) (p)->lpVtbl->OnTxUIActivate(p)<br>
+#define ITextServices_OnTxUIDeactivate(p) (p)->lpVtbl->OnTxUIDeactivate(p)<br>
+#define ITextServices_TxGetText(p,a) (p)->lpVtbl->TxGetText(p,a)<br>
+#define ITextServices_TxSetText(p,a) (p)->lpVtbl->TxSetText(p,a)<br>
+#define ITextServices_TxGetCurrentTargetX(p,a) (p)->lpVtbl->TxGetCurrentTargetX<br>
+#define ITextServices_TxGetBaseLinePos(p,a) (p)->lpVtbl->TxGetBaseLinePos(p,a)<br>
+#define ITextServices_TxGetNaturalSize(p,a,b,c,d,e,f,g,h) (p)->lpVtbl->TxGetNat<br>
+#define ITextServices_TxGetDropTarget(p,a) (p)->lpVtbl->TxGetDropTarget(p,a)<br>
+#define ITextServices_OnTxPropertyBitsChange(p,a,b) (p)->lpVtbl->OnTxPropertyBi<br>
+#define ITextServices_TxGetCachedSize(p,a,b) (p)->lpVtbl->TxGetCachedSize(p,a,b<br>
#endif<br>
<br>
#undef INTERFACE<br>
--<br>
<a href="http://1.5.6.5" target="_blank">1.5.6.5</a><br>
<br>
<br><br>
<br></blockquote></div><br>