[Home]

Summary:ASTERISK-05628: [patch] Goto with parens does not compile correctly.
Reporter:edgreenberg (edgreenberg)Labels:
Date Opened:2005-11-17 11:02:54.000-0600Date Closed:2006-07-06 11:15:42
Priority:MinorRegression?No
Status:Closed/CompleteComponents:PBX/pbx_ael
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) aelgoto.patch
Description:In ael,

goto context|extension|priority;

compiles to
Goto(context|extension|priority)
which is fine, but

Goto(context|extension|priority);

compiles to
Goto((context|extension|priority))
which, at runtime, tries to go to (context.

This creates a subtle bug with no warnings at compile time.

I'm not sure if the author of pbx_ael considers this a bug, but it bit me, so I thought I'd get it on record.

Comments:By: Russell Bryant (russell) 2005-11-21 17:08:13.000-0600

This patch will strip the parenthesis if the first and last character of the application arguments are '(' and ')', respectively.

By: Tilghman Lesher (tilghman) 2005-12-21 12:25:24.000-0600

Fixed in 1.2, merged to trunk.