[Asterisk-code-review] Astobj2: Allow reference debugging to be enabled/disabled by... (asterisk[master])

Corey Farrell asteriskteam at digium.com
Fri Apr 24 14:49:04 CDT 2015


Hello Mark Michelson, Richard Mudgett,

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

    https://gerrit.asterisk.org/140

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

Change subject: Astobj2: Allow reference debugging to be enabled/disabled by config.
......................................................................

Astobj2: Allow reference debugging to be enabled/disabled by config.

* The REF_DEBUG compiler flag no longer has any effect on code that uses
  Astobj2.  It is used to determine if reference debugging is enabled by
  default.  Reference debugging can be enabled or disabled in asterisk.conf.
* Caller information is provided in logger errors for ao2 bad magic numbers.
* Optimizes AO2 by merging internal functions with the public counterpart.
  This was possible now that we no longer require a dual ABI.

ASTERISK-24974 #close
Reported by: Corey Farrell

Change-Id: Icf3552721fe999365ba8a8cf00a965aa6b897cc1
---
M UPGRADE.txt
M apps/app_queue.c
M build_tools/cflags.xml
M channels/chan_sip.c
M configs/samples/asterisk.conf.sample
M contrib/Makefile
M contrib/scripts/refcounter.py
M include/asterisk/astobj2.h
M include/asterisk/channel.h
M include/asterisk/format_cache.h
M include/asterisk/format_cap.h
M include/asterisk/options.h
M main/asterisk.c
M main/astobj2.c
M main/astobj2_container.c
M main/astobj2_container_private.h
M main/astobj2_hash.c
M main/astobj2_private.h
M main/astobj2_rbtree.c
M main/channel.c
M main/channel_internal_api.c
M main/format_cache.c
M main/format_cap.c
M main/loader.c
M res/res_musiconhold.c
25 files changed, 433 insertions(+), 1,020 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/40/140/5
-- 
To view, visit https://gerrit.asterisk.org/140
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Icf3552721fe999365ba8a8cf00a965aa6b897cc1
Gerrit-PatchSet: 5
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>



More information about the asterisk-code-review mailing list