From c0b9b259ca1b9121dab8210188a33933a5f9bede Mon Sep 17 00:00:00 2001 From: Mark Michelson Date: Thu, 30 Oct 2008 19:23:16 +0000 Subject: [PATCH] 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: https://origsvn.digium.com/svn/asterisk/trunk@152879 65c4cc65-6c06-0410-ace0-fbb531ad65f3 --- channels/chan_sip.c | 11 +---------- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/channels/chan_sip.c b/channels/chan_sip.c index 03eb7530c6..d66ba78169 100644 --- a/channels/chan_sip.c +++ b/channels/chan_sip.c @@ -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;