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

[Matthew Fredrickson]

Matthew Fredrickson has posted comments on this change. ( https://gerrit.asterisk.org/8987 )

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


Patch Set 3:

> (4 comments)
 > 
 > All 4 are minor things that I don't really care about being changed
 > or not, everything looks fine to me!

I think I've got your concerns addressed.  Let me know if I missed anything.

Thanks!
Matt


-- 
To view, visit https://gerrit.asterisk.org/8987
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
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>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Comment-Date: Mon, 21 May 2018 16:03:23 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180521/0b35fcae/attachment.html>