[Asterisk-code-review] res pjproject: Add module providing pjproject logging and u... (asterisk[13])

George Joseph asteriskteam at digium.com
Wed Jan 20 07:16:33 CST 2016


Hello Richard Mudgett, Anonymous Coward #1000019,

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

    https://gerrit.asterisk.org/2041

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

Change subject: res_pjproject:  Add module providing pjproject logging and utils
......................................................................

res_pjproject:  Add module providing pjproject logging and utils

res_pjsip_log_forwarder has been renamed to res_pjproject
and enhanced as follows:

As a follow-on to the recent 'Add CLI "pjsip show buildopts"' patch,
a new ast_pjproject_get_buildopt function has been added.  It
allows the caller to get the value of one of the buildopts.

The initial use case is retrieving the runtime value of
PJ_MAX_HOSTNAME to insure we don't send a hostname greater
than pjproject can handle.  Since it can differ between
the version of pjproject that Asterisk was compiled against
and the version of pjproject that Asterisk is running against,
we can't use the PJ_MAX_HOSTNAME macro directly in Asterisk
source code.

Change-Id: Iab6e82fec3d7cf00c1cf6185c42be3e7569dee1e
---
M CHANGES
A include/asterisk/res_pjproject.h
M res/res_pjproject.c
A res/res_pjproject.exports.in
4 files changed, 170 insertions(+), 64 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/41/2041/2
-- 
To view, visit https://gerrit.asterisk.org/2041
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iab6e82fec3d7cf00c1cf6185c42be3e7569dee1e
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list