On Tue, Jan 10, 2017 at 3:34 PM, David Sommerseth <dav...@openvpn.net>
wrote:

> Bascially removes two independent #ifdef ENABLE_MANAGEMENT blocks into
> a single block, which makes the logic flow more easy to read.
>
> Signed-off-by: David Sommerseth <dav...@openvpn.net>
> Cc: Selva Nair <selva.n...@gmail.com>
> ---
>  src/openvpn/init.c | 6 +-----
>  1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/src/openvpn/init.c b/src/openvpn/init.c
> index 4786232..5296345 100644
> --- a/src/openvpn/init.c
> +++ b/src/openvpn/init.c
> @@ -414,11 +414,7 @@ next_connection_entry(struct context *c)
>                  break;
>              }
>          }
> -        else
> -#endif
> -
> -#ifdef ENABLE_MANAGEMENT
> -        if (ce_defined && management && management_query_proxy_enabled
> (management))
> +        else if (ce_defined && management &&
> management_query_proxy_enabled(management))
>          {
>              ce_defined = ce_management_query_proxy(c);
>              if (IS_SIG(c))
>

One more ifdef gone, and good riddance!

ACK.
------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today. http://sdm.link/xeonphi
_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to