[Home]

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-0600Date Closed:2008-01-15 15:55:17.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents: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