Hi,

ACK for the idea, but this...

On Thu, Feb 18, 2010 at 09:23:48PM +0100, David Sommerseth wrote:
> -           /* choose address randomly, for basic load-balancing capability */
> -           ia.s_addr = *(in_addr_t *) (h->h_addr_list[get_random () % n]);
> +           ia.s_addr = *(in_addr_t *) (h->h_addr_list[0]);
>           }

... is sort of redundant, because the code further up already sets

      ia.s_addr = *(in_addr_t *) (h->h_addr_list[0]);

- in about line 202 in my copy of socket.c:

---------------- snip -------------
      ia.s_addr = *(in_addr_t *) (h->h_addr_list[0]);

      if (ia.s_addr)
        { 
          if (h->h_addr_list[1]) /* more than one address returned */

---------------- snip -------------

so the if() could be reduced even further, to just print the message.

gert


-- 
USENET is *not* the non-clickable part of WWW!
                                                           //www.muc.de/~gert/
Gert Doering - Munich, Germany                             g...@greenie.muc.de
fax: +49-89-35655025                        g...@net.informatik.tu-muenchen.de

Reply via email to