oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

client: added function client_by_fd()

Browse Source 
The code becomes less complex and more readable when we move this
linear search into a separate mini function.
This commit is contained in:
Max Kellermann 2008-08-28 20:03:06 +02:00
parent d15e1e09a2
commit a091c148e6
1 changed files with 21 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

34
src/client.c
View File

@ -689,30 +689,38 @@ static void client_write_deferred(struct client *client)
}
}
int client_print(int fd, const char *buffer, size_t buflen)
static struct client *client_by_fd(int fd)
{
static unsigned int i;
assert(fd >= 0);
if (i < client_max_connections && clients[i].fd >= 0 &&
clients[i].fd == fd)
return &clients[i];
for (i = 0; i < client_max_connections; i++)
if (clients[i].fd == fd)
return &clients[i];
return NULL;
}
int client_print(int fd, const char *buffer, size_t buflen)
{
size_t copylen;
struct client *client;
assert(fd >= 0);
if (i >= client_max_connections ||
clients[i].fd < 0 || clients[i].fd != fd) {
for (i = 0; i < client_max_connections; i++) {
if (clients[i].fd == fd)
break;
}
if (i == client_max_connections)
return -1;
}
client = client_by_fd(fd);
if (client == NULL)
return -1;
/* if fd isn't found or client is going to be closed, do nothing */
if (clients[i].expired)
if (client->expired)
return 0;
client = clients + i;
while (buflen > 0 && !client->expired) {
size_t left;