apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lucian Adrian Grijincu" <lucian.griji...@gmail.com>
Subject Re: svn commit: r655138 - /apr/apr/trunk/poll/unix/select.c
Date Sat, 10 May 2008 19:06:18 GMT
On Sat, May 10, 2008 at 9:52 PM,  <fielding@apache.org> wrote:
> Author: fielding
> Date: Sat May 10 11:52:28 2008
> New Revision: 655138
>
> URL: http://svn.apache.org/viewvc?rev=655138&view=rev
> Log:
> silence warning about assignment expression
>
> Modified:
>    apr/apr/trunk/poll/unix/select.c
>
> Modified: apr/apr/trunk/poll/unix/select.c
> URL: http://svn.apache.org/viewvc/apr/apr/trunk/poll/unix/select.c?rev=655138&r1=655137&r2=655138&view=diff
> ==============================================================================
> --- apr/apr/trunk/poll/unix/select.c (original)
> +++ apr/apr/trunk/poll/unix/select.c Sat May 10 11:52:28 2008
> @@ -520,7 +520,7 @@
>             j++;
>         }
>     }
> -    if ((*num) = j)
> +    if (((*num) = j) != 0)
wouldn't just another set of parentheses fix it?
if (((*num) = j))

>         rv = APR_SUCCESS;
>
>     if (descriptors)
>
>
>



-- 
Lucian

Mime
View raw message