[Asterisk-code-review] res pjsip rfc3326.c: Account for more than one 'Reason' header (asterisk[13])

[Richard Mudgett]

Richard Mudgett has posted comments on this change. ( https://gerrit.asterisk.org/8522 )

Change subject: res_pjsip_rfc3326.c: Account for more than one 'Reason' header
......................................................................


Patch Set 5: Code-Review+1

> Uploaded patch set 5.

I simplified the loop and fixed a corner case to keep looping rather than abort.  I also made str_reason static so it doesn't get re-initialized every time it is called since it is a constant.


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

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: I0aa59a54735c6d20b95c54db1bd095dbf93e7adf
Gerrit-Change-Number: 8522
Gerrit-PatchSet: 5
Gerrit-Owner: Ross Beer <ross.beer at voicehost.co.uk>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Richard Mudgett <rmudgett at digium.com>
Gerrit-Reviewer: Sean Bright <sean.bright at gmail.com>
Gerrit-Comment-Date: Wed, 14 Mar 2018 23:49:35 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180314/4c7cd6a6/attachment-0001.html>