[Asterisk-code-review] vector: Additional enhancements and fixes (asterisk[13])

George Joseph asteriskteam at digium.com
Thu May 7 11:03:50 CDT 2015


George Joseph has posted comments on this change.

Change subject: vector:  Additional enhancements and fixes
......................................................................


Patch Set 5:

> I think we can add two more APIs (for completeness):
 > - AST_VECTOR_RESET(): Remove all elements from vector (but not
 > freeing it, simply setting current to zero).
 > - AST_VECTOR_QSORT(cmp_callback): Sort vector elements using
 > `qsort`. (or make it generic to accept qsort, merge_sort, ...).

This patch is blocking another that I'm working on so let me address these in a follow-on patch.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I05e5e47fd02f61964be13b7e8942bab5d61b29cc
Gerrit-PatchSet: 5
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Y Ateya <y.ateya at starkbits.com>
Gerrit-HasComments: No



More information about the asterisk-code-review mailing list