[asterisk-dev] [asterisk-commits] tilghman: trunk r76703 - in /trunk: ./ apps/ channels/ funcs/ include/asteris...

[Tilghman Lesher]

On Saturday 28 July 2007, Niklas Larsson wrote:
> On Mon, 23 Jul 2007 19:51:42 -0000, SVN commits to the Asterisk project 
wrote:
> > Author: tilghman
> > Date: Mon Jul 23 14:51:41 2007
> > New Revision: 76703
> >
> > URL: http://svn.digium.com/view/asterisk?view=rev&rev=76703 Log:
> > Merge the dialplan_aesthetics branch.  Most of this patch simply
> > converts applications using old methods of parsing arguments to
> > using the standard macros.  However, the big change is that the
> > really old way of specifying application and arguments separated by
> > a comma will no longer work (e.g. NoOp,foo|bar).  Instead, the way
> > that has been recommended since long before 1.0 will become the
> > only method available (e.g. NoOp(foo,bar).
>
> One humble question, should the | be used here?:
>
> static char *descrip_vm =
>  "  VoiceMail(mailbox[@context][&mailbox[@context]][...][|options]):
> This\n"
>
> Or am i missing the point?

No, you're quite right.  I'm sure that we'll continue to find various little
bits that I've missed in the conversion.  The big thing, though, is that
they're all corrected before trunk is branched for another release.

-- 
Tilghman