[Asterisk-code-review] app sendtext: Allow content types other than text/plain (asterisk[certified/13.21])

Matthew Fredrickson asteriskteam at digium.com
Mon Jun 4 14:19:14 CDT 2018


Matthew Fredrickson has posted comments on this change. ( https://gerrit.asterisk.org/9079 )

Change subject: app_sendtext:  Allow content types other than text/plain
......................................................................


Patch Set 1: Code-Review-1

(1 comment)

https://gerrit.asterisk.org/#/c/9079/1/main/channel.c
File main/channel.c:

https://gerrit.asterisk.org/#/c/9079/1/main/channel.c@4892
PS1, Line 4892: ast_begins_with(content_type, "text/") == 0
I think this needs to be changed to ast_begins_with(), without the "== 0".

Likewise below.  If I understood it correctly.



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

Gerrit-Project: asterisk
Gerrit-Branch: certified/13.21
Gerrit-MessageType: comment
Gerrit-Change-Id: I94a90cfee98b4bc8e22aa5c0b6afb7b862f979d9
Gerrit-Change-Number: 9079
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Matthew Fredrickson <creslin at digium.com>
Gerrit-Comment-Date: Mon, 04 Jun 2018 19:19:14 +0000
Gerrit-HasComments: Yes
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180604/b1beda1a/attachment.html>


More information about the asterisk-code-review mailing list