[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