rdelaage
/
gmid
mirror of https://github.com/omar-polo/gmid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

simplify loop 

todo was initially there for an optimization: don't loop to MAX_USERS
when you know the upper limit is todo.
This commit is contained in:
Omar Polo 2021-01-08 16:59:50 +00:00
parent 4c4167393a
commit 16ace04d45
1 changed files with 2 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
gmid.c
View File

@ -702,7 +702,7 @@ goodbye(struct pollfd *pfd, struct client *c)
void
loop(struct tls *ctx, int sock)
{
int i, todo;
int i;
struct client clients[MAX_USERS];
struct pollfd fds[MAX_USERS];
@ -715,7 +715,7 @@ loop(struct tls *ctx, int sock)
fds[0].fd = sock;
for (;;) {
if ((todo = poll(fds, MAX_USERS, INFTIM)) == -1) {
if (poll(fds, MAX_USERS, INFTIM) == -1) {
if (errno == EINTR) {
warnx("connected clients: %d",
connected_clients);
@ -725,8 +725,6 @@ loop(struct tls *ctx, int sock)
}
for (i = 0; i < MAX_USERS; i++) {
assert(i < MAX_USERS);
if (fds[i].revents == 0)
continue;
@ -743,8 +741,6 @@ loop(struct tls *ctx, int sock)
}
}
todo--;
if (i == 0) { /* new client */
do_accept(sock, ctx, fds, clients);
continue;