From d96548ab3ce3e16a179f04d215ce2db8e67c2881 Mon Sep 17 00:00:00 2001
From: Steve Murphy <murf@digium.com>
Date: Fri, 24 Aug 2007 18:52:15 +0000
Subject: [PATCH] From a complaint by jmls, I realize that the message in
 cdr_disposition is unnecessary. To get failure disposition, just return -1;
 no use having more than one case do that.

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@80789 65c4cc65-6c06-0410-ace0-fbb531ad65f3
---
 main/cdr.c | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/main/cdr.c b/main/cdr.c
index b81be5711d..24809c6511 100644
--- a/main/cdr.c
+++ b/main/cdr.c
@@ -740,27 +740,23 @@ void ast_cdr_noanswer(struct ast_cdr *cdr)
 	}
 }
 
+/* everywhere ast_cdr_disposition is called, it will call ast_cdr_failed() 
+   if ast_cdr_disposition returns a non-zero value */
+
 int ast_cdr_disposition(struct ast_cdr *cdr, int cause)
 {
 	int res = 0;
 
 	for (; cdr; cdr = cdr->next) {
-		switch(cause) {
+		switch(cause) {  /* handle all the non failure, busy cases, return 0 not to set disposition,
+							return -1 to set disposition to FAILED */
 		case AST_CAUSE_BUSY:
 			ast_cdr_busy(cdr);
 			break;
-		case AST_CAUSE_FAILURE:
-		case AST_CAUSE_NORMAL_CIRCUIT_CONGESTION:
-			ast_cdr_failed(cdr);
-			break;
 		case AST_CAUSE_NORMAL:
 			break;
-		case AST_CAUSE_NOTDEFINED:
-			res = -1;
-			break;
 		default:
 			res = -1;
-			ast_log(LOG_WARNING, "Cause (%d) not handled\n", cause);
 		}
 	}
 	return res;