[svn-commits] mjordan: trunk r395135 - /trunk/main/channel.c

[SVN commits to the Digium repositories]

Author: mjordan
Date: Tue Jul 23 09:49:53 2013
New Revision: 395135

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=395135
Log:
Kill the zombies

In previous versions of Asterisk, the zombies roamed freely,
unchecked and uncontrolled. They ravaged Asterisk systems with
their biting and their nashing and their pointy teeth.

Sometimes, you couldn't even hang them up.

Now, zombies are rare. They still *technically* exist in certain
places, but they are controlled. Kind of like a zombie zoo: you can
see them, but you can't touch them, and they can't touch you.
Bring your kids!

Because zombies are now population controlled with a very short lifespan,
there's no reason to rename the channels to '%s<ZOMBIE>'. The channels
are guaranteed to die off quickly; the rename really is just confusing
at this point.

This patch finally removes the renaming. On the plus side: this made
my life easier in CDRs during call pickup and attended transfers to
an Asterisk application. It will make other folks lives easier as well!

Review: https://reviewboard.astierks.org/r/2690/

(closes issue ASTERISK-21699)
Reported by: Matt Jordan


Modified:
    trunk/main/channel.c

Modified: trunk/main/channel.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/channel.c?view=diff&rev=395135&r1=395134&r2=395135
==============================================================================
--- trunk/main/channel.c (original)
+++ trunk/main/channel.c Tue Jul 23 09:49:53 2013
@@ -6518,10 +6518,6 @@
 	ast_copy_string(tmp_name, ast_channel_name(clonechan), sizeof(tmp_name));
 	ast_channel_name_set(clonechan, ast_channel_name(original));
 	ast_channel_name_set(original, tmp_name);
-
-	/* Now zombify the clonechan. This gets a real rename event. */
-	snprintf(tmp_name, sizeof(tmp_name), "%s<ZOMBIE>", ast_channel_name(clonechan)); /* quick, hide the brains! */
-	__ast_change_name_nolink(clonechan, tmp_name);
 
 	/* Swap the technologies */
 	t = ast_channel_tech(original);