Summary: | ASTERISK-05560: [patch] ast_app_separate_args, not ast_separate_app_args | ||
Reporter: | Tilghman Lesher (tilghman) | Labels: | |
Date Opened: | 2005-11-10 12:26:17.000-0600 | Date Closed: | 2008-01-15 15:55:17.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Documentation |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) 20051110__aasa_not_asaa.diff.txt | |
Description: | Confusion reigns. Let's fix it. | ||
Comments: | By: Jason Parker (jparker) 2005-11-10 12:35:34.000-0600 I think you're wrong with this one. This is from a CVS HEAD checkout on like 11/01/05. As you can see, only ast_separate_app_args is used. omni-asterisk asterisk # grep -r ast_app_separate_args * | wc -l 0 omni-asterisk asterisk # grep -r ast_separate_app_args * | wc -l 68 By: Tilghman Lesher (tilghman) 2005-11-10 12:37:23.000-0600 Good, because this is a checkout from a few minutes ago. ast_separate_app_args() is not defined anywhere. By: Tilghman Lesher (tilghman) 2005-11-10 12:40:19.000-0600 Okay, apparently the name got changed in the last iteration of app.c (1.82). ast_app_separate_args() is now the official version, and this patch fixes the last few stragglers. By: Russell Bryant (russell) 2005-11-10 18:17:47.000-0600 fixed in cvs head, thanks! By: Digium Subversion (svnbot) 2008-01-15 15:55:17.000-0600 Repository: asterisk Revision: 7050 U trunk/ChangeLog U trunk/apps/app_skel.c U trunk/doc/CODING-GUIDELINES U trunk/include/asterisk/app.h ------------------------------------------------------------------------ r7050 | russell | 2008-01-15 15:55:16 -0600 (Tue, 15 Jan 2008) | 2 lines issue ASTERISK-5560 ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=7050 |