I just noticed this construct and thought it was
silly to have a bunch of case statements with duplicated code in each case. Instead, just use the built-in fallthrough capability of case statements and reduce the code to a single instance git-svn-id: http://svn.digium.com/svn/asterisk/trunk@152879 f38db490-d61c-443f-a65b-d21fe96a405b
This commit is contained in:
parent
0d2d156f1d
commit
71dceefced
|
@ -16604,18 +16604,9 @@ static void handle_response(struct sip_pvt *p, int resp, char *rest, struct sip_
|
|||
switch(resp) {
|
||||
case 100: /* 100 Trying */
|
||||
case 101: /* 101 Dialog establishment */
|
||||
if (sipmethod == SIP_INVITE)
|
||||
handle_response_invite(p, resp, rest, req, seqno);
|
||||
break;
|
||||
case 183: /* 183 Session Progress */
|
||||
if (sipmethod == SIP_INVITE)
|
||||
handle_response_invite(p, resp, rest, req, seqno);
|
||||
break;
|
||||
case 180: /* 180 Ringing */
|
||||
if (sipmethod == SIP_INVITE)
|
||||
handle_response_invite(p, resp, rest, req, seqno);
|
||||
break;
|
||||
case 182: /* 182 Queued */
|
||||
case 182: /* 182 Queued */
|
||||
if (sipmethod == SIP_INVITE)
|
||||
handle_response_invite(p, resp, rest, req, seqno);
|
||||
break;
|
||||
|
|
Reference in New Issue