[asterisk-dev] [Code Review] Correct ast_api_register/unregister_multiple() API calls

Kevin Fleming kpfleming at digium.com
Tue Nov 18 11:28:49 CST 2008



> On 2008-11-17 18:39:22, Terry Wilson wrote:
> > /trunk/res/res_agi.c, lines 2557-2568
> > <http://reviewboard.digium.com/r/59/diff/5/?file=1581#file1581line2557>
> >
> >     Does this return -1 need to go outside the for loop?

Yep, will fix.


- Kevin


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


On 2008-11-17 18:12:33, Kevin Fleming wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.digium.com/r/59/
> -----------------------------------------------------------
> 
> (Updated 2008-11-17 18:12:33)
> 
> 
> Review request for Asterisk Developers.
> 
> 
> Summary
> -------
> 
> Revision 77787 (which came from issue 10288 in Mantis) introduced a number of API changes in res_agi.c and agi.h. Included among these was the addition of ast_agi_register_multiple() and ast_agi_unregister_multiple(), which were supposedly clones of the ast_cli_entry variants.
> 
> However, neither of these functions actually checked for failures to register or unregister the individual entries they process, and they did not have the ability to report errors back to the caller either.
> 
> This patch would change these API calls to properly check for and return errors, and fallback in as graceful a way as can be done when failures occur. It also adds Doxygen documentation for them, something which was also lacking in the original revision but it supposed to be included whenever any new API call is added.
> 
> Changing this will be an API change in the 1.6 release series, but the API is broken and must be fixed. In this patch I have renamed UPGRADE.txt to UPGRADE-1.6.txt and started a new UPGRADE.txt; for trunk and 1.6.1, this will document changes *between* 1.6 releases that users must be aware of, and of course this API change will be documented there in those two branches. In the 1.6.0 branch, I could either do the same thing (so only 1.6.0 and 1.6.0.1 would have UPGRADE.txt being relative to 1.4, and future 1.6.0.x releases would be relative to previous releases), or add a special note to the existing UPGRADE.txt file. Comments welcome :-)
> 
> 
> Diffs
> -----
> 
>   /trunk/UPGRADE.txt 157286 
>   /trunk/apps/app_stack.c 157286 
>   /trunk/include/asterisk/agi.h 157286 
>   /trunk/res/res_agi.c 157286 
> 
> Diff: http://reviewboard.digium.com/r/59/diff
> 
> 
> Testing
> -------
> 
> Compile testing.
> 
> 
> Thanks,
> 
> Kevin
> 
>




More information about the asterisk-dev mailing list