[Asterisk-code-review] netsock2: Add ast sockaddr resolve first af to netsock2 publ... (asterisk[master])

Matthew Fredrickson asteriskteam at digium.com
Mon May 21 11:02:53 CDT 2018


Hello Benjamin Keith Ford, Jenkins2, Joshua Colp, 

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

    https://gerrit.asterisk.org/8987

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

Change subject: netsock2: Add ast_sockaddr_resolve_first_af to netsock2 public API
......................................................................

netsock2: Add ast_sockaddr_resolve_first_af to netsock2 public API

This function originally was used in chan_sip to enable some simplifying
assumptions and eventually was copy and pasted into res_pjsip_logger and
res_hep.  Since it's replicated in three places, it's probably best to
move it into the public netsock2 API for these modules to use.

Change-Id: Id52e23be885601c51d70259f62de1a5e59d38d04
---
M channels/chan_sip.c
M include/asterisk/netsock2.h
M main/netsock2.c
M res/res_hep.c
M res/res_pjsip_logger.c
5 files changed, 59 insertions(+), 72 deletions(-)


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

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id52e23be885601c51d70259f62de1a5e59d38d04
Gerrit-Change-Number: 8987
Gerrit-PatchSet: 3
Gerrit-Owner: Matthew Fredrickson <creslin at digium.com>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180521/604c4f9c/attachment.html>


More information about the asterisk-code-review mailing list