remove some duplicated code;

fix indentation on one line;
mark XXX some unreachable code;
mark XXX another place where we could reduce the nesting depth.



git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@31872 65c4cc65-6c06-0410-ace0-fbb531ad65f3

channels/chan_sip.c

@@ -11222,6 +11222,8 @@ static int handle_request_notify(struct sip_pvt *p, struct sip_request *req, str
 			ast_set_flag(&p->flags[0], SIP_NEEDDESTROY);	
 		return -1;
 	} else {
+		/* XXX reduce nesting depth */
+
 		/* Handle REFER notifications */
 
 		char buf[1024];
@@ -11321,6 +11323,7 @@ static int handle_request_notify(struct sip_pvt *p, struct sip_request *req, str
 		return res;
 	};
 
+	/* XXX hey, we never reach this code! */
 	/* THis could be voicemail notification */
 	transmit_response(p, "200 OK", req);
 	if (!p->lastinvite) 
@@ -11935,22 +11938,21 @@ static int handle_request_invite(struct sip_pvt *p, struct sip_request *req, int
 		}
 	} else {
 		if (p && !ast_test_flag(&p->flags[0], SIP_NEEDDESTROY)) {
-			if (!p->jointcapability) {
+			const char *msg;
-				if (ast_test_flag(req, SIP_PKT_IGNORE))
+
-					transmit_response(p, "488 Not Acceptable Here (codec error)", req);
+			if (!p->jointcapability)
-				else
+				msg = "488 Not Acceptable Here (codec error)";
-					transmit_response_reliable(p, "488 Not Acceptable Here (codec error)", req);
+			else {
-				ast_set_flag(&p->flags[0], SIP_NEEDDESTROY);	
-			} else {
 				ast_log(LOG_NOTICE, "Unable to create/find SIP channel for this INVITE\n");
+				msg = "503 Unavailable";
+			}
 			if (ast_test_flag(req, SIP_PKT_IGNORE))
-					transmit_response(p, "503 Unavailable", req);
+				transmit_response(p, msg, req);
 			else
-					transmit_response_reliable(p, "503 Unavailable", req);
+				transmit_response_reliable(p, msg, req);
 			ast_set_flag(&p->flags[0], SIP_NEEDDESTROY);	
 		}
 	}
-	}
 	return res;
 }