[Asterisk-code-review] channel: Remove old epoll support and fixed max number of fi... (asterisk[master])

Joshua Colp asteriskteam at digium.com
Fri Mar 24 17:41:09 CDT 2017


Hello Mark Michelson, Anonymous Coward #1000019, Sean Bright, Corey Farrell,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/5283

to look at the new patch set (#3).

Change subject: channel: Remove old epoll support and fixed max number of file descriptors.
......................................................................

channel: Remove old epoll support and fixed max number of file descriptors.

This change removes the old epoll support which has not been used or
maintained in quite some time.

The fixed number of file descriptors on a channel has also been removed.
File descriptors are now contained in a growable vector. This can be
used like before by specifying a specific position to store a file
descriptor at or using a new API call, ast_channel_fd_add, which adds
a file descriptor to the channel and returns its position.

Tests have been added which cover the growing behavior of the vector
and the new API call.

ASTERISK-26885

Change-Id: I1a754b506c009b83dfdeeb08c2d2815db30ef928
---
M apps/app_dial.c
M apps/app_queue.c
M include/asterisk/channel.h
M main/channel.c
M main/channel_internal_api.c
M main/dial.c
A tests/test_channel.c
7 files changed, 201 insertions(+), 400 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/83/5283/3
-- 
To view, visit https://gerrit.asterisk.org/5283
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1a754b506c009b83dfdeeb08c2d2815db30ef928
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>



More information about the asterisk-code-review mailing list