[Asterisk-code-review] app macro: Consider '~~s~~' as a macro start extension. (asterisk[master])

Joshua Colp asteriskteam at digium.com
Thu Aug 25 08:46:02 CDT 2016


Hello Mark Michelson, Anonymous Coward #1000019,

I'd like you to reexamine a change.  Please visit

    https://gerrit.asterisk.org/3578

to look at the new patch set (#4).

Change subject: app_macro: Consider '~~s~~' as a macro start extension.
......................................................................

app_macro: Consider '~~s~~' as a macro start extension.

As described in issue ASTERISK-26282 the ael parser creates macros with
extension '~~s~~'. app_macro searches only for extension 's' so the
created extension cannot be found. with this patch app_macro searches
for both extensions and performs the right extension.

ASTERISK-26282 #close

Change-Id: I939aa2a694148cc1054dd75ec0c47c47f47c90fb
---
M apps/app_macro.c
1 file changed, 27 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/78/3578/4
-- 
To view, visit https://gerrit.asterisk.org/3578
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I939aa2a694148cc1054dd75ec0c47c47f47c90fb
Gerrit-PatchSet: 4
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: chris de rock <chris at derock.de>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Mark Michelson <mmichelson at digium.com>
Gerrit-Reviewer: chris de rock <chris at derock.de>



More information about the asterisk-code-review mailing list