[Asterisk-code-review] chan_dahdi: Change 999999 to INT_MAX to better reflect "no timeout" (asterisk[16])

Friendly Automation asteriskteam at digium.com
Wed Jan 22 07:48:43 CST 2020


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/asterisk/+/13667 )

Change subject: chan_dahdi: Change 999999 to INT_MAX to better reflect "no timeout"
......................................................................

chan_dahdi: Change 999999 to INT_MAX to better reflect "no timeout"

The no-entry timeout set to 999999 == 16⅔ minutes, change to INT_MAX
to match behavior of "no timeout" defined in comment.

ASTERISK-28702 #close

Change-Id: I4ea015986e061374385dba247b272f7aac60bf11
---
M channels/chan_dahdi.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Benjamin Keith Ford: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Friendly Automation: Approved for Submit



diff --git a/channels/chan_dahdi.c b/channels/chan_dahdi.c
index 8ffece7..a4e564d 100644
--- a/channels/chan_dahdi.c
+++ b/channels/chan_dahdi.c
@@ -9847,7 +9847,7 @@
 		/* If starting a threeway call, never timeout on the first digit so someone
 		   can use flash-hook as a "hold" feature */
 		if (p->subs[SUB_THREEWAY].owner)
-			timeout = 999999;
+			timeout = INT_MAX;
 		while (len < AST_MAX_EXTENSION-1) {
 			int is_exten_parking = 0;
 

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I4ea015986e061374385dba247b272f7aac60bf11
Gerrit-Change-Number: 13667
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Siplas <andrew at asiplas.net>
Gerrit-Reviewer: Benjamin Keith Ford <bford at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: Joshua Colp <jcolp at sangoma.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20200122/6bb877ef/attachment.html>


More information about the asterisk-code-review mailing list