From: Ian Jackson Date: Sun, 12 Oct 2014 17:00:35 +0000 (+0100) Subject: src/: Rename `udpsocket' and `nudp' (style) X-Git-Tag: adns-1.5.0-rc0~38 X-Git-Url: https://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=adns.git;a=commitdiff_plain;h=97a937c8b7052d9ad87bb20394e1753a65fa3f37 src/: Rename `udpsocket' and `nudp' (style) I prefer plurals for arrays and the count should have a near-identical name. Signed-off-by: Ian Jackson --- diff --git a/src/check.c b/src/check.c index c26dbdc..e9d6771 100644 --- a/src/check.c +++ b/src/check.c @@ -78,7 +78,7 @@ static void checkc_global(adns_state ads) { const struct sortlist *sl; int i; - assert(ads->udpsocket >= 0); + assert(ads->udpsockets >= 0); for (i=0; insortlist; i++) { sl= &ads->sortlist[i]; diff --git a/src/event.c b/src/event.c index ce5276f..1652268 100644 --- a/src/event.c +++ b/src/event.c @@ -316,8 +316,8 @@ int adns__pollfds(adns_state ads, struct pollfd pollfds_buf[MAX_POLLFDS]) { assert(MAX_POLLFDS == MAXUDP + 1); - for (i=0; inudp; i++) - ADD_POLLFD(ads->udpsocket[i].fd, POLLIN); + for (i=0; inudpsockets; i++) + ADD_POLLFD(ads->udpsockets[i].fd, POLLIN); switch (ads->tcpstate) { case server_disconnected: @@ -396,8 +396,8 @@ int adns_processreadable(adns_state ads, int fd, const struct timeval *now) { default: abort(); } - for (i=0; inudp; i++) { - udp= &ads->udpsocket[i]; + for (i=0; inudpsockets; i++) { + udp= &ads->udpsockets[i]; if (fd == udp->fd) { for (;;) { udpaddrlen= sizeof(udpaddr); diff --git a/src/internal.h b/src/internal.h index 28ef987..5d70ead 100644 --- a/src/internal.h +++ b/src/internal.h @@ -372,8 +372,8 @@ struct adns__state { struct query_queue udpw, tcpw, childw, output, intdone; adns_query forallnext; int nextid, tcpsocket; - struct udpsocket { int af; int fd; } udpsocket[MAXUDP]; - int nudp; + struct udpsocket { int af; int fd; } udpsockets[MAXUDP]; + int nudpsockets; vbuf tcpsend, tcprecv; int nservers, nsortlist, nsearchlist, searchndots, tcpserver, tcprecv_skip; enum adns__tcpstate { diff --git a/src/setup.c b/src/setup.c index 09f044b..dce6e94 100644 --- a/src/setup.c +++ b/src/setup.c @@ -580,7 +580,7 @@ static int init_begin(adns_state *ads_r, adns_initflags flags, LIST_INIT(ads->intdone); ads->forallnext= 0; ads->nextid= 0x311f; - ads->nudp= 0; + ads->nudpsockets= 0; ads->tcpsocket= -1; adns__vbuf_init(&ads->tcpsend); adns__vbuf_init(&ads->tcprecv); @@ -618,16 +618,16 @@ static int init_finish(adns_state ads) { } proto= getprotobyname("udp"); if (!proto) { r= ENOPROTOOPT; goto x_free; } - ads->nudp= 0; + ads->nudpsockets= 0; for (i=0; inservers; i++) { if (adns__udpsocket_by_af(ads, ads->servers[i].addr.sa.sa_family)) continue; - assert(ads->nudp < MAXUDP); - udp= &ads->udpsocket[ads->nudp]; + assert(ads->nudpsockets < MAXUDP); + udp= &ads->udpsockets[ads->nudpsockets]; udp->af= ads->servers[i].addr.sa.sa_family; udp->fd= socket(udp->af,SOCK_DGRAM,proto->p_proto); if (udp->fd < 0) { r= errno; goto x_free; } - ads->nudp++; + ads->nudpsockets++; r= adns__setnonblock(ads,udp->fd); if (r) { r= errno; goto x_closeudp; } } @@ -635,7 +635,7 @@ static int init_finish(adns_state ads) { return 0; x_closeudp: - for (j=0; jnudp; j++) close(ads->udpsocket[j].fd); + for (j=0; jnudpsockets; j++) close(ads->udpsockets[j].fd); x_free: free(ads); return r; @@ -752,7 +752,7 @@ void adns_finish(adns_state ads) { else if (ads->intdone.head) adns__cancel(ads->output.head); else break; } - for (i=0; inudp; i++) close(ads->udpsocket[i].fd); + for (i=0; inudpsockets; i++) close(ads->udpsockets[i].fd); if (ads->tcpsocket >= 0) close(ads->tcpsocket); adns__vbuf_free(&ads->tcpsend); adns__vbuf_free(&ads->tcprecv); diff --git a/src/transmit.c b/src/transmit.c index be72cf2..853bb5c 100644 --- a/src/transmit.c +++ b/src/transmit.c @@ -242,8 +242,8 @@ static void query_usetcp(adns_query qu, struct timeval now) { struct udpsocket *adns__udpsocket_by_af(adns_state ads, int af) { int i; - for (i=0; inudp; i++) - if (ads->udpsocket[i].af == af) return &ads->udpsocket[i]; + for (i=0; inudpsockets; i++) + if (ads->udpsockets[i].af == af) return &ads->udpsockets[i]; return 0; }