[Asterisk-code-review] main/frame: Add missing control frame names to ast_frame_subclass2str (asterisk[18])

Ivan Poddubny asteriskteam at digium.com
Mon Jan 25 10:38:52 CST 2021


Ivan Poddubny has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/15369 )


Change subject: main/frame: Add missing control frame names to ast_frame_subclass2str
......................................................................

main/frame: Add missing control frame names to ast_frame_subclass2str

Log proper control frame names instead of "Unknown control '14'", etc.

Change-Id: I1724f2f4d1b064b25a5c93a7da0cb03be5143935
---
M main/frame.c
1 file changed, 9 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/69/15369/1

diff --git a/main/frame.c b/main/frame.c
index 3a5ee91..52830cf 100644
--- a/main/frame.c
+++ b/main/frame.c
@@ -459,12 +459,21 @@
 		case AST_CONTROL_RADIO_UNKEY:
 			ast_copy_string(subclass, "Unkey Radio", slen);
 			break;
+		case AST_CONTROL_PROGRESS:
+			ast_copy_string(subclass, "Progress", slen);
+			break;
+		case AST_CONTROL_PROCEEDING:
+			ast_copy_string(subclass, "Proceeding", slen);
+			break;
 		case AST_CONTROL_HOLD:
 			ast_copy_string(subclass, "Hold", slen);
 			break;
 		case AST_CONTROL_UNHOLD:
 			ast_copy_string(subclass, "Unhold", slen);
 			break;
+		case AST_CONTROL_VIDUPDATE:
+			ast_copy_string(subclass, "Video update", slen);
+			break;
 		case AST_CONTROL_T38_PARAMETERS: {
 			char *message = "Unknown";
 			if (f->datalen != sizeof(struct ast_control_t38_parameters)) {

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

Gerrit-Project: asterisk
Gerrit-Branch: 18
Gerrit-Change-Id: I1724f2f4d1b064b25a5c93a7da0cb03be5143935
Gerrit-Change-Number: 15369
Gerrit-PatchSet: 1
Gerrit-Owner: Ivan Poddubny <ivan.poddubny at gmail.com>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20210125/0d44c026/attachment.html>


More information about the asterisk-code-review mailing list