Re: [PATCH 08/11] Place __stdcall to correct position.
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Aug 17, 2009, 17:01 UTC
- Message-ID
- <alpine.DEB.1.00.0908171859060.4991@intel-tinevez-2-302>
- In-Reply-To
- <1250525103-5184-3-git-send-email-lznuaa@gmail.com>
Hi,
On Tue, 18 Aug 2009, Frank Li wrote:
> MSVC require __stdcall is between return value and function name. > ALL Win32 API definition is as TYPE WINAPI function name > > Signed-off-by: Frank Li <lznuaa@gmail.com>
How about "... to the correct ..." and "MSVC requires _stdcall to be between return value..." and "All Win32 API functions are declared with the WINAPI attribute."?
Show 10 quoted lines
> diff --git a/compat/mingw.c b/compat/mingw.c
> index d5fa0ed..0c9c793 100644
> --- a/compat/mingw.c
> +++ b/compat/mingw.c
> @@ -1146,7 +1146,7 @@ void mingw_open_html(const char *unixpath)
>
> int link(const char *oldpath, const char *newpath)
> {
> - typedef BOOL WINAPI (*T)(const char*, const char*, LPSECURITY_ATTRIBUTES);
> + typedef BOOL (WINAPI *T)(const char*, const char*, LPSECURITY_ATTRIBUTES);An extra space slipped in, there.
Show 16 quoted lines
> diff --git a/run-command.c b/run-command.c
> index df139da..423b506 100644
> --- a/run-command.c
> +++ b/run-command.c
> @@ -295,12 +295,12 @@ int run_command_v_opt_cd_env(const char **argv, int opt, const char *dir, const
> }
>
> #if defined(__MINGW32__) || defined(_MSC_VER)
> -static __stdcall unsigned run_thread(void *data)
> +static unsigned __stdcall run_thread(void *data)
> {
> struct async *async = data;
> return async->proc(async->fd_for_proc, async->data);
> }
> -#endif
> +#endif /* __MINGW32__ || _MSC_VER */I do not think this is necessary. There are only 5 lines wrapped into those #ifdef guards, the developer should be able to see that far.
Ciao, Dscho