[vlc-commits] network: remove useless vlc_object_waitpipe() call in net_Accept()
Rémi Denis-Courmont
git at videolan.org
Tue May 12 21:39:21 CEST 2015
vlc | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Sat May 9 22:41:23 2015 +0300| [7d28f8a24c75fd5ffcd393f047208a0d7f55a187] | committer: Rémi Denis-Courmont
network: remove useless vlc_object_waitpipe() call in net_Accept()
None of the call sites use the input thread and wait pipe.
> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=7d28f8a24c75fd5ffcd393f047208a0d7f55a187
---
src/network/tcp.c | 17 ++++-------------
1 file changed, 4 insertions(+), 13 deletions(-)
diff --git a/src/network/tcp.c b/src/network/tcp.c
index 98bb3fd..f999aaf 100644
--- a/src/network/tcp.c
+++ b/src/network/tcp.c
@@ -270,26 +270,23 @@ int net_AcceptSingle (vlc_object_t *obj, int lfd)
*/
int net_Accept (vlc_object_t *p_this, int *pi_fd)
{
- int evfd = vlc_object_waitpipe (p_this);
-
assert (pi_fd != NULL);
unsigned n = 0;
while (pi_fd[n] != -1)
n++;
- struct pollfd ufd[n + 1];
+ struct pollfd ufd[n];
/* Initialize file descriptor set */
- for (unsigned i = 0; i <= n; i++)
+ for (unsigned i = 0; i < n; i++)
{
- ufd[i].fd = (i < n) ? pi_fd[i] : evfd;
+ ufd[i].fd = pi_fd[i];
ufd[i].events = POLLIN;
}
- ufd[n].revents = 0;
for (;;)
{
- while (poll (ufd, n + (evfd != -1), -1) == -1)
+ while (poll (ufd, n, -1) == -1)
{
if (net_errno != EINTR)
{
@@ -316,12 +313,6 @@ int net_Accept (vlc_object_t *p_this, int *pi_fd)
pi_fd[n - 1] = sfd;
return fd;
}
-
- if (ufd[n].revents)
- {
- errno = EINTR;
- break;
- }
}
return -1;
}
More information about the vlc-commits
mailing list