Hi,

On Tue, Oct 10, 2017 at 7:11 PM, <si...@rozman.si> wrote:

> From: Simon Rozman <si...@rozman.si>
>
> ---
>  src/openvpnserv/interactive.c | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/src/openvpnserv/interactive.c b/src/openvpnserv/interactive.c
> index 2019d38..7ff45b1 100644
> --- a/src/openvpnserv/interactive.c
> +++ b/src/openvpnserv/interactive.c
> @@ -1891,6 +1891,11 @@ FreeWaitHandles(LPHANDLE h)
>      free(h);
>  }
>
> +static BOOL
> +CmpHandle(LPVOID item, LPVOID hnd)
> +{
> +    return item == hnd;
> +}
>
>  VOID WINAPI
>  ServiceStartInteractive(DWORD dwArgc, LPTSTR *lpszArgv)
> @@ -1901,11 +1906,6 @@ ServiceStartInteractive(DWORD dwArgc, LPTSTR
> *lpszArgv)
>      list_item_t *threads = NULL;
>      PHANDLE handles = NULL;
>      DWORD handle_count;
> -    BOOL
> -    CmpHandle(LPVOID item, LPVOID hnd)
> -    {
> -        return item == hnd;
> -    }
>
>      service = RegisterServiceCtrlHandlerEx(interactive_service.name,
> ServiceCtrlInteractive, &status);
>      if (!service)
>


Well, nested function is a GNU extension so we can't blame MSVC here..
We had more of these and removed by a previous patch but it keeps coming
back -- this one from me (commit 600dd9a16fc)

ACK.
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to