[asterisk-dev] [Code Review] astobj2: Avoid using temporary objects + ao2_find() with OBJ_POINTER

Russell Bryant reviewboard at asterisk.org
Mon Apr 18 12:33:26 CDT 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1184/
-----------------------------------------------------------

(Updated 2011-04-18 12:33:25.935846)


Review request for Asterisk Developers.


Changes
-------

Updates based on Mark's review.


Summary
-------

There is a fairly common pattern making its way through the code base where we put a temporary object on the stack so we can call ao2_find() with OBJ_POINTER.  The purpose is so that it can be passed into the object hash function.  However, this really seems like a hack and potentially error prone.  This patch is a first stab at approach to avoid having to do that.

It adds a new flag, OBJ_CUSTOM, which can be used instead of OBJ_POINTER in these situations.  Then, the hash function can know whether it was given an object or some custom data to hash.

The patch also changes some uses of ao2_find() for iax2_user and iax2_peer objects to reflect how OBJ_CUSTOM would be used.


Diffs (updated)
-----

  /trunk/channels/chan_iax2.c 314080 
  /trunk/include/asterisk/astobj2.h 314080 
  /trunk/main/astobj2.c 314080 

Diff: https://reviewboard.asterisk.org/r/1184/diff


Testing
-------

It compiles.  I'm just throwing out the idea right now.  If people seem to think it makes sense, I will certainly try calls before committing anything.


Thanks,

Russell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20110418/2612d679/attachment.htm>


More information about the asterisk-dev mailing list