On 5 Nov 2004, Daniel Kegel <[EMAIL PROTECTED]> wrote:
> Oh, I see now. It's the line after Jean's patch where
> distcc checks errno:
>
> while ((connect(fd, sa, salen) == -1) && errno == EINTR)
> ;
>
> if (errno && errno != EINPROGRESS) {
>
> So Jean's patch is right. It might be cleaner to avoid
> setting errno, though, and write this all as
>
> int failed = 0;
> do
> failed = connect(fd, sa, salen);
> while (failed && errno == EINTR);
>
> if (failed && errno != EINPROGRESS) {Or you could even omit the initial assignment. Thanks, guys. There are actually several of these. I will do a 2.18.2 soon, plus a fix for bugs.debian.org/280399. -- Martin
signature.asc
Description: Digital signature
__ distcc mailing list http://distcc.samba.org/ To unsubscribe or change options: http://lists.samba.org/mailman/listinfo/distcc
