apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rainer Jung <rainer.j...@kippdata.de>
Subject Re: svn commit: r1140899 - /apr/apr/trunk/crypto/apr_crypto_openssl.c
Date Wed, 29 Jun 2011 00:12:27 GMT
On 29.06.2011 01:15, rjung@apache.org wrote:
> Author: rjung
> Date: Tue Jun 28 23:15:58 2011
> New Revision: 1140899
> 
> URL: http://svn.apache.org/viewvc?rev=1140899&view=rev
> Log:
> Remove unused variable.
> 
> Modified:
>     apr/apr/trunk/crypto/apr_crypto_openssl.c
> 
> Modified: apr/apr/trunk/crypto/apr_crypto_openssl.c
> URL: http://svn.apache.org/viewvc/apr/apr/trunk/crypto/apr_crypto_openssl.c?rev=1140899&r1=1140898&r2=1140899&view=diff
> ==============================================================================
> --- apr/apr/trunk/crypto/apr_crypto_openssl.c (original)
> +++ apr/apr/trunk/crypto/apr_crypto_openssl.c Tue Jun 28 23:15:58 2011
> @@ -98,7 +98,6 @@ static apr_status_t crypto_shutdown(void
>  }
>  
>  static apr_status_t crypto_shutdown_helper(void *data) {
> -    apr_pool_t *pool = (apr_pool_t *) data;
>      return crypto_shutdown();
>  }

Note that I removed the unsed variable to prevent a compiler warning.

The code in 1.4.x and 1.5.x is different: it passes the pool along to
crypto_shutdown() which then doesn't use it. I guess we want the code to
be consistent, but I'm not sure, which way to prefer - pass the pool
to crypto_shutdown and ignore there (1.4, 1.5), or already dropp it in
crypto_shutdown_helper (trunk).

Regards,

Rainer

Mime
View raw message