[Asterisk-code-review] app dial: Enable early-media video (asterisk[13])

[Jenkins2]

Jenkins2 has submitted this change and it was merged. ( https://gerrit.asterisk.org/8398 )

Change subject: app_dial: Enable early-media video
......................................................................

app_dial: Enable early-media video

Certain applications (e.g. door-phone) require that also video is transmitted
before a call is accepted.

Change-Id: I9842e1dc2f6e1c2c49dc33fe615255007d2f821e
---
M UPGRADE.txt
M apps/app_dial.c
2 files changed, 10 insertions(+), 0 deletions(-)

Approvals:
  Kevin Harwell: Looks good to me, but someone else must approve
  Richard Mudgett: Looks good to me, but someone else must approve
  Joshua Colp: Looks good to me, approved
  Jenkins2: Approved for Submit



diff --git a/UPGRADE.txt b/UPGRADE.txt
index bbb083e..133e8a8 100644
--- a/UPGRADE.txt
+++ b/UPGRADE.txt
@@ -21,6 +21,14 @@
 === UPGRADE-12.txt  -- Upgrade info for 11 to 12
 ===========================================================
 
+From 13.20.0 to 13.21.0:
+
+app_dial
+------------------
+ * The Dial application now supports early-media video (in addition to
+   audio) on both the calling as well as the called party.
+   Be aware that this is a change in behavior.
+
 From 13.19.0 to 13.20.0:
 
 app_confbridge
diff --git a/apps/app_dial.c b/apps/app_dial.c
index a37ff77..ea7ddd1 100644
--- a/apps/app_dial.c
+++ b/apps/app_dial.c
@@ -1609,6 +1609,7 @@
 					break;
 				}
 				break;
+			case AST_FRAME_VIDEO:
 			case AST_FRAME_VOICE:
 			case AST_FRAME_IMAGE:
 				if (caller_entertained) {
@@ -1707,6 +1708,7 @@
 						ast_log(LOG_WARNING, "Unable to send URL\n");
 					}
 					break;
+				case AST_FRAME_VIDEO:
 				case AST_FRAME_VOICE:
 				case AST_FRAME_IMAGE:
 					if (!single || caller_entertained) {

-- 
To view, visit https://gerrit.asterisk.org/8398
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: merged
Gerrit-Change-Id: I9842e1dc2f6e1c2c49dc33fe615255007d2f821e
Gerrit-Change-Number: 8398
Gerrit-PatchSet: 4
Gerrit-Owner: Florian Floimair <f.floimair at commend.com>
Gerrit-Reviewer: Florian Floimair <f.floimair at commend.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180319/345080bd/attachment.html>