List Info

Thread: Re: tilghman: trunk r76703 - in /trunk: ./ apps/ channels/ funcs/




Re: tilghman: trunk r76703 - in /trunk: ./ apps/ channels/ funcs/
country flaguser name
Sweden
2007-07-28 16:12:16
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=7
6703 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]): Thisn"

Or am i missing the point?

/niklas

_______________________________________________
--Bandwidth and Colocation Provided by http://www.api-digital.c
om--

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

Re: tilghman: trunk r76703 - in /trunk: ./ apps/ channels/ funcs/
country flaguser name
United States
2007-07-29 00:57:37
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=7
6703 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]):
> Thisn"
>
> 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

_______________________________________________
--Bandwidth and Colocation Provided by http://www.api-digital.c
om--

asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
   http://lists.digium.com/mailman/listinfo/asterisk-dev

[1-2]

about | contact  Other archives ( Real Estate discussion Medical topics )