[Asterisk-code-review] func_lookup: Fork of HINT function (asterisk[master])

Joshua Colp asteriskteam at digium.com
Mon Jul 19 17:52:12 CDT 2021


Attention is currently required from: N A.
Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/16075 )

Change subject: func_lookup: Fork of HINT function
......................................................................


Patch Set 4:

(1 comment)

Patchset:

PS4: 
> What you're saying sounds like it would work, but it also sounds like reinventing the wheel, and a b […]
The values can be substituted and done just like normal, there's nothing fancy there and there are PBX primitives to do exactly that and it's even done elsewhere in some cases.

As for your desire to have it in the dialplan, I disagree. I believe it should not be present. The way you use Gosub is not common, and just because it is possible to use it that way does not mean that we should continue to extend the dialplan in the direction you want with this.

As it is, this is not going to be accepted.



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/16075
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Iad81019689674c9f4ac77d235f5d7234adbb1432
Gerrit-Change-Number: 16075
Gerrit-PatchSet: 4
Gerrit-Owner: N A <mail at interlinked.x10host.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-Reviewer: Sarah Autumn <sarah at endlesstemple.org>
Gerrit-Attention: N A <mail at interlinked.x10host.com>
Gerrit-Comment-Date: Mon, 19 Jul 2021 22:52:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: N A <mail at interlinked.x10host.com>
Comment-In-Reply-To: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210719/938e3f17/attachment.html>


More information about the asterisk-code-review mailing list