various devicestate fixes (issue #5081, take two)

git-svn-id: http://svn.digium.com/svn/asterisk/trunk@6496 f38db490-d61c-443f-a65b-d21fe96a405b

1 files changed, 19 insertions, 28 deletions

diff --git a/channel.c b/channel.c
index 7e63bbb2a..34ae2bf7d 100755
--- a/channel.c
+++ b/channel.c
@@ -2808,34 +2808,25 @@ void ast_set_callerid(struct ast_channel *chan, const char *callerid, const char
 
 int ast_setstate(struct ast_channel *chan, int state)
 {
-	if (chan->_state != state) {
-		int oldstate = chan->_state;
-		chan->_state = state;
-		if (oldstate == AST_STATE_DOWN) {
-			ast_device_state_changed(chan->name);
-			manager_event(EVENT_FLAG_CALL, "Newchannel",
-				"Channel: %s\r\n"
-				"State: %s\r\n"
-				"CallerID: %s\r\n"
-				"CallerIDName: %s\r\n"
-				"Uniqueid: %s\r\n",
-				chan->name, ast_state2str(chan->_state), 
-				chan->cid.cid_num ? chan->cid.cid_num : "<unknown>", 
-				chan->cid.cid_name ? chan->cid.cid_name : "<unknown>", 
-				chan->uniqueid);
-		} else {
-			manager_event(EVENT_FLAG_CALL, "Newstate", 
-				"Channel: %s\r\n"
-				"State: %s\r\n"
-				"CallerID: %s\r\n"
-				"CallerIDName: %s\r\n"
-				"Uniqueid: %s\r\n",
-				chan->name, ast_state2str(chan->_state), 
-				chan->cid.cid_num ? chan->cid.cid_num : "<unknown>", 
-				chan->cid.cid_name ? chan->cid.cid_name : "<unknown>", 
-				chan->uniqueid);
-		}
-	}
+	int oldstate = chan->_state;
+
+	if (oldstate == state)
+		return 0;
+
+	chan->_state = state;
+	ast_device_state_changed(chan->name);
+	manager_event(EVENT_FLAG_CALL,
+		      (oldstate == AST_STATE_DOWN) ? "Newchannel" : "Newstate",
+		      "Channel: %s\r\n"
+		      "State: %s\r\n"
+		      "CallerID: %s\r\n"
+		      "CallerIDName: %s\r\n"
+		      "Uniqueid: %s\r\n",
+		      chan->name, ast_state2str(chan->_state), 
+		      chan->cid.cid_num ? chan->cid.cid_num : "<unknown>", 
+		      chan->cid.cid_name ? chan->cid.cid_name : "<unknown>", 
+		      chan->uniqueid);
+
 	return 0;
 }