[Asterisk-code-review] stringfields: Update extended string fields for master only. (asterisk[master])

George Joseph asteriskteam at digium.com
Wed Apr 13 15:02:55 CDT 2016


Hello Anonymous Coward #1000019,

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

    https://gerrit.asterisk.org/2581

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

Change subject: stringfields:  Update extended string fields for master only.
......................................................................

stringfields:  Update extended string fields for master only.

In 13, the new ast_string_field_header structure had to be dynamically
allocated and assigned to a pointer in ast_string_field_mgr to preserve ABI
compatability.  In master, it can be converted to being a structure-in-place in
ast_string_field_mgr to eliminate the extra alloc and free calls.

Change-Id: Ia97c5345eec68717a15dc16fe2e6746ff2a926f4
---
M include/asterisk/stringfields.h
M main/stringfields.c
M tests/test_stringfields.c
3 files changed, 28 insertions(+), 62 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ia97c5345eec68717a15dc16fe2e6746ff2a926f4
Gerrit-PatchSet: 2
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Anonymous Coward #1000019



More information about the asterisk-code-review mailing list