mshtml: Make the wrapper_func_xxx() functions static.

Francois Gouget fgouget at free.fr
Tue Dec 6 10:39:36 CST 2011


---

It's strange that only one of the implementations was marked static. 
Does static cause trouble with __ASM functions?
(it compiles at least<g>)

 dlls/mshtml/ifacewrap.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/mshtml/ifacewrap.c b/dlls/mshtml/ifacewrap.c
index 006afd2..5a625f9 100644
--- a/dlls/mshtml/ifacewrap.c
+++ b/dlls/mshtml/ifacewrap.c
@@ -88,7 +88,7 @@ static HRESULT WINAPI wrapper_Release(IUnknown *iface)
 #ifdef __i386__
 
 #define DEFINE_WRAPPER_FUNC(n, off, x)          \
-    HRESULT WINAPI wrapper_func_##n(IUnknown*); \
+    static HRESULT WINAPI wrapper_func_##n(IUnknown*); \
     __ASM_GLOBAL_FUNC(wrapper_func_##n,         \
         "movl 4(%esp), %eax\n\t"                \
         "movl 4(%eax), %eax\n\t"                \
@@ -99,7 +99,7 @@ static HRESULT WINAPI wrapper_Release(IUnknown *iface)
 #elif defined(__x86_64__)
 
 #define DEFINE_WRAPPER_FUNC(n, x, off)          \
-    HRESULT WINAPI wrapper_func_##n(IUnknown*); \
+    static HRESULT WINAPI wrapper_func_##n(IUnknown*); \
     __ASM_GLOBAL_FUNC(wrapper_func_##n,         \
         "movq 8(%rcx), %rcx\n\t"                \
         "movq 0(%rcx), %rax\n\t"                \
-- 
1.7.7.3




More information about the wine-patches mailing list