[PATCH 2/3] nptl: fix compiler warning due to unused-but-set-variable

Carmelo AMOROSO carmelo.amoroso at st.com
Mon Mar 26 08:01:58 UTC 2012


On 14/03/2012 16.41, Carmelo AMOROSO wrote:
> gcc 4.6.x raises useful warnings due to unused-but-set-variable,
> This patch fixes the following one by using properly preprocessor
> guard
> 
> libpthread/nptl/sysdeps/pthread/pthread_cond_timedwait.c: In function '__pthread_cond_timedwait':
> libpthread/nptl/sysdeps/pthread/pthread_cond_timedwait.c:103:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
> 
> Signed-off-by: Carmelo Amoroso <carmelo.amoroso at st.com>
> ---
>  .../nptl/sysdeps/pthread/pthread_cond_timedwait.c  |    6 ++++--
>  1 files changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/libpthread/nptl/sysdeps/pthread/pthread_cond_timedwait.c b/libpthread/nptl/sysdeps/pthread/pthread_cond_timedwait.c
> index 7f36753..92c6f94 100644
> --- a/libpthread/nptl/sysdeps/pthread/pthread_cond_timedwait.c
> +++ b/libpthread/nptl/sysdeps/pthread/pthread_cond_timedwait.c
> @@ -100,8 +100,10 @@ __pthread_cond_timedwait (
>        {
>  #ifdef __NR_clock_gettime
>  	INTERNAL_SYSCALL_DECL (err);
> -	int ret;
> -	ret = INTERNAL_SYSCALL (clock_gettime, err, 2,
> +# ifndef __ASSUME_POSIX_TIMERS
> +	int ret =
> +# endif
> +	INTERNAL_SYSCALL (clock_gettime, err, 2,
>  				(cond->__data.__nwaiters
>  				 & ((1 << COND_NWAITERS_SHIFT) - 1)),
>  				&rt);

Applied.

Carmelo


More information about the uClibc mailing list