Am 05.06.2014 10:05, schrieb Stepan Kasal: > mingw.c defines several wrapper functionsi, like mingw_unlink(). > These wrappers are deployed by macros like this: > #define unlink mingw_unlink > The function itself is preceded by #undef, leaving the wrapper out > of the game for the rest of mingw.c. > > This was not probably intentional; for example, there are three > calls to open() below the definition mingw_open() that probably > have no reason to circumvent the wrapper. > OTOH, there is one call to gethostbyname() before it was undefined; > probably happy that it actually calls mingw_gethostbyname(). > > This patch adds back the #define after each wrapper definition. > > Signed-off-by: Stepan Kasal <kasal@xxxxxx> > --- > compat/mingw.c | 20 ++++++++++++++++++++ > 1 file changed, 20 insertions(+) > > diff --git a/compat/mingw.c b/compat/mingw.c > index a0e13bc..e7193c0 100644 > --- a/compat/mingw.c > +++ b/compat/mingw.c > @@ -224,6 +224,7 @@ int mingw_unlink(const char *pathname) > ret = unlink(pathname); > return ret; > } > +#define unlink mingw_unlink (etc...) I don't particularly like this approach: It robs the precise control of which function we can invoke from other places in mingw.c. Within mingw.c, if some other function inside mingw.c wants to use mingw_unlink, then it should be written as 'mingw_unlink(foo)', not 'unlink(foo)'. So, IMO the macros should be #undef'ed at the top of the file, and all users (like the open() and gethostbyname() invocations that you identified) should be audited and changed to call the function they actually need (i.e., the system open vs. mingw_open). -- Hannes -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html