apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Maxime Petazzoni <maxime.petazz...@bulix.org>
Subject Re: svn commit: r411637 - /apr/apr/trunk/network_io/unix/sendrecv.c
Date Mon, 05 Jun 2006 09:52:26 GMT
Hi,

> -    int rv, i;
> +    int rv;
> +#if defined(__FreeBSD_version) && __FreeBSD_version < 460001
> +    int i;
> +#endif

I don't want to mess up with APR development too much, but why don't
you declare i in the else if block just a few lines after ? i is
obviously only used there, and this block is already in a similar
pre-compilator conditional block. Would improve code readability I
guess.

My 2 ยข,
- Sam
-- 
Maxime Petazzoni (http://www.bulix.org)
 -- gone crazy, back soon. leave message.

Mime
View raw message