[asterisk-dev] [Code Review] Create API for doing variable substitution with dynamic buffers

Russell Bryant russell at digium.com
Mon Mar 30 09:58:17 CDT 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/174/#review646
-----------------------------------------------------------


I know you've done a lot of updates to this code in your branch as you did further testing.  Could you post an updated patch here when you have time?

- Russell


On 2009-03-03 18:44:17, Tilghman Lesher wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/174/
> -----------------------------------------------------------
> 
> (Updated 2009-03-03 18:44:17)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> We currently have a variable substitution system that depends upon static buffers.  This has issues, both because it takes up 12k of stack each time through nested variable expressions, but also because in some newer cases, the 4k limit on the size of the buffer is no longer sufficient.  So this is a conversion that allows us to use dynamic string buffers in place of the old static buffers.
> 
> Additionally, note that this is a subset of the changes done, to make review of the changes easier.
> 
> 
> Diffs
> -----
> 
>   /trunk/apps/app_exec.c 180116 
>   /trunk/apps/app_macro.c 180116 
>   /trunk/apps/app_minivm.c 180116 
>   /trunk/apps/app_voicemail.c 180116 
>   /trunk/cdr/cdr_custom.c 180116 
>   /trunk/include/asterisk/ast_expr.h 180116 
>   /trunk/include/asterisk/pbx.h 180116 
>   /trunk/main/ast_expr2f.c 180116 
>   /trunk/main/pbx.c 180116 
>   /trunk/main/strings.c 180116 
> 
> Diff: http://reviewboard.digium.com/r/174/diff
> 
> 
> Testing
> -------
> 
> Extensive testing has been done on the new core API substitution call, though limited testing has only been done on the multiple places in the code where the new API has replaced the old substitution.
> 
> 
> Thanks,
> 
> Tilghman
> 
>




More information about the asterisk-dev mailing list