I fixed it in master.

Seriously though, the original patch has been submitted by an external 
contributor and *reviewed*. This is a three line patch that is obviously 
wrong.

Please take care when reviewing!

--
Tom.

On 18/11/15 09:43, Stefan Schmidt wrote:
> stefan pushed a commit to branch master.
>
> http://git.enlightenment.org/core/efl.git/commit/?id=2892e54d881f6bbe7d730539d2d713479db6d3e1
>
> commit 2892e54d881f6bbe7d730539d2d713479db6d3e1
> Author: Stefan Schmidt <[email protected]>
> Date:   Wed Nov 18 10:26:25 2015 +0100
>
>      Revert "eet: removing useless assignment variables."
>
>      This reverts commit 2a01c4517b5b97c2b988ce590d7dae913ec1f085.
>
>      Declaring these variable void breaks the build with disabled crypto.
>
>      Fixes T2855
> ---
>   src/lib/eet/eet_cipher.c | 6 +++---
>   1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/lib/eet/eet_cipher.c b/src/lib/eet/eet_cipher.c
> index bb85452..9bff406 100644
> --- a/src/lib/eet/eet_cipher.c
> +++ b/src/lib/eet/eet_cipher.c
> @@ -236,9 +236,9 @@ on_error:
>
>   # endif /* ifdef HAVE_GNUTLS */
>   #else
> -   void (certificate_file);
> -   void (private_key_file);
> -   void (cb);
> +   certificate_file = NULL;
> +   private_key_file = NULL;
> +   cb = NULL;
>   #endif /* ifdef HAVE_SIGNATURE */
>      return NULL;
>   }
>


------------------------------------------------------------------------------
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to